[PATCH] D46651: [OpenCL] Support placement new/delete in Sema
Anastasia Stulova via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Jun 8 07:36:19 PDT 2018
Anastasia requested changes to this revision.
Anastasia added inline comments.
This revision now requires changes to proceed.
================
Comment at: lib/Sema/SemaExprCXX.cpp:2030
+ }
+ }
----------------
svenvh wrote:
> 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.
The decision by Khronos is to allow all user defined new/delete operators (even global). I have submitted the change to the spec. The next publishing date is however in July.
Repository:
rC Clang
https://reviews.llvm.org/D46651
More information about the cfe-commits
mailing list