[PATCH] D134950: [CodeGen] Allow targets to define vreg flags
Quentin Colombet via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Oct 24 15:56:07 PDT 2022
qcolombet added a comment.
Hi,
> It seems like a more complicated way to propagate the flags.
It is definitely more complicated, but it doesn't bake any target specific knowledge in the generic code. In particular, in your case, you don't need any merging of the flags during coalescing and whatnot, but that may not be true for other targets if they want to adopt it and I'm not willing to open that door at the moment. Hence the suggested trade-off.
What I'm suggesting is:
- Create your own pass that would attach your delegate to MRI before the point you need it.
- Do whatever you want with your delegate.
- Create another pass to unregister your delegate (you may even leave it there if it doesn't hurt anything.)
Cheers,
-Quentin
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D134950/new/
https://reviews.llvm.org/D134950
More information about the llvm-commits
mailing list