[PATCH] D120073: [LLD] Fix for race condition.

Alexander Yermolovich via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 4 17:49:49 PDT 2022


ayermolo added a comment.
Herald added a subscriber: StephenFan.
Herald added a project: All.

Circling back to this. I tried on debug build of clang-15, and didn't see any real difference. 
base
0:10.03 real,	69.61 user,	43.99 sys,	0 amem,	13275464 mmem
0:09.40 real,	68.85 user,	42.86 sys,	0 amem,	13264212 mmem
0:09.61 real,	68.96 user,	44.91 sys,	0 amem,	13271248 mmem

with this change:
0:09.80 real,	68.88 user,	42.82 sys,	0 amem,	13267852 mmem
0:09.62 real,	68.14 user,	44.47 sys,	0 amem,	13270128 mmem
0:09.46 real,	68.71 user,	41.80 sys,	0 amem,	13276648 mmem

I can try a bigger binary?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D120073

STAMPS
actor(@ayermolo) application(Differential) author(@ayermolo) herald(H105) herald(H123) herald(H208) herald(H311) herald(H428) herald(H576) herald(H597) herald(H667) herald(H671) herald(H700) herald(H723) herald(H864) monogram(D120073) object-type(DREV) phid(PHID-DREV-wavldni3mjse36ak3i2e) reviewer(@MaskRay) revision-repository(rG) revision-status(needs-review) subscriber(@arichardson) subscriber(@emaste) subscriber(@hoy) subscriber(@llvm-commits) subscriber(@modimo) subscriber(@StephenFan) subscriber(@wenlei) tag(#all) tag(#llvm) via(web)



More information about the llvm-commits mailing list