[PATCH] D71974: [Attributor][WIP] Connect AAIsDead with AAUndefinedBehavior
Stefanos Baziotis via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Dec 30 06:57:44 PST 2019
baziotis marked an inline comment as done.
baziotis added inline comments.
================
Comment at: llvm/lib/Transforms/IPO/Attributor.cpp:2720-2723
+ if (!AAUB.isKnownToCauseUB(&Front)) {
+ ToBeExploredFrom.insert(&Front);
+ assumeLive(A, EntryBlock);
+ }
----------------
With the known info it's easy because we don't need to remember that we were based on assumed info (in the `updateImpl()`). But now that I see that again, it probably is as simple as inserting `Front` in the `ToBeExploredFrom` (always) and only make the `EntryBlock` live if it is not assumed to cause UB.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D71974/new/
https://reviews.llvm.org/D71974
More information about the llvm-commits
mailing list