[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