[PATCH] D13321: [LLVM] Fix Clang-tidy modernize-use-nullptr warnings in source directories and generated files; other minor cleanups.

Hans Wennborg via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 6 16:25:55 PDT 2015


hans accepted this revision.
hans added a comment.
This revision is now accepted and ready to land.

lgtm. I'll commit this.


================
Comment at: lib/Support/Unix/Path.inc:157
@@ -157,2 +156,3 @@
+      if (getprogpath(exe_path, argv0))
           return exe_path;
   }
----------------
Looks like indent is off. Might as well fix that while we're here.

================
Comment at: utils/TableGen/SubtargetEmitter.cpp:1418
@@ -1416,3 +1417,3 @@
   Enumeration(OS, "SubtargetFeature");
-  OS << "} // End llvm namespace \n";
+  OS << "} // end llvm namespace \n";
   OS << "#endif // GET_SUBTARGETINFO_ENUM\n\n";
----------------
Might as well remove the trailing space while here.


http://reviews.llvm.org/D13321





More information about the llvm-commits mailing list