SemaAccess bug (protected members of derived)

[Alex L via cfe-commits]

Hi,

Can you please add a suitable test-case? Clang's code-completion tests are
normally in test/CodeCompletion or test/Index.

Cheers,
Alex

On 21 March 2017 at 12:05, Dixon Ryan (ETAS/ERS-PD2) via cfe-commits <
cfe-commits at lists.llvm.org> wrote:

> There are a number of users of the clang static analyser back-end for
> intelligent code completion. Irony-Mode for emacs, for example. For a while
> people have been reporting an issue with not getting completions for
> protected members of parent classes and I believe this patch solves the
> bug: simply that the arguments to IsDerivedFromInclusive were the wrong way
> around.
>
>
>
> URL: http://llvm.org/svn/llvm-project/cfe/trunk/lib/Sema
>
> Relative URL: ^/cfe/trunk/lib/Sema
>
>
>
> Index: SemaAccess.cpp
>
> ===================================================================
>
> --- SemaAccess.cpp      (revision 297956)
>
> +++ SemaAccess.cpp   (working copy)
>
> @@ -823,7 +823,7 @@
>
>          continue;
>
>        }
>
> -      switch (IsDerivedFromInclusive(InstanceContext, ECRecord)) {
>
> +      switch (IsDerivedFromInclusive(ECRecord, InstanceContext)) {
>
>        case AR_accessible: return AR_accessible;
>
>        case AR_inaccessible: continue;
>
>        case AR_dependent: OnFailure = AR_dependent; continue;
>
>
>
> Thank you.
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20170321/f9dec497/attachment.html>