foad added inline comments.
================
Comment at: llvm/docs/AMDGPUUsage.rst:389
- xnack
scratch - *pal-amdpal*
+ ``gfx1013`` ``amdgcn`` dGPU - cumode -
Absolute - *rocm-amdhsa* *TBA*
+ - wavefrontsize64 flat
- *pal-amdhsa*
----------------
Is it dGPU or APU?
Every other entry with `*TBA*` also has a `TODO::` message
================
Comment at: llvm/lib/Target/AMDGPU/AMDGPU.td:468
+def FeatureGFX10_AEncoding : SubtargetFeature<"gfx10_a-encoding",
+ "GFX10_AEncoding",
----------------
What is this new encoding? It doesn't seem to be used for anything.
================
Comment at: llvm/lib/Target/AMDGPU/GCNSubtarget.h:879
+ }
+
bool hasGFX10_BEncoding() const {
----------------
Stray whitespace on this line.
================
Comment at: llvm/lib/Target/AMDGPU/Utils/AMDGPUBaseInfo.cpp:1453
+}
+
bool isGFX10_BEncoding(const MCSubtargetInfo &STI) {
----------------
Stray whitespace.
================
Comment at: llvm/test/CodeGen/AMDGPU/GlobalISel/llvm.amdgcn.intersect_ray.ll:4
+; RUN: llc -global-isel -march=amdgcn -mcpu=gfx1013 -verify-machineinstrs < %s
| FileCheck -check-prefix=GCN %s
+; RUN: llc -global-isel -march=amdgcn -mcpu=gfx1012 -verify-machineinstrs < %s
| FileCheck -check-prefix=GCN %s
----------------
This test surely should not pass for gfx1012, since it does not have these
instructions. And with your patch as written it should fail for gfx1013 too,
since they are predicated on HasGFX10_BEncoding.
@rampitec any idea what is wrong here? Apparently the backend will happily
generate image_bvh_intersect_ray instructions even for gfx900!
================
Comment at: llvm/test/CodeGen/AMDGPU/llvm.amdgcn.intersect_ray.ll:3
+; RUN: llc -march=amdgcn -mcpu=gfx1013 -verify-machineinstrs < %s | FileCheck
-check-prefix=GCN %s
+; RUN: llc -march=amdgcn -mcpu=gfx1012 -verify-machineinstrs < %s | FileCheck
-check-prefix=GCN %s
----------------
Likewise.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D103663/new/
https://reviews.llvm.org/D103663
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
