[PATCH] D18073: Add memory allocating functions
Alexander Riccio via cfe-commits
cfe-commits at lists.llvm.org
Mon Aug 8 17:13:18 PDT 2016
ariccio added a comment.
In https://reviews.llvm.org/D18073#504216, @dcoughlin wrote:
> No. The identifier info will be null for C++ operators.
I assume you mean `operator new/new/delete/delete
> > Thus, when`! isWindowsMSVCEnvironment`, I leave the Windows-only memory allocating functions initialized to `nullptr`, which will never equal a non-null `IdentifierInfo*`, and never trigger on a non-Windows platform.
> It is probably better to be explicit about this.
More information about the cfe-commits