[PATCH] D18073: Add memory allocating functions

[Alexander Riccio via cfe-commits]

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.





https://reviews.llvm.org/D18073