[PATCH] D100016: [GreedyRA ORE] Separate Folder Reloads and Zero Cost Folder Reloads

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 8 13:22:31 PDT 2021


reames added a comment.

In D100016#2676314 <https://reviews.llvm.org/D100016#2676314>, @skatkov wrote:

> In D100016#2675709 <https://reviews.llvm.org/D100016#2675709>, @reames wrote:
>
>> 
>
> Actually I tried to be consistent with other statistic. All other stat collect number of instructions, not number of stack slot used.

I think this code may simply be expecting that no instruction has one than more folded stack operand.  For most architectures I'm aware of, that's true for all the actual instructions.


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

https://reviews.llvm.org/D100016



More information about the llvm-commits mailing list