[PATCH] D56567: [ADT] Force attribute used on functions marked as always_inline.

Davide Italiano via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 10 17:08:42 PST 2019


davide added a comment.

In D56567#1353608 <https://reviews.llvm.org/D56567#1353608>, @mehdi_amini wrote:

> What about creating a new macrothat carry the semantic we're looking for here:  "LLVM_KEEP_FOR_DEBUG"?
>  We can conditionally define it based on NDEBUG or add a CMake flag for it that is enabled in debug build (the latter has the advantage that it would allow to turn it on even in release builds when needing to debug these)


In case we decide to go this route, I guess something like this will do.


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

https://reviews.llvm.org/D56567





More information about the llvm-commits mailing list