[PATCH] D84327: [SCEVExpander] Add helper to clean up instrs inserted while expanding.

Jordan Rupprecht via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 21 12:52:08 PDT 2020


rupprecht added a comment.

In D84327#2230474 <https://reviews.llvm.org/D84327#2230474>, @fhahn wrote:

> I recommitted the change in 8eded24bf46c05ffd110d521f58320cdee93866e <https://reviews.llvm.org/rG8eded24bf46c05ffd110d521f58320cdee93866e> with a fix for the issue reported by @rupprecht, which was caused due to the expander also adding existing, re-used values to InsertedValues, which we should not try to remove.

I can confirm the recommit looks good, thanks!

I forgot to mention we actually saw this as a miscompile when building w/ a release build of clang; the assertion caught the underlying issue. Both the miscompile (with the release build) and the assertion (with the debug build) are gone now.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84327



More information about the llvm-commits mailing list