[PATCH] D46651: [OpenCL] Support placement new/delete in Sema

Sven van Haastregt via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri May 11 10:41:36 PDT 2018


svenvh added inline comments.


================
Comment at: lib/Sema/SemaExprCXX.cpp:2030
+      }
+    }
 
----------------
rjmccall wrote:
> I think a better interpretation of this rule would be to just error on attempts to use the standard non-placement operator new/delete instead of trying to outlaw the operator declarations.  For example, I don't know why a user-defined non-global operator new would be problematic.
Good point, I have raised this with Khronos, so I will hold this off until we have clarification.


Repository:
  rC Clang

https://reviews.llvm.org/D46651





More information about the cfe-commits mailing list