[PATCH] D138868: AMDGPU/clang: Remove target features from address space test builtins

Joe Nash via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Dec 29 12:38:06 PST 2022


Joe_Nash accepted this revision.
Joe_Nash added inline comments.
This revision is now accepted and ready to land.


================
Comment at: clang/test/CodeGenOpenCL/builtins-amdgcn-flat-address-space.cl:8
+// be initialized to something useful. The proper way to diagnose invalid flat
+// usage is to forbid flat pointers on unsupported targets.
+
----------------
arsenm wrote:
> Joe_Nash wrote:
> > What part of the toolchain is responsible for forbiding flat pointers on unsupported targets?
> The frontend should just error on languages with flat pointers. Really we should just implement software flat pointers, it wouldn’t be difficult and might only require minimal driver cooperation if any 
Ok, I just want to ask if that is in place or should be in place before this patch lands. So we don't accidentally convert a compile time error into a runtime error. Otherwise LGTM.


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

https://reviews.llvm.org/D138868



More information about the cfe-commits mailing list