[PATCH] D80317: [SYCL] Prohibit arithmetic operations for incompatible pointers

John McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu May 21 10:15:37 PDT 2020


rjmccall added a comment.

Please add a C test case just using the address_space attribute.



================
Comment at: clang/include/clang/AST/Type.h:1069
+  /// qualifiers.
+  bool isAddressSpaceOverlapping(const QualType &T) const {
+    Qualifiers Q = getQualifiers();
----------------
It's idiomatic to take `QualType` by value rather than `const &`.

Can you rewrite the `PointerType` method to delegate to this?  Assuming it isn't completely dead, that is.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80317





More information about the cfe-commits mailing list