ariccio added a comment. In http://reviews.llvm.org/D18073#393613, @zaks.anna wrote:
> You will have to add one test function to smoke test that the newly added API > is modeled correctly. Isn't that what I've already done? > We also have a lot of existing tests that verify that each of the original > APIs (malloc. free, new) function correctly in all possible settings. > What is the contradiction in asking to add the smoke tests in a separate > file? Ahh, I was confused as to whether you wanted me to remove them entirely - moving them to a different file is self-consistent. I'm not exactly sure of the benefit of moving these tests to a separate file? Sidenote: As someone who has worked a little bit with electronics and simple hardware design & construction, I love this use of "smoke test". http://reviews.llvm.org/D18073 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits