[PATCH] D129677: Disable machine function splitting for functions with inline asm br

Wed Jul 13 14:46:49 PDT 2022

nickdesaulniers added a comment.

In D129677#3649869 <https://reviews.llvm.org/D129677#3649869>, @efriedma wrote:

>> In the inline assembly, there may be branch that can only jump a limited distance, if we run MFS on the function, the resulting distance maybe to far to encode into the instruction.
>
> If there's a rule like this, we should explicitly state in LangRef which branches are/are not allowed.

It also seems like such a case is easy to fix; in the inline asm the user should just use the wider encoding.

Maybe would be a surprising failure though, since you'd think the labels are nearby.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D129677