[PATCH] D92080: [Clang] Mutate long-double math builtins into f128 under IEEE-quad

Craig Topper via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Dec 9 11:46:40 PST 2020


craig.topper added a comment.

I don't think I understand the whole picture here. Why would only builtins get mutated? Does a call to "modf" still call "modf"? But __builtin_modf will call modf128? Is there a corresponding patch to the runtime libcalls table in the backend? With -ffast-math __builtin_sinl becomes llvm.sin.f128 and the backend will emit a call to sinl if ISD::SIN isn't legal.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D92080



More information about the cfe-commits mailing list