[PATCH] D130910: [NewGVN][PHIOFOPS] Bail out if an operand is in OpSafeForPHIOfOps but it is not safe for the current basic block.
Alina Sbirlea via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Aug 16 15:42:06 PDT 2022
asbirlea accepted this revision.
asbirlea added a comment.
This revision is now accepted and ready to land.
I don't feel like I fully understand NewGVN either, but it does resolve the issue and the testing I did so far did not reveal any additional issues. Thank you for resolving this!
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D130910/new/
https://reviews.llvm.org/D130910
More information about the llvm-commits
mailing list