Pierre-vh wrote:

I added b8 support too.

I understand it's quite a big patch, I could probably split it up into one 
patch per bit width if it helps.
The problem with this type of change is the massive amount of 
tests/autogenerated checks added, and splitting up the patch doesn't really 
help with reviewing those I think. It's still hundreds of lines.

https://github.com/llvm/llvm-project/pull/229429
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to