[PATCH] D91924: [X86] Have indirect calls take 64-bit operands in 64-bit modes

Harald van Dijk via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 26 14:58:21 PST 2020


hvdijk added a comment.

In D91924#2418221 <https://reviews.llvm.org/D91924#2418221>, @RKSimon wrote:

> We already have fast-isel-call tests - could we add gnux32 test coverage there?

I took a look, and none of them would be affected by this change (none of them use PIC or indirect calls). It does seem like it might be a useful separate improvement, but it's a bit tricky: `fast-isel-call.ll` tests x86-specific calling conventions so probably wouldn't work, `fast-isel-call-bool.ll` and `fast-isel-call-cleanup.ll` look like they test for specific bugs that may or may not be/have been a problem for x32.

(For completeness, I mentioned it already but not in the description of this diff: this change is covered by a test, this fixes a problem exposed by `pic.ll`.)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D91924



More information about the llvm-commits mailing list