This revision was automatically updated to reflect the committed changes.
Closed by commit rL332417: [analyzer] Make plist-html diagnostic consumer
produce multi-file reports. (authored by dergachev, committed by ).
Herald added a subscriber: llvm-commits.
Changed prior to commit:
https://revie
george.karpenkov accepted this revision.
george.karpenkov added a comment.
This revision is now accepted and ready to land.
LGTM! with stable-filename option we could even avoid the regexp.
Repository:
rC Clang
https://reviews.llvm.org/D46902
___
NoQ added inline comments.
Comment at: test/Analysis/diagnostics/plist-multi-file.c:202
+// CHECK-NEXT:
+// CHECK-NEXT: report-{{([0-9a-f]{6})}}.html
+// CHECK-NEXT:
Yay we have regular expressions.
`()` are needed so that not to confuse `}``}}` with `}}``}
NoQ created this revision.
NoQ added reviewers: dcoughlin, xazax.hun, a.sidorin, george.karpenkov, szepet,
rnkovacs.
Herald added subscribers: cfe-commits, baloghadamsoftware.
Plist diagnostics support multi-file reports since forever. HTML diagnostics
support multi-file reports since
https://r
