This is an archive of the discontinued LLVM Phabricator instance.

Workaround an assertion failure during module build
ClosedPublic

Authored by steven_wu on Dec 13 2022, 10:37 AM.

Details

Summary

After the change in https://reviews.llvm.org/D131858, clang cannot
bootstrap itself with modules due to assertion failure:
(lvaluePath->getType() == elemTy && "Unexpected type reference!")

Workaround the assertion by converting some of the includes into forward
declares.

Diff Detail

Event Timeline

steven_wu created this revision.Dec 13 2022, 10:37 AM
Herald added a project: Restricted Project. · View Herald TranscriptDec 13 2022, 10:37 AM
Herald added a subscriber: ributzka. · View Herald Transcript
steven_wu requested review of this revision.Dec 13 2022, 10:37 AM
Herald added a project: Restricted Project. · View Herald TranscriptDec 13 2022, 10:37 AM
arphaman accepted this revision.Dec 13 2022, 10:41 AM
This revision is now accepted and ready to land.Dec 13 2022, 10:41 AM
This revision was landed with ongoing or failed builds.Dec 13 2022, 10:42 AM
This revision was automatically updated to reflect the committed changes.