aaron.ballman added a comment.

I applied your patch locally and got a few compile errors with MSVC 2015:

'bool 
llvm::opt::ArgList::hasArg(llvm::opt::OptSpecifier,llvm::opt::OptSpecifier,llvm::opt::OptSpecifier)
 const': cannot convert argument 3 from 'bool' to 'llvm::opt::OptSpecifier'    
E:\llvm\llvm\tools\clang\lib\Driver\Tools.cpp   3187    
'bool 
llvm::opt::ArgList::hasArg(llvm::opt::OptSpecifier,llvm::opt::OptSpecifier,llvm::opt::OptSpecifier)
 const': cannot convert argument 3 from 'bool' to 'llvm::opt::OptSpecifier'    
E:\llvm\llvm\tools\clang\lib\Driver\Tools.cpp   4608

There is no hasArg() overload that accepts a bool argument; perhaps that's a 
feature of one of the dependencies this patch depends on?


http://reviews.llvm.org/D20352



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to