[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