[PATCH] D91053: [PowerPC] Lump the constants to save one addis for each constant access

Nemanja Ivanovic via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 24 03:07:17 PST 2023


nemanjai added a comment.

In D91053#4068538 <https://reviews.llvm.org/D91053#4068538>, @nemanjai wrote:

> In D91053#3962199 <https://reviews.llvm.org/D91053#3962199>, @stefanp wrote:
>
>> I'm going to commandeer this patch and rebase it to see if I can revive it.
>
> @stefanp I think you're working on a re-implementation of this. Since the implementation will likely be fundamentally different, perhaps it would be good to just abandon this patch.

I believe you know more about the planned direction for this patch now. Is it time to abandon this or are you plan to re-work this patch in some way?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D91053



More information about the llvm-commits mailing list