tra added a comment.

> The one downside to this is that we are no longer stable under multiple 
> compilations of the same file.

This is a moderately serious issue. Some users care about the build 
reproducibility. Recompiling the same sources and getting different results 
will trigger all sorts of red flags that would need to be investigated in order 
to make sure the build is not broken.

> However, this is a very niche use-case and is not supported by Nvidia's CUDA 
> compiler so it likely to be good enough.

The fact that NVCC didn't always generate the same output **was** an issue when 
we were using it for CUDA compilation.
In general, "not supported by NVCC" is not quite applicable here, IMO. The goal 
here is to make clang work correctly.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D125904

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

Reply via email to