[llvm] r235483 - [patchpoint] Add support for symbolic patchpoint targets to SelectionDAG and the

Philip Reames listmail at philipreames.com
Wed Jul 15 09:44:48 PDT 2015


On 07/11/2015 09:18 PM, Hal Finkel wrote:
>> +; Trivial symbolic patchpoint codegen.
>> >+;
>> >+
>> >+declare i64 @foo(i64 %p1, i64 %p2)
>> >+define i64 @trivial_symbolic_patchpoint_codegen(i64 %p1, i64 %p2) {
>> >+entry:
>> >+; CHECK-LABEL: trivial_symbolic_patchpoint_codegen:
>> >+; CHECK:       movabsq $_foo, %r11
>> >+; CHECK-NEXT:  callq   *%r11
> Wouldn't it be better just to emit:
>
>    callq	_foo
>
> instead of the move and then the indirect call?
Does look that way unless this is for a large code model.  Can you file 
a bug?
>
>   -Hal
>




More information about the llvm-commits mailing list