ChuanqiXu accepted this revision.
ChuanqiXu added a comment.
This revision is now accepted and ready to land.

In D108905#2971773 <https://reviews.llvm.org/D108905#2971773>, @MaskRay wrote:

> In D108905#2971761 <https://reviews.llvm.org/D108905#2971761>, @ChuanqiXu 
> wrote:
>
>> Thanks for looking into this!
>> I am not familiar with exception handling. But from the comments, I worried 
>> that if it is a problem if there are exceptions whose destructor could throw 
>> (although it is not usual nor good).
>
> GCC does not emit call site records for the `__cxa_end_catch` region, so I 
> believe it makes the same assumption.
> Setting `nounwind` should be safe...

Make sense to me. Thanks for looking into this!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D108905

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

Reply via email to