[PATCH] D103221: [CUDA][HIP] Change default lang std to c++14

[Artem Belevich via Phabricator via cfe-commits]

tra added inline comments.


================
Comment at: clang/test/SemaCUDA/asm_delayed_diags.cu:31
 static __device__ __host__ double t3(double x) {
-  register long double result;
+  register long double result; // expected-warning {{'register' storage class specifier is deprecated and incompatible with C++17}}
   __asm __volatile("frndint"
----------------
Perhaps we should just remove the `register`.
TBH, I'm not quite sure what this test is supposed to do for NVPTX --  `t` is not a register class in NVPTX nor does it seem to be a generic constraint.

I guess the idea was that we should ignore asm errors in the host code, while we're compiling for device. In that case, we certainly do not need `register`


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

https://reviews.llvm.org/D103221