[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