https://github.com/AaronBallman commented:

I'm by no means a fixed-point expert, so this is a bit of a drive-by review.

One thing to note is that we shouldn't be using N1169 because TR 18037 is 
freely available: 
https://standards.iso.org/ittf/PubliclyAvailableStandards/c051126_ISO_IEC_TR_18037_2008.zip

The changes should come with a release note so that users know about them.

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

Reply via email to