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

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 13 14:09:41 PDT 2022


efriedma added a comment.

> The ones in the inline assembly cannot.

This should be fixable on top of D129288 <https://reviews.llvm.org/D129288>.  But an incremental fix separate from that would be okay.

Not sure the extra flag carries its weight; it's already hard enough to keep track of all the various flags we put on MachineFunction , and you can already easily check IsInlineAsmBrIndirectTarget() while you're iterating over basic blocks.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D129677



More information about the llvm-commits mailing list