- User Since
- Jan 14 2020, 12:20 AM (123 w, 3 d)
Mon, May 2
Fix further review comments by Stas.
Fix review comments by Stas and Matt.
Fix review comments (by Matt), by materializing custom lowering of
scalar_to_vector via build_vector since build_vector already specially
handles v8i16/v8f16 types.
Sun, May 1
Fix review comments.
Fri, Apr 29
This patch helped me to understand - what is going on with custom lowering of few vector operations, but, otherwise, the change within this patch itself for INSERT_SUBVECTOR does not any make sense. Hence abandoning it. We be coming up with update patch(es).
Thu, Apr 28
Apr 21 2022
Apr 20 2022
Fix minor comments by Matt and CD.
Apr 19 2022
Rebase to D123973 and update lit tests accordingly.
Apr 18 2022
Within spill-vgpr.ll, remove 2>&1 from RUN line for gfx900.
Usage of AGPR in reject-agpr-usage-before-gfx908.ll should also fail for gfx906.
Apr 16 2022
Apr 15 2022
Rebase to latest trunk and to D123809.
Apr 14 2022
Restore spill-vgpr-to-agpr.ll and spill-agpr.ll.
Move code related to shifting of reserved VGPR to a lower range within FrameLowering.
Apr 13 2022
Fix assert message.
Minor update to member initialization.
Fix few review comments by CD.
Apr 12 2022
Here is the update where we always reserve highest avaialble VGPR irrespective of the
Apr 11 2022
Refine the logic within indirectCopyToAGPR() while chosing lowest available
VGPR for AGPR copy. Basically, find out the total avaialble unused free
VGPRs (after RegAlloc), and call scavenger (only) that many times to find
the lowest available unused free VGPR.
Mar 25 2022
Mar 24 2022
Fix review comments by Stas.
If I understand this patch correctly, the main intention of this patch is to check if the entry point function makes any call(s) to device functions, and if so, create and allocate module lds. Otherwise don't.
Mar 23 2022
Fix review comments by Jay.
hasGFX90AInsts() is true for both GFX90A and GFX940 subtargets.
Mar 22 2022
Rebase and remove newly added confusing TODO comments.
Fix further review comments by Matt.
Fix review comments by Matt.
Mar 21 2022
Jan 4 2022
Nov 16 2021
I do not think this patch makes any progress. Enough is enough, I am abandoning it.
Nov 15 2021
Nov 9 2021
Minor changes to comment.
Hi @arsenm, The pre-requisite front-end patch (https://reviews.llvm.org/D110257) has landed to llvm trunk. Now, this patch needs an action to be taken. Please have a look at this patch during your free time.
Nov 7 2021
Nov 3 2021
This patch is waiting for an action for a long time. I am expecting at-least anyone of the reviewers to either say "yes" or "no" or "further required improvements" on this patch so that I can take further action on this patch. If you say "no" to this patch with a strong valid reason, then I will abandon this patch and move-on instead of uncertainly waiting.