================ @@ -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