[llvm] [AMDGPU] [SIFrameLowering] Use LiveRegUnits instead of LivePhysRegs (PR #65962)
Pranav Taneja via llvm-commits
llvm-commits at lists.llvm.org
Thu Sep 14 23:41:28 PDT 2023
pranavTanejaAMD wrote:
> > > Whole stack LGTM. Thanks for working on this! Have you measured any compile-time difference from this change?
> >
> >
> > [compileTimeDiff.log](https://github.com/llvm/llvm-project/files/12595643/compileTimeDiff.log)
> > Here's the difference between compile time from both the original and modified builds on compiling tests from llvm-test-suite
>
> So didn't really do anything for x86, which is expected. Ideally we would have GPU compute compile time tracking but we don't
If it serves as a proxy for AMDGPU compilation time, here's the difference in time it takes to complete testing the lit tests in "llvm/test/CodeGen/AMDGPU", with the modified branch with LiveRegUnits taking about a second less in multiple trials.
[litTestingTime.patch](https://github.com/llvm/llvm-project/files/12616379/litTestingTime.patch)
https://github.com/llvm/llvm-project/pull/65962
More information about the llvm-commits
mailing list