[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