[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

[Jim Grosbach]

On May 13, 2009, at 6:37 PM, Eli Friedman wrote:

> 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?

Probably not, but this allowed me to get it working for now for now as  
a first step that let me get far enough to unwind SjLj stack frames.  
More accurately modeling the specifics is one of the things I'm  
planning on tweaking in a follow-up.

Thanks!

-Jim

>
>
>> -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
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits