llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-clang <details> <summary>Changes</summary> PowerPC AIX backend does not support float128 at all. Diagnose even when specifying -mfloat128 to avoid backend crash. --- Full diff: https://github.com/llvm/llvm-project/pull/67298.diff 2 Files Affected: - (modified) clang/lib/Basic/Targets/PPC.cpp (+1-1) - (modified) clang/test/Sema/128bitfloat.cpp (+1) ``````````diff diff --git a/clang/lib/Basic/Targets/PPC.cpp b/clang/lib/Basic/Targets/PPC.cpp index e0abc069032e1ce..119ed9c1ac4504e 100644 --- a/clang/lib/Basic/Targets/PPC.cpp +++ b/clang/lib/Basic/Targets/PPC.cpp @@ -52,7 +52,7 @@ bool PPCTargetInfo::handleTargetFeatures(std::vector<std::string> &Features, HasDirectMove = true; } else if (Feature == "+htm") { HasHTM = true; - } else if (Feature == "+float128") { + } else if (Feature == "+float128" && !getTriple().isOSAIX()) { HasFloat128 = true; } else if (Feature == "+power9-vector") { HasP9Vector = true; diff --git a/clang/test/Sema/128bitfloat.cpp b/clang/test/Sema/128bitfloat.cpp index b98b42496e8db27..bd9f2889c98f6aa 100644 --- a/clang/test/Sema/128bitfloat.cpp +++ b/clang/test/Sema/128bitfloat.cpp @@ -1,6 +1,7 @@ // RUN: %clang_cc1 -verify -std=gnu++11 %s // RUN: %clang_cc1 -verify -std=c++11 %s // RUN: %clang_cc1 -triple powerpc64-linux -verify -std=c++11 %s +// RUN: %clang_cc1 -triple powerpc64-ibm-aix -target-feature +float128 -verify -std=c++11 %s // RUN: %clang_cc1 -triple i686-windows-gnu -verify -std=c++11 %s // RUN: %clang_cc1 -triple x86_64-windows-gnu -verify -std=c++11 %s // RUN: %clang_cc1 -triple x86_64-windows-msvc -verify -std=c++11 %s `````````` </details> https://github.com/llvm/llvm-project/pull/67298 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
