[PATCH] D126984: [clang] Add support for optimize function attribute
Aaron Ballman via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Jun 14 10:49:45 PDT 2022
aaron.ballman added a comment.
In D126984#3582508 <https://reviews.llvm.org/D126984#3582508>, @steplong wrote:
> I'm open to tabling this and just implementing support for an empty optimization list for the pragma (i.e. `#pragma optimize("", on | off)`). For our use case, at the moment, we only see the pragma being used this way. `on` would be a noop (honor command-line) and `off` would mean add optnone to the functions below the pragma
If that meets your needs, that is certainly a way to get you unblocked. I'm fine with that approach if you are.
I appreciate your patience while we figure out the right approach here; we don't usually have this many false starts when working through a feature review. :-)
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D126984/new/
https://reviews.llvm.org/D126984
More information about the cfe-commits
mailing list