commit 62a46bbe31a34237673d94c6f3e82fcbd8e56a37
author peter klausler <pklausler@nvidia.com> Tue Oct 12 17:14:14 2021 -0700
committer Copybara-Service <copybara-worker@google.com> Tue Oct 12 17:20:42 2021 -0700
tree 5709629abbb460abf605ab63eeb56e96b6aa8a93
parent 5a121fb3f84c21b8d623f2b581fc6456e68f722a

[flang] Fix Windows build

A recently added class constructor needs to be "explicit" to
prevent it from being available for use as a conversion, which
is breaking the MSVC build of flang.

GitOrigin-RevId: 02410df530fbef5256711ec2220c4c34468c894c

include/flang/Common/uint128.h

diff --git a/include/flang/Common/uint128.h b/include/flang/Common/uint128.h
index 68098da..adbeeb3 100644
--- a/include/flang/Common/uint128.h
+++ b/include/flang/Common/uint128.h
@@ -47,16 +47,19 @@
   constexpr Int128 &operator=(const Int128 &) = default;
   constexpr Int128 &operator=(Int128 &&) = default;
 
-  constexpr Int128(const Int128<!IS_SIGNED> &n)
+  explicit constexpr Int128(const Int128<!IS_SIGNED> &n)
       : low_{n.low()}, high_{n.high()} {}
-  constexpr Int128(Int128<!IS_SIGNED> &&n) : low_{n.low()}, high_{n.high()} {}
+  explicit constexpr Int128(Int128<!IS_SIGNED> &&n)
+      : low_{n.low()}, high_{n.high()} {}
   constexpr Int128 &operator=(const Int128<!IS_SIGNED> &n) {
     low_ = n.low();
     high_ = n.high();
+    return *this;
   }
   constexpr Int128 &operator=(Int128<!IS_SIGNED> &&n) {
     low_ = n.low();
     high_ = n.high();
+    return *this;
   }
 
   constexpr Int128 operator+() const { return *this; }