[clang] [llvm] [Clang][Driver][LLVM] Add -fno-inline-functions-called-once; expose p… (PR #160343)
via cfe-commits
cfe-commits at lists.llvm.org
Mon Sep 29 07:45:00 PDT 2025
Karthikdhondi wrote:
> While GCC has -finline-functions-called-once, we don't blindly copy every option it supports. Please provide clear, technical justification. The description looks AI-generated, making it difficult to assess its value.
>
> > Thanks! I’ve split the change as suggested.
>
> I don't think you split the LLVM and Clang side changes.
Apologies for not conveying better.
We are working on a large application code base that was earlier compiled with GNU toolchain. Compiler option "-fno-inline-functions-called-once" was used. Now considering the benefits of LLVM toolchain, they want to have similar option with LLVM. Using __attribute__((no-inline)), requires making changes to application which is not a feasible option.
So, I have enabled the pass in LLVM it worked fine, and thought to create a PR for the same. So that, it might be helpful for others too.
By splitting the changes means, Do I need to create two different PRs for LLVM and Clang side changes?
https://github.com/llvm/llvm-project/pull/160343
More information about the cfe-commits
mailing list