[PATCH] D70366: Add new 'flatten' LLVM attribute to fix clang's 'flatten' function attribute
Duncan P. N. Exon Smith via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Apr 8 15:14:07 PDT 2020
dexonsmith added a comment.
In D70366#1970375 <https://reviews.llvm.org/D70366#1970375>, @jdoerfert wrote:
> TBH, I would issue a warning if we see `flatten` in O0 that says this will not work and be done with it.
I would argue against diagnostics that depend on optimization level, since that leads to an inconsistent developer experience. In practice developers tend to use Debug and Release configurations fairly interchangeably, with different optimization levels for each.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D70366/new/
https://reviews.llvm.org/D70366
More information about the cfe-commits
mailing list