[PATCH] D158489: [X86][GlobalISel] Move GlobalISel source files to a dedicated subdir

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 22 10:59:23 PDT 2023


MaskRay added a comment.

In D158489#4607498 <https://reviews.llvm.org/D158489#4607498>, @RKSimon wrote:

> In D158489#4607468 <https://reviews.llvm.org/D158489#4607468>, @MaskRay wrote:
>
>> In D158489#4605997 <https://reviews.llvm.org/D158489#4605997>, @RKSimon wrote:
>>
>>> Since only AArch64 + RiscV currently use a subdir - why not just move them up instead?
>>
>> I am confused. `{AArch64,M68k,PowerPC,RISCV}/GISel` are present. This patch creates `X86/GISel/`. What do you suggest?
>
> I'm confused too :) If the plan is to eventually just have GISel why have them in a subdir?

This `X86/GISel` directory makes it clear what files are GlobalISel related. `X86/` contains other files (Subtarget,TargetMachine,X86-specific codegen optimizations,etc) and some SelectionDAG related files.
If eventually we can get rid of SelectionDAG (not sure how feasible it is, but looks like a project of 10+ years), we would still have some `X86/` files.
This `X86/GISel` directory would still be useful to separate GlobalISel related files.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D158489



More information about the llvm-commits mailing list