[llvm-commits] [llvm] r66735 - in /llvm/trunk: lib/Target/X86/AsmPrinter/X86ATTAsmPrinter.cpp lib/Target/X86/X86Instr64bit.td lib/Target/X86/X86InstrInfo.td test/CodeGen/X86/call-imm.ll

[Evan Cheng]

On Mar 12, 2009, at 9:12 AM, Dan Gohman wrote:

>
> On Mar 11, 2009, at 9:58 PM, Evan Cheng wrote:
>
>>
>> On Mar 11, 2009, at 4:01 PM, Dan Gohman wrote:
>>
>>> Author: djg
>>> Date: Wed Mar 11 18:01:47 2009
>>> New Revision: 66735
>>>
>>> URL: http://llvm.org/viewvc/llvm-project?rev=66735&view=rev
>>> Log:
>>> Revert r66024. The JIT encoding for CALLpcrel32 is wrong -- see
>>> PR3773, and the
>>
>> Ok.
>>
>>> assembly text output uses an indirect call ("call *") instead of a
>>> direct call.
>>
>> That should be fine.  "call 0x1234" is not legal assembly,
>
> It is legal.
>
>> "call
>> *0x1234" really would branch to address 0x1234 as far as I can tell.
>
> call *0x1234 loads the destination from 0x1234.

Ah, alright. Thanks.

Evan

>
> Dan
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits