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



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

Reply via email to