[PATCH] D14919: Fix IssueHash generation

Sean Eveson via cfe-commits cfe-commits at lists.llvm.org
Thu Nov 26 01:49:13 PST 2015


seaneveson added a comment.

In http://reviews.llvm.org/D14919#296597, @o.gyorgy wrote:

> I did not create a test checker for the NormalizeLine error in the patch.
>  Should I add a test checker for this?


I was wondering what sort of source code causes the crash, but I do think it would be good to have a regression test.

> Do you have any suggestions where to put it if required?


Maybe in bug_hash_test.cpp, although I'm not sure its necessary to check the plist output when testing for a crash.


http://reviews.llvm.org/D14919





More information about the cfe-commits mailing list