[PATCH] D150681: [SCEVExpander] Remember phi nodes inserted by LCSSA construction

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 16 08:16:44 PDT 2023

nikic created this revision.
nikic added a reviewer: fhahn.
Herald added subscribers: hoy, StephenFan, javed.absar, hiraditya.
Herald added a project: All.
nikic requested review of this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

SCEVExpander keeps track of all instructions it inserted. However, it currently misses some phi nodes created during LCSSA construction. Fix this by collecting these into another argument.

This is a bit awkward because the LCSSA phi nodes themselves go through the Builder and its insertion callback, so we only need to deal with the phi nodes created by SSAUpdater.

This was reported as a regression on D149344 <https://reviews.llvm.org/D149344>, but the reduced test case also reproduces without it.



-------------- next part --------------
A non-text attachment was scrubbed...
Name: D150681.522632.patch
Type: text/x-patch
Size: 6870 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20230516/e2c3fd3e/attachment-0001.bin>

More information about the llvm-commits mailing list