[PATCH] D16608: [In Progress][GVN] Common code for local and non-local load availability

Daniel Berlin via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 26 15:35:47 PST 2016


dberlin added a comment.

I'm more than happy to have someone clean this up, and overall, i think this is a good way to go.


================
Comment at: lib/Transforms/Scalar/GVN.cpp:1324
@@ -1321,1 +1323,3 @@
+    InsertPt = LI;
+  
   Value *Res;
----------------
Yeah, i also independently discovered this was broken if you tried to use the function anywhere outside of it's current callers :)



http://reviews.llvm.org/D16608





More information about the llvm-commits mailing list