[PATCH] D90760: [InstCombiner] Make LibCallSimplifier add extension attribute to ldexp arg.

[Jonas Paulsson via Phabricator via llvm-commits]

jonpa added a comment.

In D90760#2382851 <https://reviews.llvm.org/D90760#2382851>, @lebedev.ri wrote:

> In D90760#2382819 <https://reviews.llvm.org/D90760#2382819>, @jonpa wrote:
>
>> ping
>>
>> @lebedev.ri   Thanks for review so far. Since this is keeping a SystemZ buildbot failing, I would appreciate some further advice here... thanks
>
> Given a function declaration, (e.g. that `@ldexp`), will/should all calls agree how the argument should be extended?

I believe all calls to @ldexp should have the signext attribute. It is then up to the target to perform the sign extension if needed per the calling convention. The attribute is needed to inform the target what kind of extension should be performed (sign/zero).


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D90760/new/

https://reviews.llvm.org/D90760