[llvm] [AMDGPU] Fix nondeterminism in SIFixSGPRCopies (PR #70644)
Joe Nash via llvm-commits
llvm-commits at lists.llvm.org
Mon Oct 30 07:57:58 PDT 2023
https://github.com/Sisyph commented:
Can you add a test? I assume it would be a flaky test before this patch, but should pass every time after this patch.
https://llvm.org/docs/ProgrammersManual.html#llvm-adt-mapvector-h
The usage manual recommends removing elements in bulk, since with this container removal is slower. Can you extract the removal at L1043 V2SCopies.erase(C.ID)
to a removeList and do it a after the loop?
https://github.com/llvm/llvm-project/pull/70644
More information about the llvm-commits
mailing list