[PATCH] D46052: GNUstep Objective-C ABI version 2

[David Chisnall via Phabricator via cfe-commits]

theraven added inline comments.


================
Comment at: lib/CodeGen/CGObjCGNU.cpp:512
   /// used to return an untyped selector (with the types field set to NULL).
-  llvm::Value *GetSelector(CodeGenFunction &CGF, Selector Sel,
+  virtual llvm::Value *GetSelector(CodeGenFunction &CGF, Selector Sel,
                            const std::string &TypeEncoding);
----------------
xbolva00 wrote:
> theraven wrote:
> > xbolva00 wrote:
> > > This causes 
> > > CGObjCGNU.cpp:589:16: warning: ‘virtual llvm::Value* {anonymous}::CGObjCGNU::GetSelector(clang::CodeGen::CodeGenFunction&, const clang::ObjCMethodDecl*)’ was hidden [-Woverloaded-virtual]
> > >    llvm::Value *GetSelector(CodeGenFunction &CGF,
> > > 
> > I can't reproduce this, and I'm not sure what the issue is.  The two lines have different overloads, so one shouldn't be hiding the other.
> It is a warning from GCC 7.2
Sounds like it's a spurious one.  Any idea how to silence it?


Repository:
  rC Clang

https://reviews.llvm.org/D46052