[PATCH] Rename eh_sjlj_{setjmp|longjmp} intrinsic to lightweight_{setjmp|longjmp}

John McCall rjmccall at gmail.com
Mon May 18 14:05:06 PDT 2015


I can't review changes to LLVM targets.

I agree with not naming the intrinsics after __builtin_setjmp / __builtin_longjmp, because using the word "builtin" in a list of intrinsics feels wrong, but please do mention in the documentation that these intrinsics implement __builtin_setjmp / __builtin_longjmp and are not their own new thing.


REPOSITORY
  rL LLVM

http://reviews.llvm.org/D9589

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list