[PATCH] D124866: [CUDA][HIP] support __noinline__ as keyword

Yaxun Liu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon May 9 10:03:41 PDT 2022


yaxunl added a comment.

In D124866#3500761 <https://reviews.llvm.org/D124866#3500761>, @aaron.ballman wrote:

> Should we do `__forceinline__` at the same time so that there's consistency?

`__forceinline__` does not have the issue as `__noinline__` has since it is not a GCC attribute. The current CUDA/HIP implementation of `__forceinline__` in header files is sufficient. I do not see the benefit of implementing `__forceinline__` as a keyword.


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

https://reviews.llvm.org/D124866



More information about the cfe-commits mailing list