[PATCH] D101011: [Attr] Add "noipa" function attribute

Dávid Bolvanský via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 15 09:55:00 PDT 2022


xbolva00 added a comment.

>> I don't think there's any need for our noipa to differ from GCC's semantics - their documentation that says noipa implies noinline can be seen as a description of behavior, not a constraint on how that behavior is achieved. noipa should logically disable inlining as a consequnce of disallowing interprodecural analysis/information.

GCC’s __attribute__(noipa) (frontend) -> noipa, noinline, noclone, no_icf (midend).

Any good reason why not do it same way as well? Avoid hidden logical assumption that noipa is basically noinline + something.

And then llvm optimizers can check - if (attrs.contains (“noipa”) do not propagate constants, etc…


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101011



More information about the llvm-commits mailing list