rsmith accepted this revision.
rsmith added a comment.

This looks OK to me.

I think we'll also need updates for `_ExtInt` now too -- we have more 
fundamental types with padding than only `long double` and `bool` now. (I 
continue to be displeased that we assume 8-bit `char` in `__builtin_bit_cast`, 
but that's not new in this patch.)


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D76323/new/

https://reviews.llvm.org/D76323

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D76323: [... Raul Tambre via Phabricator via cfe-commits
    • [PATCH] D763... Richard Smith - zygoloid via Phabricator via cfe-commits
    • [PATCH] D763... Erik Pilkington via Phabricator via cfe-commits

Reply via email to