[PATCH] D60220: [CUDA][Windows] Final fix for bug 38811 (Step 3 of 3)

Artem Belevich via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Apr 5 10:08:31 PDT 2019


tra added a comment.

In D60220#1456430 <https://reviews.llvm.org/D60220#1456430>, @emankov wrote:

> Oooh, sorry, but I've just pushed the fix. But with the following words: "Add missing long double device functions' declarations. Provide only declarations to prevent any use of long double on the device side, because CUDA does not support long double on the device side."


It's not a big deal at the moment -- there are no `long double` users in CUDA on linux yet. You can clean up in another commit.
BTW, you may want to make commit description somewhat more concise than rC357779 <https://reviews.llvm.org/rC357779>. Including all details of reproduction, etc is way too much info which is better suited for the bug report or review.


Repository:
  rC Clang

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

https://reviews.llvm.org/D60220





More information about the cfe-commits mailing list