[llvm] ARM: Remove redundant or buggy config of __aeabi_d2h (PR #152126)
Eli Friedman via llvm-commits
llvm-commits at lists.llvm.org
Tue Aug 5 14:28:58 PDT 2025
https://github.com/efriedma-quic approved this pull request.
See https://gcc.gnu.org/bugzilla/show_bug.cgi?id=120897 for a summary of the history here; it's very messy. Really, we should be calling `__gnu_d2h_ieee` on targets using libgcc because `__aeabi_d2h` doesn't exist, but the patch (https://reviews.llvm.org/D94557) never got merged.
I think we have sufficient test coverage in llvm/test/CodeGen/ARM/fp16.ll, to detect if anyone accidentally makes further changes here.
This seems fine for now; if someone wants to revive D94557, they can do it once all the TableGen changes are in.
LGTM
https://github.com/llvm/llvm-project/pull/152126
More information about the llvm-commits
mailing list