[PATCH] D39137: Add CallSiteSplitting pass

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 30 14:11:39 PDT 2020


fhahn added inline comments.


================
Comment at: llvm/trunk/lib/Transforms/Scalar/CallSiteSplitting.cpp:370
+    }
+    break;
+  }
----------------
thakis wrote:
> Was this break supposed to be in the if above? As-is, it unconditionally breaks after the first iteration of the loop.
> 
> But even if it was a line further up, we'd only check the first Phi in the parent and check if it's an arg. I don't know if that's enough.
It looks like the code indeed bails out too early. I think we should check all PHIs. I've put up D77089. It should not matter if there are unrelated PHIs in between, but maybe the goal was to be extremely conservative @junbuml ?


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D39137/new/

https://reviews.llvm.org/D39137





More information about the llvm-commits mailing list