hliao added inline comments.

================
Comment at: clang/test/CodeGenCUDA/amdgpu-kernel-arg-pointer-type.cu:19
+// COMMON-LABEL: define amdgpu_kernel void @_Z7kernel1Pi(i32*{{.*}} %x)
+// OPT: [[VAL:%.*]] = load i32, i32* %x, align 4
 // OPT: [[INC:%.*]] = add nsw i32 [[VAL]], 1
----------------
arsenm wrote:
> hliao wrote:
> > arsenm wrote:
> > > This is still a regression. Fixing up AA does not solve the problem this 
> > > promotions this is intended to solve. Generic accesses are worse 
> > > independently of the aliasing properties
> > Do you mean FLAT load/store has worse addressing mode than GLOBAL ones?
> Yes. The flat offsets have a smaller range, and do not have the saddr mode. 
> Flat accesses also won't avoid the extra lgmkcnt wait
I plan to add support to select GLOBAL ones once we could confirm that pointer 
could only point to GLOBAL/CONSTANT address spaces. Do you think that's a 
reasonable solution?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D89980/new/

https://reviews.llvm.org/D89980

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to