[PATCH] D36073: [CGP] Extends the scope of optimizeMemoryInst optimization

[Daniel Berlin via Phabricator via llvm-commits]

dberlin added inline comments.


================
Comment at: lib/CodeGen/CodeGenPrepare.cpp:4381
+// Try to match PHI node to Candidate. Matcher tracks the matched Phi nodes.
+static bool MatchPhiNode(PHINode *PHI, PHINode *Candidate,
+                         DenseSet<PHIPair> &Matcher,
----------------
FWIW, i'd just hash them, rather thancheck the possible matches again and again, i think.

Happy to give you some code to crib for that.




https://reviews.llvm.org/D36073