================
@@ -774,6 +774,18 @@ static bool isWave32Capable(StringRef GPU, const Triple 
&T) {
   return IsWave32Capable;
 }
 
+static bool isWave64Capable(StringRef GPU, const Triple &T) {
+  if (T.isAMDGCN()) {
----------------
yxsamliu wrote:

I agree with Matt. We already have fillAMDGPUFeatureMap() that is supposed to 
fill the default features. On line 434/435, it already fills wave32/64 for 
spirv. It is kind of redundant and may cause inconsistency to have 
isWave32Capable() and isWave64Capable(). If we need a way to get wave64 
capability, we could cache the default target features returned by 
fillAMDGPUFeatureMap() in TargetInfo and look it up for wave64 capability.

https://github.com/llvm/llvm-project/pull/153693
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to