[PATCH] D97108: [Statepoint Lowering] Allow dead gc pointer from deopt section to be on register.

Serguei Katkov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 26 06:39:33 PST 2021


skatkov added a comment.

Philip, I've published alternative patch https://reviews.llvm.org/D97554 wit your idea to show the impact.
>From my point of view it looks better in terms that we explicitly process deopt gc pointers as other gc values.
The disadvantages is that now we will have a lot of dead tied-def registers and deopt value will be listed twice in uses.

So generally the approach you proposed is a bit cleaner in understanding but costs a bot more in terms of compile time.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D97108/new/

https://reviews.llvm.org/D97108



More information about the llvm-commits mailing list