[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 14 21:06:24 PDT 2020


rupprecht added a comment.

FYI, I reverted this in 38884641f28e373ce291dc5ea93416756216e536 <https://reviews.llvm.org/rG38884641f28e373ce291dc5ea93416756216e536> due to the assertion failure being triggered. The reduction is taking longer than usual, so I wasn't able to finish it today. I should be able to have something on Monday.

It's down to a couple hundred lines of C++, but when I try to get an IR reduction (e.g. using `-Xclang -disable-O0-optnone -S -emit-llvm`), the crash does not reproduce. And creduce doesn't seem to know how to reduce a bunch of the stuff (heavy on the templates), so I'm paring it down manually.

The assertion failure is only with the new pass manager, btw.


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