This is an archive of the discontinued LLVM Phabricator instance.

[OpenMP] Ensure the DefaultMapperId has a location
ClosedPublic

Authored by jdoerfert on Apr 15 2021, 11:12 PM.

Details

Summary

A user reported an assertion (below) but without a reproducer. I failed to
create a test myself but from the assertion one can derive the problem.
I set the DefaultMapperId location now to make sure this doesn't cause
trouble.

clang-13: .../DeclTemplate.h:1940:
void clang::ClassTemplateSpecializationDecl::setPointOfInstantiation(clang::SourceLocation):
Assertion `Loc.isValid() && "point of instantiation must be valid!"' failed.

Diff Detail

Unit TestsFailed

Event Timeline

jdoerfert created this revision.Apr 15 2021, 11:12 PM
jdoerfert requested review of this revision.Apr 15 2021, 11:12 PM
Herald added a project: Restricted Project. · View Herald TranscriptApr 15 2021, 11:12 PM
JonChesterfield accepted this revision.May 3 2021, 8:20 AM

Not ideal to have unreproducible failures but the fix looks safe regardless

This revision is now accepted and ready to land.May 3 2021, 8:20 AM
This revision was landed with ongoing or failed builds.May 6 2021, 12:10 AM
This revision was automatically updated to reflect the committed changes.