[PATCH] D157836: [RFC][IR] Correct lowering of `f128` intrinsics

Trevor Gross via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 14 00:48:38 PDT 2023


tmgross added a comment.

I did not yet complete this change since I am unsure how to handle libc support. Glibc supports f128 math since 2.26 (2017, [1]), but I am not sure how to handle older versions or platforms where the math isn't yet supported. For targets where `long double` is `__float128` this doesn't matter and the `cosl`-style calls already work properly (should probably lower to f128 instead to be less ambiguous), but for anything else the lowering is broken. What is the best way to handle this?

[1]: https://sourceware.org/git/?p=glibc.git;a=blob;f=NEWS;hb=81325b12b14c44887f1633a2c180a413afc2b504#l143


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D157836



More information about the llvm-commits mailing list