[PATCH] D60831: [DebugInfo at O2][LoopVectorize] pr39024: Vectorized code linenos step through loop even after completion

Paul Robinson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 14 08:40:54 PDT 2019


probinson added a comment.

It sounds like inlining a function with a loop has a bug with respect to how the loop metadata is handled, and your patch merely tripped over that.  Would it be reasonable to fix the inlining-loop-metadata bug separately first?  And then the original patch is likely to Just Work?
Fixing one bug at a time is more in line with project practices.


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

https://reviews.llvm.org/D60831





More information about the llvm-commits mailing list