[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