[llvm] Update the EBO static_assert to dheck against APFloat::Storage instead of IEEEFloat. (PR #112589)

Nikita Popov via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 18 06:14:56 PDT 2024


https://github.com/nikic requested changes to this pull request.

I would like to keep the assertion as-is. It found a legitimate issue with your downstream changes: That they increase the size of APFloat.

You should drop the assertion downstream for now. When you upstream your changes, we can consider whether we are willing to take that size increase or not, and adjust the assertion accordingly at that time.

https://github.com/llvm/llvm-project/pull/112589


More information about the llvm-commits mailing list