Skip to content

Commit 0c85c48

Browse files
committed
utils: Tweak clang-parse-diagnostics-file for modules includes
Diagnostics from modules do not have a `main-file` listed. Tweak `clang-parse-diagnostics-file` to patch this up. Previously, the call to `os.path.basename` would crash. Radar-Id: rdar://problem/59000292
1 parent 90455db commit 0c85c48

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

llvm/utils/clang-parse-diagnostics-file

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -66,6 +66,10 @@ Utility for dumping Clang-style logged diagnostics.\
6666
for file_diags in diags:
6767
file = file_diags.get('main-file')
6868

69+
# Diagnostics from modules don't have a main-file listed.
70+
if not file:
71+
file = '<module-includes>'
72+
6973
# Ignore diagnostics for 'conftest.c', which is the file autoconf uses
7074
# for its tests (which frequently will have warnings).
7175
if os.path.basename(file) == 'conftest.c':

0 commit comments

Comments
 (0)