[llvm] [NewGVN] Set parent to the temporal instructions that are generated during phi-of-ops optimization (PR #66314)

Nikita Popov via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 15 00:08:51 PDT 2023


================
@@ -2666,6 +2671,7 @@ Value *NewGVN::findLeaderForInst(Instruction *TransInst,
   TempToBlock.insert({TransInst, PredBB});
   InstrDFS.insert({TransInst, IDFSNum});
 
+  PhiOfOpsCandidate = TransInst;
----------------
nikic wrote:

This never gets reset -- isn't it going to leak into all future calls of performSymbolicEvaluation?

https://github.com/llvm/llvm-project/pull/66314


More information about the llvm-commits mailing list