[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