[PATCH] D80598: Start migrating away from statepoint's inline length prefixed argument bundles
Anna Thomas via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue May 26 20:10:17 PDT 2020
anna accepted this revision.
anna added a comment.
This revision is now accepted and ready to land.
LGTM. thanks for working on this! Apart from dropping all of the extra handling code in RS4GC and verifier (which itself is a really good reason for this support btw), I'm assuming this for optimizations which know about the deopt_op bundle (or gc_transition bundle) after relocations are made explicit? i.e., a general robustness patch.
================
Comment at: llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp:2778
assert(!I.hasOperandBundlesOtherThan({LLVMContext::OB_deopt,
+ LLVMContext::OB_gc_transition,
LLVMContext::OB_funclet,
----------------
hmm. so we currently don't actually have any optimizations leveraging OB_gc_transition.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D80598/new/
https://reviews.llvm.org/D80598
More information about the llvm-commits
mailing list