steakhal added a comment.

In D118996#3473934 <https://reviews.llvm.org/D118996#3473934>, @whisperity 
wrote:

> After adding improvements to the documentation, I think this will be good to 
> go, and thank you! Perhaps just for a safety measure you could run it on a 
> few projects (LLVM itself?) to ensure we didn't miss a case where it might 
> magically crash, but I wonder how many specifically "C++14" projects will use 
> signal handlers in the first place.

It did not crash on llvm, nor on our internal test set.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D118996

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

Reply via email to