[PATCH] D70366: Add new 'flatten' LLVM attribute to fix clang's 'flatten' function attribute

Duncan P. N. Exon Smith via Phabricator via llvm-commits llvm-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 llvm-commits mailing list