This is an archive of the discontinued LLVM Phabricator instance.

[IRMover] Implement name based structure type mapping
ClosedPublic

Authored by evgeny777 on Feb 12 2018, 10:02 AM.

Details

Summary

This patch was separated from D43077 after discussion with @tejohnson. It implements name based structure type mapping to
avoid mapping to wrong type which is isomorphic to one we're about to import.

Not sure whom to add for review.

Diff Detail

Event Timeline

evgeny777 created this revision.Feb 12 2018, 10:02 AM

+rafael who is a bit more familiar with this part of the IR linker.

evgeny777 updated this revision to Diff 134196.Feb 14 2018, 4:24 AM
evgeny777 added a reviewer: dblaikie.
evgeny777 added a subscriber: dblaikie.

Rebased + added @dblaikie

evgeny777 updated this revision to Diff 134444.Feb 15 2018, 9:27 AM

Addressed review comments in the mailing list:

  • added --initial-module flag to llvm-link
  • test cases now use llvm-link instead of llvm-lto
evgeny777 accepted this revision.Feb 21 2018, 7:15 AM

Approved in the mailing list

This revision is now accepted and ready to land.Feb 21 2018, 7:15 AM
This revision was automatically updated to reflect the committed changes.