[PATCH] D47149: [SCCP] Mark DominatorTreeAnalysis as preserved.

Davide Italiano via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 21 09:57:06 PDT 2018


davide requested changes to this revision.
davide added a comment.
This revision now requires changes to proceed.

If I recall correctly the sparse constant propagation implementation in LLVM (interproceturally, at least), calls changeToUnreachable which modifies the CFG.
(this is, quite unfortunate, as it's declared in `Local.h`). So, I'm afraid the patch as is is not correct. That said, it's been a while since I last worked on this area so I might recall incorrectly :)


https://reviews.llvm.org/D47149





More information about the llvm-commits mailing list