[PATCH] D122963: [X86] Extend the integer parameter if the function isn't local linked

John McCall via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 19 22:10:30 PDT 2022


rjmccall added a comment.

If people think that we'll reliably get the right behavior by the frontend adding `sext` and `zext` to call sites but not to function arguments, I can walk you through how to make that change in the frontend.

What we can't do in the frontend is just switch to unconditionally not using `sext` and `zext` at all on x86_64, because we need to maintain compatibility with old versions of clang.  But we can introduce a flag that makes clang generate incompatible code if someone wants to opt in to it for performance.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D122963



More information about the llvm-commits mailing list