[Lldb-commits] [lldb] [lldb][darwin] force BuiltinHeadersInSystemModules to be always false (PR #144913)

2025-06-26 Thread Ian Anderson via lldb-commits
ian-twilightcoder wrote: It's not _just_ `BuiltinHeadersInSystemModules` either. There are lots of things that the driver toolchain does that you shouldn't be skipping. https://github.com/llvm/llvm-project/pull/144913 ___ lldb-commits mailing list lld

[Lldb-commits] [lldb] [lldb][ClangExpressionParser] Set BuiltinHeadersInSystemModules depending on SDK version (PR #102309)

2024-08-07 Thread Ian Anderson via lldb-commits
@@ -259,6 +259,27 @@ bool XcodeSDK::SupportsSwift() const { } } +bool XcodeSDK::SDKSupportsBuiltinModules(const llvm::Triple &target_triple, ian-twilightcoder wrote: But you didn't pick up the environment check https://github.com/llvm/llvm-project/pull/102

[Lldb-commits] [lldb] [lldb][ClangExpressionParser] Set BuiltinHeadersInSystemModules depending on SDK version (PR #102309)

2024-08-07 Thread Ian Anderson via lldb-commits
@@ -259,6 +259,27 @@ bool XcodeSDK::SupportsSwift() const { } } +bool XcodeSDK::SDKSupportsBuiltinModules(const llvm::Triple &target_triple, ian-twilightcoder wrote: This function got changed in https://github.com/llvm/llvm-project/pull/102239 https://gith

[Lldb-commits] [lldb] [lldb][ClangExpressionParser] Set BuiltinHeadersInSystemModules depending on SDK version (PR #102309)

2024-08-07 Thread Ian Anderson via lldb-commits
ian-twilightcoder wrote: This really isn't the right approach. `BuiltinHeadersInSystemModules` isn't the only important default that the driver sets up. Can we change the invocation setup to use clang::createInvocation or clang::CompilerInvocation::CreateFromArgs or clang::BuildCompilation? Or