[PATCH] D12906: [RFC] Bug identification("issue_hash") change for CmpRuns.py

Honggyu Kim via cfe-commits cfe-commits at lists.llvm.org
Mon Sep 21 21:30:40 PDT 2015


honggyu.kim added a comment.

In http://reviews.llvm.org/D12906#250237, @zaks.anna wrote:

> I agree with Gabor and do not think we should have 2 entries in the plist file for bug identification. This is confusing and I do not see a need for this.
>
> Any comparison script can check for 2 fields in a plist file and concatenate them. We've already discussed this during the patch review for the other patch. For example, the latest version does not include the file name.


I didn't put 2 entries in plist, I made issue_hash as it was while adding bugid to html report only.
So this patch makes issue_hash based on Babati's patch after removing redundent fields. And it also makes BugID that contains all the information so that it doesn't require post processing.
BugId is for HTML report comparison.

issue_hash is a subset of BugId and CmpRuns.py script will collect more fields with issue_hash so that it contains all information just like BugId here.

Thanks.


http://reviews.llvm.org/D12906





More information about the cfe-commits mailing list