search

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

Friend class decl should not be visible in its context. #616

Closed balazske closed 5 years ago

balazske commented 5 years ago

The addDeclInternal makes the decl already visible in its semantic context. So we need to do this only if addDeclInternal was not called (but redundant call seems to work, if the lexical DC is different it is done anyway). And it should be enough to do this in the semantic context (that should be not a transparent context).

martong commented 5 years ago

The addDeclInternal makes the decl already visible in its semantic context. So we need to do this only if addDeclInternal was not called (but redundant call seems to work, if the lexical DC is different it is done anyway). And it should be enough to do this in the semantic context (that should be not a transparent context).

OK