[PATCH] D127915: [windows][support] Improve backtrace emitted in crash report without llvm-symbolizer
    David Blaikie via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Fri Jun 17 15:39:58 PDT 2022
    
    
  
dblaikie added a comment.
In D127915#3592534 <https://reviews.llvm.org/D127915#3592534>, @rnk wrote:
> lgtm
>
> +1 for the exception code. I think it may appear as the process return code and some systems will log that, but logging it ourselves, too, is for the best.
>
> I think we could probably go further here. The unsymbolized output when we use llvm-symbolizer and it can't find PDB files is practically the same as the new output with your change. It would be nice to be able to fetch symbols and then re-pipe the crash report text through llvm-symbolizer and get a symbolized stack trace. That can be future work, this is a nice improvement as is.
That sounds like this: https://reviews.llvm.org/D126980 perhaps? (we could change the crash dump output to include the markup required for the filtering, specifically when we can't already symbolize at crash-time)
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D127915/new/
https://reviews.llvm.org/D127915
    
    
More information about the llvm-commits
mailing list