[PATCH] D81672: [Driver] When forcing a crash print the bug report message

John Brawn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 25 12:29:07 PDT 2020


john.brawn marked an inline comment as done.
john.brawn added inline comments.


================
Comment at: clang/tools/driver/driver.cpp:515
+
+      llvm::dbgs() << llvm::getBugReportMsg();
     }
----------------
MaskRay wrote:
> Why ` llvm::dbgs() << llvm::getBugReportMsg();` when -gen-reproducer is specified? The user requests to generate a reproduce file, but this does not suggest that clang has a bug. Dumping an URL is not very appropriate.
Emitting the bug report message is primarily for when FORCE_CLANG_DIAGNOSTICS_CRASH is set (because we can get here either due to that or -gen-reproducer). I could adjust this to only emit the message when that is set if you'd like? Though either way we get the "PLEASE ATTACH THE FOLLOWING FILES TO THE BUG REPORT:" message.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D81672/new/

https://reviews.llvm.org/D81672





More information about the llvm-commits mailing list