ahh added a comment.

On my workstation's checkout of head, one test fails (Clang :: 
Driver/response-file.c) both with and without this change; everything else 
appears to pass.

I believe that between the tests I add to delete.cpp and the ones that are 
already there (and destroying-delete.cpp) we cover every case that has to get a 
nullptr check, and pretty much every one that should *not*.

Name of the helper function is, uh, good enough for me, but no objections to 
changing it.


Repository:
  rC Clang

https://reviews.llvm.org/D43430



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to