[PATCH] D147982: Account for PATCHABLE instrs in Branch Relaxation
Jessica Paquette via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Apr 12 10:25:26 PDT 2023
paquette added a comment.
I think it's okay to keep all of this in one patch.
I put the MIR test through Compiler Explorer to see what is outlined (and removed some of the stuff that isn't necessary in MIR):
https://godbolt.org/z/e85n9Wh8r
It looks like this, without this patch, does outline something with `PATCHABLE_FUNCTION_EXIT`, but we don't get any outlining behaviour for `PATCHABLE_FUNCTION_ENTRY`.
Would it be possible to add a case where `PATCHABLE_FUNCTION_ENTRY` is outlined without this patch?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D147982/new/
https://reviews.llvm.org/D147982
More information about the llvm-commits
mailing list