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

Serguei Katkov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 9 01:26:13 PDT 2021


skatkov updated this revision to Diff 336339.
skatkov edited the summary of this revision.
skatkov added a comment.

@reames, I moved from instruction computation to stack slot usages.
@thegameg, I landed NFC before to move from if-else to if-continue. Is it readable now or you still want me to separate added logic into utility function?

I've added output of register allocator to the test to be able to verify the correctness of the stat manually.

By some reason utils/update_llc_test_checks.py does not work for me, so I did it manually.


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

https://reviews.llvm.org/D100016

Files:
  llvm/lib/CodeGen/RegAllocGreedy.cpp
  llvm/test/CodeGen/X86/statepoint-ra.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D100016.336339.patch
Type: text/x-patch
Size: 14976 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210409/f8b400af/attachment.bin>


More information about the llvm-commits mailing list