This is an archive of the discontinued LLVM Phabricator instance.

Don't adjust field offsets for external record layouts
ClosedPublic

Authored by zturner on Sep 29 2015, 5:24 PM.

Diff Detail

Event Timeline

zturner updated this revision to Diff 36054.Sep 29 2015, 5:24 PM
zturner retitled this revision from to Don't adjust field offsets for external record layouts.
zturner updated this object.
zturner added a reviewer: majnemer.
zturner added a subscriber: cfe-commits.
majnemer accepted this revision.Oct 1 2015, 9:36 AM
majnemer edited edge metadata.

LGTM with a test added, see test/CodeGenCXX/override-layout.cpp for an example of how to test external record layout.

This revision is now accepted and ready to land.Oct 1 2015, 9:36 AM

It seems our infrastructure is not up to the task of making this testable via Clang, please commit.

This revision was automatically updated to reflect the committed changes.