[PATCH] D79344: [cuda] Start diagnosing variables with bad target.

Michael Liao via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu May 7 14:43:33 PDT 2020


hliao added a comment.

In D79344#2026025 <https://reviews.llvm.org/D79344#2026025>, @tra wrote:

> We're calling `copysign( int, double)`. The standard library provides `copysign(double, double)`, CUDA provides only `copysign(float, double)`.  As far as C++ is concerned, both require one type conversion. I guess previously we would give `__device__` one provided by CUDA a higher preference, considering that the callee is a device function. Now both seem to have equal weight. I'm not sure how/why,


@yaxunl, that may be related to the change of overload resolution. Back to this change, that error should not be related to the non-local variable checks.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79344





More information about the cfe-commits mailing list