[lld] [test] Fixes #120514 (PR #125450)

Pengying Xu via llvm-commits llvm-commits at lists.llvm.org
Sun Feb 2 21:00:18 PST 2025


Colibrow wrote:

> > @MaskRay The #120514 failed on some LLD tests on different hosts. I try to not setting the order between s5 and s4 to fix the test issue. Also, remove any redundant items from the orderfile. Could you please review?
> 
> On what hosts do the tests have different behaviors? Why is there a difference?

https://lab.llvm.org/buildbot/#/builders/24/builds/4900
https://lab.llvm.org/buildbot/#/builders/25/builds/6143

> If there is a problem due to DenseMap, DenseSet iteration order, we should fix the source code to avoid host differences.

I haven't found the reason which the problem reproduced on my local mac build which flip the order of s5 and s4..

https://github.com/llvm/llvm-project/pull/125450


More information about the llvm-commits mailing list