[PATCH] D86657: Add new hidden option -print-crash-IR that prints out IR that caused opt pipeline to crash

Jamie Schmeiser via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 27 14:07:32 PDT 2020


jamieschmeiser added inline comments.


================
Comment at: llvm/lib/Passes/StandardInstrumentations.cpp:362
 
-  sys::AddSignalHandler(CrashedIRReporter, this);
+  sys::AddSignalHandler(crashedIRReporter, this);
 
----------------
yrouban wrote:
> do we need to remove this signal handler when //this// is destructed?
I don't think it is necessary because typically, control does not return from the signal handler and destructors are not run.


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

https://reviews.llvm.org/D86657



More information about the llvm-commits mailing list