[PATCH] D14312: [LLVM] Fix some Clang-tidy modernize warnings, other minor fixes

Eugene Zelenko via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 4 14:28:08 PST 2015


Eugene.Zelenko added inline comments.

================
Comment at: lib/Support/TimeValue.cpp:27
@@ -25,3 +26,3 @@
 void
-TimeValue::normalize( void ) {
+TimeValue::normalize() {
   if ( nanos_ >= NANOSECONDS_PER_SECOND ) {
----------------
hans wrote:
> should we join this with the previous line while here?
Will fix it in commit.

================
Comment at: lib/Transforms/Scalar/MergedLoadStoreMotion.cpp:111
@@ -110,3 +110,3 @@
   static char ID; // Pass identification, replacement for typeid
-  explicit MergedLoadStoreMotion(void)
+  explicit MergedLoadStoreMotion()
       : FunctionPass(ID), MD(nullptr), MagicCompileTimeControl(250) {
----------------
hans wrote:
> might as well, drop the explicit I suppose
Will fix it in commit.


Repository:
  rL LLVM

http://reviews.llvm.org/D14312





More information about the llvm-commits mailing list