[PATCH] D140438: [IR/MachineOutliner] Add a "nooutline" function attr and respect it

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 21 10:51:41 PST 2022


arsenm added a comment.

In D140438#4011132 <https://reviews.llvm.org/D140438#4011132>, @paquette wrote:

> In D140438#4010322 <https://reviews.llvm.org/D140438#4010322>, @nikic wrote:
>
>> FWIW, if something affects IR semantics, I'd generally expect it to be a first-class attribute.
>
> Would it be better for me to move it to Attributes.td, then?

I don't have a particularly strong opinion either way. I don't think I'd go so far as to say this changes IR semantics, it just turns off one transform


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

https://reviews.llvm.org/D140438



More information about the llvm-commits mailing list