https://github.com/jhuber6 approved this pull request.

This looks good for me as a first step, but we'll definitely need some of the 
others to chime in before merging. In general I'd like to see us move towards 
something that more generic, considering that it seems like SPIRV, NVPTX, and 
AMDGPU now all have differing opinions on how to solve this. It would be nice 
to have a common LLVM intrinsic for this similar to `llvm.is.constant`.

https://github.com/llvm/llvm-project/pull/134016
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to