https://github.com/AaronBallman commented:

The changes seem reasonable to me but I'd feel more comfortable if the 
functionality was also being used (so that we'd get test coverage verifying its 
correctness). Do you think it would be reasonable to include the RISCV changes 
as well?

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

Reply via email to