SemaAccess bug (protected members of derived)

Dixon Ryan (ETAS/ERS-PD2) via cfe-commits cfe-commits at lists.llvm.org
Tue Mar 21 05:05:07 PDT 2017


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.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20170321/208a59ba/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: SemaAccessBug.diff
Type: application/octet-stream
Size: 502 bytes
Desc: SemaAccessBug.diff
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20170321/208a59ba/attachment.obj>


More information about the cfe-commits mailing list