[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