[llvm-commits] [llvm] r71758 - in /llvm/trunk: docs/ExceptionHandling.html include/llvm/Intrinsics.td lib/Target/ARM/ARMISelLowering.cpp lib/Target/ARM/ARMISelLowering.h lib/Target/ARM/ARMInstrInfo.cpp lib/Target/ARM/ARMInstrInfo.td

Eli Friedman eli.friedman at gmail.com
Wed May 13 18:37:21 PDT 2009


On Wed, May 13, 2009 at 5:46 PM, Jim Grosbach <grosbach at apple.com> wrote:
>  let Properties = [IntrNoMem] in {

Not that this is a part of the current patch, but is it really
accurate to mark the intrinsics as not touching memory?

> -def int_builtinsetjmp  : Intrinsic<[llvm_i32_ty],  [llvm_ptr_ty]>;
> -def int_builtinlongjmp : Intrinsic<[llvm_void_ty], [llvm_ptr_ty, llvm_i32_ty]>;
> +def int_eh_sjlj_setjmp  : Intrinsic<[llvm_i32_ty],  [llvm_ptr_ty]>;
> +def int_eh_sjlj_longjmp : Intrinsic<[llvm_void_ty], [llvm_ptr_ty, llvm_i32_ty]>;
>  }

-Eli




More information about the llvm-commits mailing list