[PATCH] D27288: [GVN] When merging blocks update LoopInfo if it's available

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 1 11:08:30 PST 2016


efriedma added inline comments.


================
Comment at: llvm/trunk/lib/Transforms/Scalar/GVN.cpp:2697
 
+    auto *LIWP = getAnalysisIfAvailable<LoopInfoWrapperPass>();
+
----------------
This doesn't make sense... getAnalysisUsage says that GVN doesn't preserve LoopInfo, so it shouldn't be required to preserve it.  How exactly is this causing a problem?


Repository:
  rL LLVM

https://reviews.llvm.org/D27288





More information about the llvm-commits mailing list