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] Check visibility of EnumDecl. #632

Closed balazske closed 5 years ago

balazske commented 5 years ago

Similar enum decls in different anonymous namespace were not handled correctly. Added one unit test (maybe add for non-namespace vs namespace case too?), this discovered that the brace-range was not imported (SourceLocation check failure).

balazske commented 5 years ago

Updated with new diff and new commit message (same code as before).

balazske commented 5 years ago

The check was cancelled (not failed) because it was the same code as already tested.

martong commented 5 years ago

The check was cancelled (not failed) because it was the same code as already tested.

Okay, I thought there was an error.

balazske commented 5 years ago

634 is the next, for "enum classes".

balazske commented 5 years ago

upstream: https://reviews.llvm.org/D62484