[llvm] Delete the incorrect assert that assumes size of `APFloat` is the same as `IEEEFloat`. (PR #111780)
via llvm-commits
llvm-commits at lists.llvm.org
Wed Oct 9 19:32:17 PDT 2024
Ariel-Burton wrote:
> This assertion is to make sure that EBO is performed on APFloat. What is the compiler you use to build llvm?
The issue is that APFloat has a union (`Storage`). Downstream this includes a member / field for another floating point type. We are in the process of upstreaming this type. The representation of this type is larger than IEEE. Note also that you are lucky that the `DoubleAPFloat` is not larger than `IEEEFloat`.
It's not a matter of the compiler used to build, but the assumption that no member of the `Storage` union is larger than `IEEEFloat` not being true.
https://github.com/llvm/llvm-project/pull/111780
More information about the llvm-commits
mailing list