[PATCH] D157836: [RFC][IR] Correct lowering of `f128` intrinsics
Eli Friedman via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Aug 14 21:33:10 PDT 2023
efriedma added a comment.
There needs to be some communication between clang and the backend if we want to support -mlong-double-128. But probably we want some reasonable default, sure.
Making the backends in question override the libcall name probably works, but we probably want to share the code across backends? That said, the simplest way to accomplish it is to just stick a helper in TargetLowering, and make each backend call it as appropriate.
We could consider something more abstract, but I don't think it's worth it unless we find other places need the information.
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