[PATCH] D26196: AMDGPU: Translate null pointers in private and local addr space

Matt Arsenault via cfe-commits cfe-commits at lists.llvm.org
Tue Nov 1 11:05:46 PDT 2016


arsenm added inline comments.


================
Comment at: test/CodeGenOpenCL/amdgpu-nullptr.cl:49
+}
+
----------------
I think there need to be a lot more tests. A few I can think of:

  - Tests that compare to NULL instead of 0
  - Pointer compares without the explicit 0, i.e. if (p)/if(!p)
  - Storage of a pointer value
  - Calls with a pointer argument
  - Set of tests with constant address space pointers
  - Cast null to integer
  - Compare of null with null
  - Compare null in one address space with null in another





https://reviews.llvm.org/D26196





More information about the cfe-commits mailing list