[llvm] [FuncSpec] Update function specialization to handle phi-chains (PR #72903)

Chuanqi Xu via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 21 01:31:09 PST 2023


================
@@ -217,6 +217,10 @@ class InstCostVisitor : public InstVisitor<InstCostVisitor, Constant *> {
   Cost estimateSwitchInst(SwitchInst &I);
   Cost estimateBranchInst(BranchInst &I);
 
+  bool discoverTransitivelyIncomingValues(
----------------
ChuanqiXu9 wrote:

How about adding comments here so that code readers can understand what is transitive phis and why should we care about them?

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


More information about the llvm-commits mailing list