[PATCH] D116016: [Clang] [PowerPC] Emit module flag for current float abi

Fangrui Song via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Dec 21 09:28:57 PST 2021


MaskRay added a comment.

In D116016#3203005 <https://reviews.llvm.org/D116016#3203005>, @nemanjai wrote:

> We should not be emitting the attribute in modules that do not have any use of `long double`.

Right, otherwise it would be a bit unfortunate that a library not using long double has to be compiled differently to use ThinLTO with other programs, since the module flags metadata's behavior is `Error` (1).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D116016



More information about the cfe-commits mailing list