[PATCH] D71178: [x86] Don't assume sign-extension of arguments smaller than 32-bits.

Emilio Cobos Álvarez via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Apr 2 04:40:59 PDT 2022


emilio added a comment.

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

> This seems backwards.
> Who added those `signext`/`zeroext` attrs to the function arguments?
> Presumably they just shouldn't be added if that is not what ABI states.

Yes, see D72742 <https://reviews.llvm.org/D72742> for the right approach.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71178



More information about the llvm-commits mailing list