[PATCH] D57629: [LoopIdiomRecognize] @llvm.dbg values shouldn't affect the transformation.

Jonas Paulsson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Feb 2 09:32:32 PST 2019


jonpa added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/LoopIdiomRecognize.cpp:1467
+  uint32_t HeaderSize =
+      std::distance(InstWithoutDebugIt.begin(), InstWithoutDebugIt.end());
+
----------------
aprantl wrote:
> aprantl wrote:
> > I would be surprised if this is the only occurrence of this exact bug...
> We should probably grep for "cost" in the optimizer.
Found another one, or at least related, I think: https://bugs.llvm.org/show_bug.cgi?id=40573


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D57629/new/

https://reviews.llvm.org/D57629





More information about the llvm-commits mailing list