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



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

Reply via email to