[clang] [PowerPC][RFC] Make power9-vector indicate isa-3.0 and power9-altivec (PR #86905)
github-actions[bot] wrote: :warning: C/C++ code formatter, clang-format found issues in your code. :warning: You can test this locally with the following command: ``bash git-clang-format --diff 4d315ff382de912e5129b417c997116851088d4b 99e76dbe0016332ed1d21edbf5297cd5189afe3d -- clang/lib/Basic/Targets/PPC.cpp `` View the diff from clang-format here. ``diff diff --git a/clang/lib/Basic/Targets/PPC.cpp b/clang/lib/Basic/Targets/PPC.cpp index 14caa1898a..b6b968d638 100644 --- a/clang/lib/Basic/Targets/PPC.cpp +++ b/clang/lib/Basic/Targets/PPC.cpp @@ -606,7 +606,8 @@ bool PPCTargetInfo::initFeatureMap( .Case("pwr8", true) .Default(false); - Features["power9-altivec"] = llvm::is_contained(FeaturesVec, "+power9-vector"); + Features["power9-altivec"] = + llvm::is_contained(FeaturesVec, "+power9-vector"); // Power10 includes all the same features as Power9 plus any features specific // to the Power10 core. `` https://github.com/llvm/llvm-project/pull/86905 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
[clang] [PowerPC][RFC] Make power9-vector indicate isa-3.0 and power9-altivec (PR #86905)
llvmbot wrote: @llvm/pr-subscribers-clang @llvm/pr-subscribers-backend-powerpc Author: Kai Luo (bzEq) Changes This is to address https://github.com/llvm/llvm-project/issues/84703. However this might not be a long-term solution in my view. --- Full diff: https://github.com/llvm/llvm-project/pull/86905.diff 1 Files Affected: - (modified) clang/lib/Basic/Targets/PPC.cpp (+3) ``diff diff --git a/clang/lib/Basic/Targets/PPC.cpp b/clang/lib/Basic/Targets/PPC.cpp index aebe51bfa4daad..14caa1898a9eb8 100644 --- a/clang/lib/Basic/Targets/PPC.cpp +++ b/clang/lib/Basic/Targets/PPC.cpp @@ -597,6 +597,7 @@ bool PPCTargetInfo::initFeatureMap( .Default(false); Features["isa-v30-instructions"] = + llvm::is_contained(FeaturesVec, "+power9-vector") or llvm::StringSwitch(CPU).Case("pwr9", true).Default(false); Features["quadword-atomics"] = @@ -605,6 +606,8 @@ bool PPCTargetInfo::initFeatureMap( .Case("pwr8", true) .Default(false); + Features["power9-altivec"] = llvm::is_contained(FeaturesVec, "+power9-vector"); + // Power10 includes all the same features as Power9 plus any features specific // to the Power10 core. if (CPU == "pwr10" || CPU == "power10") { `` https://github.com/llvm/llvm-project/pull/86905 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
[clang] [PowerPC][RFC] Make power9-vector indicate isa-3.0 and power9-altivec (PR #86905)
https://github.com/bzEq created https://github.com/llvm/llvm-project/pull/86905 This is to address https://github.com/llvm/llvm-project/issues/84703. However this might not be a long-term solution in my view. >From 99e76dbe0016332ed1d21edbf5297cd5189afe3d Mon Sep 17 00:00:00 2001 From: Kai Luo Date: Thu, 28 Mar 2024 03:31:30 + Subject: [PATCH] Indicate pwr9 and p9 altivec --- clang/lib/Basic/Targets/PPC.cpp | 3 +++ 1 file changed, 3 insertions(+) diff --git a/clang/lib/Basic/Targets/PPC.cpp b/clang/lib/Basic/Targets/PPC.cpp index aebe51bfa4daad..14caa1898a9eb8 100644 --- a/clang/lib/Basic/Targets/PPC.cpp +++ b/clang/lib/Basic/Targets/PPC.cpp @@ -597,6 +597,7 @@ bool PPCTargetInfo::initFeatureMap( .Default(false); Features["isa-v30-instructions"] = + llvm::is_contained(FeaturesVec, "+power9-vector") or llvm::StringSwitch(CPU).Case("pwr9", true).Default(false); Features["quadword-atomics"] = @@ -605,6 +606,8 @@ bool PPCTargetInfo::initFeatureMap( .Case("pwr8", true) .Default(false); + Features["power9-altivec"] = llvm::is_contained(FeaturesVec, "+power9-vector"); + // Power10 includes all the same features as Power9 plus any features specific // to the Power10 core. if (CPU == "pwr10" || CPU == "power10") { ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits