[PATCH] D18073: Add memory allocating functions

Anna Zaks via cfe-commits cfe-commits at lists.llvm.org
Thu Mar 10 17:44:11 PST 2016


zaks.anna added a comment.

Since we are adding support for so many new APIs that are only available on Windows, could you please condition checking them only when we build for Windows. You probably can look and Language Options to figure that out.

Also, we should not duplicate all of our tests. Instead, we should add a single test per added API checking that it is modeling the operation we think it should model.


http://reviews.llvm.org/D18073





More information about the cfe-commits mailing list