================
@@ -2554,9 +2683,12 @@ bool Triple::isCompatibleWith(const Triple &Other) const 
{
 
 std::string Triple::merge(const Triple &Other) const {
   // If vendor is apple, pick the triple with the larger version number.
-  if (getVendor() == Triple::Apple)
-    if (Other.isOSVersionLT(*this))
-      return str();
+  if (getVendor() == Triple::Apple && Other.isOSVersionLT(*this))
+    return str();
+
+  if (isAMDGCN() && getOS() == Other.getOS() &&
----------------
ro-i wrote:

doesn't this need to check `Other.isAMDGCN()`, as well?

https://github.com/llvm/llvm-project/pull/206480
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to