Rename the functions to use camel case.

Author: Tarun Prabhu

clang/lib/Driver/ToolChains/Clang.cpp

@@ -7686,8 +7686,8 @@ void Clang::ConstructJob(Compilation &C, const JobAction &JA,
   // By default, -gno-record-gcc-switches is set on and no recording.
   auto GRecordSwitches = false;
   auto FRecordSwitches = false;
-  if (ShouldRecordCommandLine(TC, Args, FRecordSwitches, GRecordSwitches)) {
-    auto FlagsArgString = RenderEscapedCommandLine(TC, Args);
+  if (shouldRecordCommandLine(TC, Args, FRecordSwitches, GRecordSwitches)) {
+    auto FlagsArgString = renderEscapedCommandLine(TC, Args);
     if (TC.UseDwarfDebugFlags() || GRecordSwitches) {
       CmdArgs.push_back("-dwarf-debug-flags");
       CmdArgs.push_back(FlagsArgString);
@@ -8687,10 +8687,10 @@ void ClangAs::ConstructJob(Compilation &C, const JobAction &JA,
 
     SmallString<256> Flags;
     const char *Exec = getToolChain().getDriver().getClangProgramPath();
-    EscapeSpacesAndBackslashes(Exec, Flags);
+    escapeSpacesAndBackslashes(Exec, Flags);
     for (const char *OriginalArg : OriginalArgs) {
       SmallString<128> EscapedArg;
-      EscapeSpacesAndBackslashes(OriginalArg, EscapedArg);
+      escapeSpacesAndBackslashes(OriginalArg, EscapedArg);
       Flags += " ";
       Flags += EscapedArg;
     }

clang/lib/Driver/ToolChains/CommonArgs.cpp

@@ -2961,7 +2961,7 @@ void tools::addMCModel(const Driver &D, const llvm::opt::ArgList &Args,
   }
 }
 
-void tools::EscapeSpacesAndBackslashes(const char *Arg,
+void tools::escapeSpacesAndBackslashes(const char *Arg,
                                        llvm::SmallVectorImpl<char> &Res) {
   for (; *Arg; ++Arg) {
     switch (*Arg) {
@@ -2976,7 +2976,7 @@ void tools::EscapeSpacesAndBackslashes(const char *Arg,
   }
 }
 
-const char *tools::RenderEscapedCommandLine(const ToolChain &TC,
+const char *tools::renderEscapedCommandLine(const ToolChain &TC,
                                             const llvm::opt::ArgList &Args) {
   const Driver &D = TC.getDriver();
   const char *Exec = D.getClangProgramPath();
@@ -2986,18 +2986,18 @@ const char *tools::RenderEscapedCommandLine(const ToolChain &TC,
     Arg->render(Args, OriginalArgs);
 
   llvm::SmallString<256> Flags;
-  EscapeSpacesAndBackslashes(Exec, Flags);
+  escapeSpacesAndBackslashes(Exec, Flags);
   for (const char *OriginalArg : OriginalArgs) {
     llvm::SmallString<128> EscapedArg;
-    EscapeSpacesAndBackslashes(OriginalArg, EscapedArg);
+    escapeSpacesAndBackslashes(OriginalArg, EscapedArg);
     Flags += " ";
     Flags += EscapedArg;
   }
 
   return Args.MakeArgString(Flags);
 }
 
-bool tools::ShouldRecordCommandLine(const ToolChain &TC,
+bool tools::shouldRecordCommandLine(const ToolChain &TC,
                                     const llvm::opt::ArgList &Args,
                                     bool &FRecordCommandLine,
                                     bool &GRecordCommandLine) {

clang/lib/Driver/ToolChains/CommonArgs.h

@@ -236,15 +236,15 @@ void addMCModel(const Driver &D, const llvm::opt::ArgList &Args,
 /// Add backslashes to escape spaces and other backslashes.
 /// This is used for the space-separated argument list specified with
 /// the -dwarf-debug-flags option.
-void EscapeSpacesAndBackslashes(const char *Arg,
+void escapeSpacesAndBackslashes(const char *Arg,
                                 llvm::SmallVectorImpl<char> &Res);
 
 /// Join the args in the given ArgList, escape spaces and backslashes and
 /// return the joined string. This is used when saving the command line as a
 /// result of using either the -frecord-command-line or -grecord-command-line
 /// options. The lifetime of the returned c-string will match that of the Args
 /// argument.
-const char *RenderEscapedCommandLine(const ToolChain &TC,
+const char *renderEscapedCommandLine(const ToolChain &TC,
                                      const llvm::opt::ArgList &Args);
 
 /// Check if the command line should be recorded in the object file. This is
@@ -253,7 +253,7 @@ const char *RenderEscapedCommandLine(const ToolChain &TC,
 /// is currently only supported on ELF platforms. The last two boolean
 /// arguments are out parameters and will be set depending on the command
 /// line options that were passed.
-bool ShouldRecordCommandLine(const ToolChain &TC,
+bool shouldRecordCommandLine(const ToolChain &TC,
                              const llvm::opt::ArgList &Args,
                              bool &FRecordCommandLine,
                              bool &GRecordCommandLine);

clang/lib/Driver/ToolChains/Flang.cpp

@@ -887,8 +887,8 @@ void Flang::ConstructJob(Compilation &C, const JobAction &JA,
 
   bool FRecordCmdLine = false;
   bool GRecordCmdLine = false;
-  if (ShouldRecordCommandLine(TC, Args, FRecordCmdLine, GRecordCmdLine)) {
-    const char *CmdLine = RenderEscapedCommandLine(TC, Args);
+  if (shouldRecordCommandLine(TC, Args, FRecordCmdLine, GRecordCmdLine)) {
+    const char *CmdLine = renderEscapedCommandLine(TC, Args);
     if (FRecordCmdLine) {
       CmdArgs.push_back("-record-command-line");
       CmdArgs.push_back(CmdLine);