JonChesterfield wrote:

Dropping this in favour of 
[93362](https://github.com/llvm/llvm-project/pull/93362) on risk assessment 
grounds.

This commit enabled ad hoc testing from wasm, x64, and aarch64. However if it's 
buggy, it'll show up on those targets, which should make the code owners 
reluctant to land it.

Going with pure amdgpu first is not great from the perspective of finding 
errors in this pass - since va_arg is currently a fatal_error, existing 
software on amdgpu doesn't tend to use variadic functions - but it does 
completely derisk the initial commit.

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

Reply via email to