https://github.com/HighCommander4 closed
https://github.com/llvm/llvm-project/pull/69283
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/HighCommander4 approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/69283
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -203,6 +214,7 @@ template <> struct DenseMapInfo {
Val.Driver,
Val.StandardIncludes,
Val.StandardCXXIncludes,
+Val.Stdlib,
cjc25 wrote:
Done, and resorted these to match the order in `DriverArgs` definition so it'll
be
https://github.com/cjc25 updated https://github.com/llvm/llvm-project/pull/69283
>From f9986bfc0a1202ade8e641e7a1b1e73576070a73 Mon Sep 17 00:00:00 2001
From: Chris Carlon
Date: Sat, 14 Oct 2023 15:17:44 -0400
Subject: [PATCH 1/2] `clangd`: support `-stdlib=` flags from
@@ -203,6 +214,7 @@ template <> struct DenseMapInfo {
Val.Driver,
Val.StandardIncludes,
Val.StandardCXXIncludes,
+Val.Stdlib,
HighCommander4 wrote:
nit: it looks like the previous patch which introduced `Target` forgot to add
llvmbot wrote:
@llvm/pr-subscribers-clangd
Author: Chris Carlon (cjc25)
Changes
The `--stdlib` flag can affect the system headers used by `clang` during
compilation. By default, `clang` will use the platform-installed C++ standard
headers, but with `--stdlib=libc++`, `clang` can use
https://github.com/cjc25 created https://github.com/llvm/llvm-project/pull/69283
The `--stdlib` flag can affect the system headers used by `clang` during
compilation. By default, `clang` will use the platform-installed C++ standard
headers, but with `--stdlib=libc++`, `clang` can use headers