[PATCH] D47757: [Sema] Produce diagnostics when unavailable aligned allocation/deallocation functions are called

Akira Hatanaka via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Aug 17 12:57:15 PDT 2018


ahatanak added a comment.

In https://reviews.llvm.org/D47757#1204561, @tra wrote:

> It's a regression. There's a decent chance it breaks someone and this patch, if committed by itself, will end up being rolled back.


Is the regression you are referring to about the static function case? I don't see a difference between ToT clang and my patch in the diagnostics they produce when I compile the following code:

  __host__ void f();
  static __host__ __device__ void g() { f(); }
  __host__ __device__ void g2() { g(); } 

Both error out when `-fcuda-is-device` is provided. If I comment out the definition of g2, it compiles fine.


Repository:
  rC Clang

https://reviews.llvm.org/D47757





More information about the cfe-commits mailing list