[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 mailing list