[PATCH] D12928: GVN small fix

Piotr Padlewski via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 17 00:43:41 PDT 2015


Prazek created this revision.
Prazek added reviewers: rsmith, nlewycky, majnemer.
Prazek added a subscriber: llvm-commits.

Under the if (Dep.isClobber()) there was another if (Dep.isClobber())

http://reviews.llvm.org/D12928

Files:
  lib/Transforms/Scalar/GVN.cpp

Index: lib/Transforms/Scalar/GVN.cpp
===================================================================
--- lib/Transforms/Scalar/GVN.cpp
+++ lib/Transforms/Scalar/GVN.cpp
@@ -1939,10 +1939,8 @@
       ++NumGVNLoad;
       return true;
     }
-  }
 
-  // If the value isn't available, don't do anything!
-  if (Dep.isClobber()) {
+    // If the value isn't available, don't do anything!
     DEBUG(
       // fast print dep, using operator<< on instruction is too slow.
       dbgs() << "GVN: load ";


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D12928.34970.patch
Type: text/x-patch
Size: 503 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150917/b214180f/attachment.bin>


More information about the llvm-commits mailing list