uweigand wrote: I like the idea in general, but I'm a bit confused about why we now have two separate intrinsics: - it seems odd to have to pair `setjmp` with `eh_sjlj_longjmp` - nobody is using `eh_sjlj_setjmp` any more, so what's the point of keeping all the related code
Why can't we just modify the definition of `eh_sjlj_setjmp` ? Even if some old front-end are still using it expecting the old semantics, the new semantics is compatible (it would just be little less efficient). Either way, the documentation needs to be updated to explain what the intrinsics do and how they should be used. https://github.com/llvm/llvm-project/pull/186843 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
