[PATCH] D78010: [CodeGen] Add new function unionImplicitOps() to union implicit register
Matt Arsenault via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jun 2 13:10:43 PDT 2020
arsenm added a comment.
In D78010#2029073 <https://reviews.llvm.org/D78010#2029073>, @ZhangKang wrote:
> @arsenm , I have removed calling unionImplicitOps() in PPCEarlyReturn.cpp.
> Now there is no scene to call unionImplicitOps().
> Do you think I should abandon this patch?
Yes, unless there's a concrete use case (I can maybe imagine one, but I wouldn't encourage use of this without a better idea)
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D78010/new/
https://reviews.llvm.org/D78010
More information about the llvm-commits
mailing list