[PATCH] D119053: [nfc][codegen] Move RegisterBank[Info].cpp under CodeGen
Min-Yih Hsu via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Feb 15 10:04:54 PST 2022
myhsu added a comment.
Just want to double check: You have a (possibly downstream) use case that wants to leverage RegisterBank without linking GlobalISel, is that correct?
My two-cent is that since none of the upstream code uses RegisterBank outside GlobalISel context, I don't see much justification on making this change. That said, if this situation changes in the upstream in the future (i.e. some other components also use RegisterBank) moving to CodeGen might be a way.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D119053/new/
https://reviews.llvm.org/D119053
More information about the llvm-commits
mailing list