Skip to content

Commit

Permalink
Fix predefined macros check after recent clang change
Browse files Browse the repository at this point in the history
This is an alternative to #75.
  • Loading branch information
sbc100 authored and sunfishcode committed Jul 16, 2019
1 parent a413650 commit 8df0d4c
Showing 1 changed file with 6 additions and 0 deletions.
6 changes: 6 additions & 0 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -467,9 +467,15 @@ finish: startup_files libc
# which we don't need to track here. For the __*_ATOMIC_*_LOCK_FREE
# macros, squash individual compiler names to attempt, toward keeping
# these files compiler-independent.
#
# We have to add `-isystem $(SYSROOT_INC)` because otherwise clang puts
# its builtin include path first, which produces compiler-specific
# output.
#
# TODO: Undefine __FLOAT128__ for now since it's not in clang 8.0.
# TODO: Filter out __FLT16_* for now, as not all versions of clang have these.
"$(WASM_CC)" $(WASM_CFLAGS) "$(SYSROOT_SHARE)/include-all.c" \
-isystem $(SYSROOT_INC) \
-E -dM -Wno-\#warnings \
-D_ALL_SOURCE \
-U__llvm__ \
Expand Down

0 comments on commit 8df0d4c

Please # to comment.