[PATCH] D83100: [PPC][GlobalISel] Add initial GlobalIsel infrastructure

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 6 13:11:09 PDT 2020


arsenm added a comment.

In D83100#2134028 <https://reviews.llvm.org/D83100#2134028>, @kbarton wrote:

> In D83100#2130726 <https://reviews.llvm.org/D83100#2130726>, @tschuett wrote:
>
> > AArch64 has a sub-directory for GlobalISel related things:
> >  https://reviews.llvm.org/D81116
>
>
> I like this idea. I will do the refactoring in a subsequent commit and update the patch, so if others do not like it it's easy to undo.


I don't particularly like the AArch64 split. There isn't an entirely clean break between globalisel parts and the rest of the backend. The other target subdirectories are used for places where there's a separate library


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83100





More information about the llvm-commits mailing list