[PATCH] D88511: [flang][msvc] long double is not different from double.

[Michael Kruse via Phabricator via llvm-commits]

Meinersbur updated this revision to Diff 296026.
Meinersbur retitled this revision from "[flang][msvc] double is same as long double under Windows AMD64 ABI." to "[flang][msvc] long double is not different from double.".
Meinersbur edited the summary of this revision.
Meinersbur added a comment.

- long double == double is generally true with msvc, not just with AMD64.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D88511

Files:
  flang/runtime/descriptor-io.h


Index: flang/runtime/descriptor-io.h
===================================================================
--- flang/runtime/descriptor-io.h
+++ flang/runtime/descriptor-io.h
@@ -279,7 +279,10 @@
         return FormattedRealIO<24, float, DIR>(io, descriptor);
       case 8:
         return FormattedRealIO<53, double, DIR>(io, descriptor);
-#if __x86_64__
+#if _MSC_VER
+        // long double is the same as double with the Microsoft compiler.
+        // Treat as unimplemented.
+#elif __x86_64__
       case 10:
         return FormattedRealIO<64, long double, DIR>(io, descriptor);
 #else
@@ -298,7 +301,10 @@
         return FormattedComplexIO<24, float, DIR>(io, descriptor);
       case 8:
         return FormattedComplexIO<53, double, DIR>(io, descriptor);
-#if __x86_64__
+#if _MSC_VER
+        // long double is the same as double with the Microsoft compiler.
+        // Treat as unimplemented.
+#elif __x86_64__
       case 10:
         return FormattedComplexIO<64, long double, DIR>(io, descriptor);
 #else


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D88511.296026.patch
Type: text/x-patch
Size: 1022 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20201004/c549eaf7/attachment.bin>