From 0c85c488e2b5be54ce26c9f772e66d48a9290c51 Mon Sep 17 00:00:00 2001 From: "Duncan P. N. Exon Smith" Date: Thu, 2 Apr 2020 10:27:42 -0700 Subject: [PATCH] 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 --- llvm/utils/clang-parse-diagnostics-file | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/llvm/utils/clang-parse-diagnostics-file b/llvm/utils/clang-parse-diagnostics-file index 59b13f306505..1f720c34544a 100755 --- a/llvm/utils/clang-parse-diagnostics-file +++ b/llvm/utils/clang-parse-diagnostics-file @@ -66,6 +66,10 @@ Utility for dumping Clang-style logged diagnostics.\ for file_diags in diags: file = file_diags.get('main-file') + # Diagnostics from modules don't have a main-file listed. + if not file: + file = '' + # Ignore diagnostics for 'conftest.c', which is the file autoconf uses # for its tests (which frequently will have warnings). if os.path.basename(file) == 'conftest.c':