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

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 9 08:19:46 PST 2020


lebedev.ri added a comment.

In D90760#2383015 <https://reviews.llvm.org/D90760#2383015>, @jonpa wrote:

>> So, can you try if the miscompile goes away if you manually annotate the @ldexp declaration in IR with said signext?
>
> It seems that it is enough to have either the signext on the call operand or in the declaration, for the extension to be emitted by the SystemZ backend.

Great.
So back to my original idea - can we simply annotate the `LibFunc_ldexp*`'s argument with `signext` attribute?
@xbolva00 might be able to help.


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

https://reviews.llvm.org/D90760



More information about the llvm-commits mailing list