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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to