[PATCH] D18073: Add memory allocating functions

Alexander Riccio via cfe-commits cfe-commits at lists.llvm.org
Mon May 23 14:06:39 PDT 2016


ariccio added a comment.

I should elaborate. The principle of operation of this latest patch is that the `FunctionDecl` in `IsCMemFunction` should never return a `nullptr` `IdentifierInfo*` from `getIdentifier` (is that a valid assumption?)... 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.


http://reviews.llvm.org/D18073





More information about the cfe-commits mailing list