[PATCH] D18429: Add lowering support for llvm.experimental.deoptimize

Sanjoy Das via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 24 13:29:16 PDT 2016


sanjoy added inline comments.

================
Comment at: lib/CodeGen/SelectionDAG/StatepointLowering.cpp:925
@@ +924,3 @@
+void SelectionDAGBuilder::LowerDeoptimizeCall(ImmutableCallSite CS) {
+  const auto &TLI = DAG.getTargetLoweringInfo();
+
----------------
reames wrote:
> It looks like you're only handling calls here.  Using CallSite is fine, but add an assertion.
In retrospect, this can be easily combined with `LowerCallSiteWithDeoptBundle`.  I'll do that in a separate change if feasible.


Repository:
  rL LLVM

http://reviews.llvm.org/D18429





More information about the llvm-commits mailing list