[PATCH] D43430: Omit nullptr check for sufficiently simple delete-expressions

[Andrew Hunter via Phabricator via cfe-commits]

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