[PATCH] D115441: [X86][MS] Add 80bit long double support for Windows

[Phoebe Wang via Phabricator via llvm-commits]

pengfei added a comment.

In D115441#3182938 <https://reviews.llvm.org/D115441#3182938>, @craig.topper wrote:

> Doesn’t icc also emit code into main to change the FPCW precision control field? Is making long double 80 bits useful if you don’t increase the precision in hardware?

Yes, but it is controlled by an independent option `/Qpc80`. We can do a follow-up if it is required. Currently, I think it's still useful for library build and functional compatibility with ICC. Even considering the missing `/Qpc80`, we can still get the less precise result rather than the wrong one when linking each other. Besides, users are always able to set it manually.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115441