[PATCH] D81622: [Clang] Search computed sysroot for libc++ header paths

Stephen Hines via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jun 10 21:02:14 PDT 2020


srhines added a comment.

Thanks, Ryan, for diagnosing and addressing this bug.



================
Comment at: clang/lib/Driver/ToolChains/Linux.h:45
                         llvm::opt::ArgStringList &CmdArgs) const override;
-  virtual std::string computeSysRoot() const;
+  virtual std::string computeSysRoot() const override;
 
----------------
Can drop `virtual` here.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D81622/new/

https://reviews.llvm.org/D81622





More information about the cfe-commits mailing list