[PATCH] D65699: [Driver] Prioritize SYSROOT/usr/include over RESOURCE_DIR/include on linux-musl

Khem Raj via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Sep 9 07:28:01 PDT 2019


raj.khem added a comment.

In D65699#1662915 <https://reviews.llvm.org/D65699#1662915>, @dalias wrote:

> That's a separate issue of clang having a slight types-ABI mismatch with some 32-bit archs whose original ABIs used `long` instead of `int` for `wchar_t`. The wrong header order makes it quickly apparent, but these are independent; wrong header order is still wrong and will break (other) things even without this type mismatch. Backport of the fix would be much appreciated.


@dalias I agree its an issue that comes to fore due to this change and was latent, this however now fails to build libedit, which is a prerequisite for building clang itself, so its kind of pesky problem now.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D65699





More information about the cfe-commits mailing list