[llvm-commits] [llvm] r71610 - in /llvm/trunk: include/llvm/CodeGen/MachineFunction.h include/llvm/Intrinsics.td lib/CodeGen/MachineFunction.cpp lib/CodeGen/PrologEpilogInserter.cpp lib/CodeGen/SelectionDAG/SelectionDAGBuild.cpp lib/Target/ARM/AR

[Eli Friedman]

On Wed, May 13, 2009 at 11:09 AM, Jim Grosbach <grosbach at apple.com> wrote:
>>> +let Properties = [IntrNoMem] in {
>>> +def int_builtinsetjmp  : Intrinsic<[llvm_i32_ty],  [llvm_ptr_ty]>;
>>> +def int_builtinlongjmp : Intrinsic<[llvm_void_ty], [llvm_ptr_ty,
>>> llvm_i32_ty]>;
>>
>> If there is no reason to have both arguments, please drop the extra
>> one.
>>
>
> I'm not sure I follow. builtinsetjmp takes a single argument, and
> builtinlongjmp takes two. All arguments will be used.

How exactly will the second argument to builtinlongjmp be used?  gcc
requires that the second argument to __builtin_longjmp is a constant
1.

-Eli