[PATCH] D112860: [X86] Fix X32 indirect call generation

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 3 10:23:42 PDT 2021


MaskRay added inline comments.


================
Comment at: llvm/test/CodeGen/X86/call-structfp.ll:1
+; NOTE: Assertions have been autogenerated by utils/update_llc_test_checks.py
+; RUN: llc < %s -mtriple=x86_64-unknown-linux-gnux32 | FileCheck %s
----------------
hvdijk wrote:
> MaskRay wrote:
> > Consider moving this into `x32-function_pointer-*.ll`
> Sure. This comment was after the push, so it'd have to be a new separate change, but I could rename the test to `x32-function_pointer-4.ll` and add a `-fast-isel` `RUN` line like the other `x32-function_pointer-*.ll` tests if you like.
(When I pushed the comment with "Accept", the revision had been closed.)

May not need a separate file. May just place several cases into one file. It is sometimes easier for readers to follow when placed in one file testing multiple variants.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D112860



More information about the llvm-commits mailing list