[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