[PATCH] D158647: [CodeGen][AArch64] Don't split inline asm goto blocks or their targets

Han Shen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 29 11:04:34 PDT 2023


shenhan accepted this revision.
shenhan added inline comments.
This revision is now accepted and ready to land.


================
Comment at: llvm/lib/Target/AArch64/AArch64InstrInfo.cpp:8440
+  };
+  if (llvm::any_of(MBB, isAsmGoto) || MBB.isInlineAsmBrIndirectTarget())
+    return false;
----------------
This test (`MBB.isInlineAsmBrIndirectTarget()`) is quicker and cheaper than acquiring and iterating over jump-tbl , we may rearrange the tests so that the quickest and cheapest tests are placed earlier than those less cheap.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D158647



More information about the llvm-commits mailing list