ldionne added a comment.

In D45639#2706589 <https://reviews.llvm.org/D45639#2706589>, @dexonsmith wrote:

> I'm not sure I'm totally following, but just want to double check that the 
> tests won't somehow use the libc++ from the SDK instead of ToT? I guess the 
> test uses `-nostdinc++` or something?
>
> Assuming that's doing what we want... I also wanted to highlight this adds a 
> dependency on bot's host clang working with the ToT libc++ headers. That's 
> probably fine -- I assume we keep the bot's Xcode relatively up-to-date -- 
> but I know @ldionne at some point was trying to avoid requiring new libc++ to 
> work with older clangs.

LLDB should either use the libc++ headers from the SDK *and* the dylib from the 
SDK, or build libc++ from scratch and test against both the trunk headers and 
the trunk dylib built for their target, not for the host as they seem to do 
right now. I'm following offline with @JDevlieghere to try and disentangle that.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D45639

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

Reply via email to