Ericsson / clang

Cross Translation Unit analysis capability for Clang Static Analyzer. (Fork of official clang at http://llvm.org/git/clang)
http://clang.llvm.org/
Other
15 stars 10 forks source link

[ASTImporter] Corrected import of NestedNameSpecifierLoc TypeSpec case. #553

Closed balazske closed 5 years ago

balazske commented 5 years ago

This correction fixes an assertion:

Pos.isValid() && Pos.asLocation().isValid() && "PathDiagnosticSpotPiece's must have a valid location."' failed.
balazske commented 5 years ago

The unit tests can be extended with a check that tests for invalid source location after every Import (but not every occurrence is problem)?

balazske commented 5 years ago

Phabricator review: https://reviews.llvm.org/D55358