This is an archive of the discontinued LLVM Phabricator instance.

[lld] fix data race in ELF/ICF.cpp
ClosedPublic

Authored by inglorion on Apr 2 2018, 4:49 PM.

Details

Summary

r328610 fixed a data race in the COFF linker. This change makes a
similar fix to the ELF linker.

Diff Detail

Repository
rLLD LLVM Linker

Event Timeline

inglorion created this revision.Apr 2 2018, 4:49 PM
ruiu accepted this revision.Apr 2 2018, 7:41 PM

LGTM

lld/ELF/ICF.cpp
399 ↗(On Diff #140710)

This looks a bit too dense, so please insert a newline here

402 ↗(On Diff #140710)

and here

This revision is now accepted and ready to land.Apr 2 2018, 7:41 PM

Thanks, ruiu. Added the newlines before committing.

This revision was automatically updated to reflect the committed changes.
This revision was automatically updated to reflect the committed changes.