[llvm] Account for inline assembly instructions in inlining cost. (PR #146628)
Kazu Hirata via llvm-commits
llvm-commits at lists.llvm.org
Tue Jul 1 20:51:03 PDT 2025
================
@@ -777,6 +786,42 @@ class InlineCostCallAnalyzer final : public CallAnalyzer {
addCost(SwitchCost);
}
+
+ // Parses the inline assembly argument to account for its cost. Inline
+ // assembly instructions incur higher costs for inlining since they cannot be
+ // analyzed and optimized.
+ void onInlineAsm(InlineAsm &Arg) override {
+ SmallVector<StringRef, 4> Fragments;
+ Arg.getAsmString().split(Fragments, "\n");
+ int SectionLevel = 0;
+ int InlineAsmInstrCount = 0;
+ for (const auto &Fragment : Fragments) {
+ // Trim whitespaces and comments.
+ auto Trimmed = Fragment.trim();
----------------
kazutakahirata wrote:
I would spell out the type here again:
```suggestion
StringRef Trimmed = Fragment.trim();
```
https://github.com/llvm/llvm-project/pull/146628
More information about the llvm-commits
mailing list