Addressed review comments.

Author: zahiraam

clang/lib/Driver/ToolChains/Clang.cpp

@@ -5750,7 +5750,7 @@ void Clang::ConstructJob(Compilation &C, const JobAction &JA,
       StringRef Footer = D.getIntegrationFooter(Input.getBaseInput());
       if (types::getPreprocessedType(Input.getType()) != types::TY_INVALID &&
           !Args.hasArg(options::OPT_fno_sycl_use_footer) && !Footer.empty()) {
-        // Add the -include-internal-footer option to add the integration foote
+        // Add the -include-internal-footer option to add the integration footer
         CmdArgs.push_back("-include-internal-footer");
         CmdArgs.push_back(Args.MakeArgString(Footer));
         // When creating dependency information, filter out the generated

clang/lib/Frontend/PrintPreprocessedOutput.cpp

@@ -265,26 +265,22 @@ void PrintPPOutputPPCallbacks::WriteFooterContent(StringRef CodeFooter) {
   *OS << '\n';
 }
 
-static bool is_separator(char value) {
-  if (value == '\\')
-    return true;
-  return false;
-}
+static bool is_separator(char value) { return value == '\\'; }
 
 void PrintPPOutputPPCallbacks::WriteLineInfo(unsigned LineNo,
                                              const char *Extra,
                                              unsigned ExtraLen) {
   startNewLineIfNeeded();
 
-  // Emit #line directives or GNU line markers depending on what mode we're in.
   StringRef CurFilenameWithNoLeaningDotSlash =
-      llvm::sys::path::remove_leading_dotbackslah(CurFilename.str());
+      llvm::sys::path::remove_leading_dotbackslash(CurFilename.str());
   if ((CurFilenameWithNoLeaningDotSlash ==
               PP.getPreprocessorOpts().IncludeFooter) ||
       CurFilenameWithNoLeaningDotSlash ==
               PP.getPreprocessorOpts().IncludeHeader) {
     CurFilename = "<uninit>";
   }
+    // Emit #line directives or GNU line markers depending on what mode we're in.
     if (UseLineDirectives) {
       *OS << "#line" << ' ' << LineNo << ' ' << '"';
       OS->write_escaped(CurFilename);

llvm/include/llvm/Support/Path.h

@@ -205,7 +205,7 @@ StringRef remove_leading_dotslash(StringRef path, Style style = Style::native);
 ///
 /// @param path Input path.
 /// @result The cleaned-up \a path.
-StringRef remove_leading_dotbackslah(StringRef path);
+StringRef remove_leading_dotbackslash(StringRef path);
 
 /// In-place remove any './' and optionally '../' components from a path.
 ///

llvm/lib/Support/Path.cpp

@@ -711,7 +711,7 @@ StringRef remove_leading_dotslash(StringRef Path, Style style) {
   return Path;
 }
 
-StringRef remove_leading_dotbackslah(StringRef Path) {
+StringRef remove_leading_dotbackslash(StringRef Path) {
   // Remove leading ".\\".
   if (Path.size() > 2 && Path[0] == '.' && Path[1] == '\\')
     Path = Path.substr(2);