aaron.ballman added inline comments.

Comment at: lib/Driver/ToolChains/Gnu.h:253
+    void AddDefaultGCCPrefixes(const llvm::Triple &TargetTriple,
+                               SmallVectorImpl<std::string> &Prefixes,
fedor.sergeev wrote:
> aaron.ballman wrote:
> > Might as well drop the `llvm::` since the namespace isn't used for 
> > `SmallVectorImpl`.
> suggested change caused a build failure since Triple resolves into 
> Toolchain::Triple data member, not to llvm::Triple.
That's really good to know, thanks (and sorry for the churn)!


cfe-commits mailing list

Reply via email to