[PATCH] D159383: [Headers] Remove musl-related comment about NULL

Derek Schuff via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Sep 5 14:08:37 PDT 2023


dschuff added subscribers: sbc100, sunfish.
dschuff added a comment.

The 2 common WebAssembly toolchain variants (Emscripten and wasi-sdk) use libcs that are derived from musl (a subset along with additions specific to their environments); they share the system-include configuration in `WebAssembly::AddClangSystemIncludeArgs()` in `clang/lib/Driver/Toolchains/WebAssembly.cpp`, which adds the ResourceDir before the Sysroot (and that I believe results in this include order). So we should maybe consider fixing that (something @aheejin and I should discuss with @sunfish and @sbc100 and the rest of the Emscripten and/or wasi folks).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D159383



More information about the cfe-commits mailing list