This is an archive of the discontinued LLVM Phabricator instance.

[lld-link] Warn on exported deleting dtor
ClosedPublic

Authored by zequanwu on Apr 15 2021, 4:25 PM.

Details

Summary

MSVC linker has this warning, so lld-link should also warn on this.

Diff Detail

Event Timeline

zequanwu requested review of this revision.Apr 15 2021, 4:25 PM
zequanwu created this revision.
Herald added a project: Restricted Project. · View Herald TranscriptApr 15 2021, 4:25 PM
rnk added a comment.Apr 16 2021, 3:31 PM

Seems reasonable, but I see harbormaster test failures.

zequanwu updated this revision to Diff 338269.Apr 16 2021, 7:05 PM

Update testcase.

rnk added a comment.Apr 19 2021, 1:54 PM

Seems reasonable, but I see harbormaster test failures.

The export.test seems to be crashing still both on Debian and Windows. Do you see that locally? Maybe it needs assertions to be enabled/disabled?

zequanwu updated this revision to Diff 338658.Apr 19 2021, 3:27 PM
  • Check e.sym for null. If the flag /export=foo is provided and foo is not in the object file. e.sym will be null.
  • Remove $ in exported name to have same name on windows and linux.
rnk accepted this revision.Apr 20 2021, 12:57 PM

lgtm

This revision is now accepted and ready to land.Apr 20 2021, 12:57 PM
This revision was automatically updated to reflect the committed changes.