Skip to content

Commit 84403eb

Browse files
committed
Remove sjlj stuff from rust_upcall and don't pass -Werror to libuv.
1 parent a2c111a commit 84403eb

File tree

4 files changed

+12
-18
lines changed

4 files changed

+12
-18
lines changed

mk/platform.mk

+5-5
Original file line numberDiff line numberDiff line change
@@ -231,7 +231,7 @@ CFG_LIB_NAME_x86_64-apple-darwin=lib$(1).dylib
231231
CFG_STATIC_LIB_NAME_x86_64-apple-darwin=lib$(1).a
232232
CFG_LIB_GLOB_x86_64-apple-darwin=lib$(1)-*.dylib
233233
CFG_LIB_DSYM_GLOB_x86_64-apple-darwin=lib$(1)-*.dylib.dSYM
234-
CFG_GCCISH_CFLAGS_x86_64-apple-darwin := -Wall -g -fPIC -m64 -arch x86_64
234+
CFG_GCCISH_CFLAGS_x86_64-apple-darwin := -Wall -Werror -g -fPIC -m64 -arch x86_64
235235
CFG_GCCISH_CXXFLAGS_x86_64-apple-darwin := -fno-rtti
236236
CFG_GCCISH_LINK_FLAGS_x86_64-apple-darwin := -dynamiclib -pthread -framework CoreServices -m64
237237
CFG_GCCISH_DEF_FLAG_x86_64-apple-darwin := -Wl,-exported_symbols_list,
@@ -257,7 +257,7 @@ CFG_LIB_NAME_i686-apple-darwin=lib$(1).dylib
257257
CFG_STATIC_LIB_NAME_i686-apple-darwin=lib$(1).a
258258
CFG_LIB_GLOB_i686-apple-darwin=lib$(1)-*.dylib
259259
CFG_LIB_DSYM_GLOB_i686-apple-darwin=lib$(1)-*.dylib.dSYM
260-
CFG_GCCISH_CFLAGS_i686-apple-darwin := -Wall -g -fPIC -m32 -arch i386
260+
CFG_GCCISH_CFLAGS_i686-apple-darwin := -Wall -Werror -g -fPIC -m32 -arch i386
261261
CFG_GCCISH_CXXFLAGS_i686-apple-darwin := -fno-rtti
262262
CFG_GCCISH_LINK_FLAGS_i686-apple-darwin := -dynamiclib -pthread -framework CoreServices -m32
263263
CFG_GCCISH_DEF_FLAG_i686-apple-darwin := -Wl,-exported_symbols_list,
@@ -396,7 +396,7 @@ CFG_LIB_NAME_i686-pc-mingw32=$(1).dll
396396
CFG_STATIC_LIB_NAME_i686-pc-mingw32=$(1).lib
397397
CFG_LIB_GLOB_i686-pc-mingw32=$(1)-*.dll
398398
CFG_LIB_DSYM_GLOB_i686-pc-mingw32=$(1)-*.dylib.dSYM
399-
CFG_GCCISH_CFLAGS_i686-pc-mingw32 := -Wall -g -m32 -march=i686 -D_WIN32_WINNT=0x0600 -I$(CFG_SRC_DIR)src/etc/mingw-fix-include
399+
CFG_GCCISH_CFLAGS_i686-pc-mingw32 := -Wall -Werror -g -m32 -march=i686 -D_WIN32_WINNT=0x0600 -I$(CFG_SRC_DIR)src/etc/mingw-fix-include
400400
CFG_GCCISH_CXXFLAGS_i686-pc-mingw32 := -fno-rtti
401401
CFG_GCCISH_LINK_FLAGS_i686-pc-mingw32 := -shared -fPIC -g -m32
402402
CFG_GCCISH_DEF_FLAG_i686-pc-mingw32 :=
@@ -452,7 +452,7 @@ CFG_LIB_NAME_i686-w64-mingw32=$(1).dll
452452
CFG_STATIC_LIB_NAME_i686-w64-mingw32=$(1).lib
453453
CFG_LIB_GLOB_i686-w64-mingw32=$(1)-*.dll
454454
CFG_LIB_DSYM_GLOB_i686-w64-mingw32=$(1)-*.dylib.dSYM
455-
CFG_GCCISH_CFLAGS_i686-w64-mingw32 := -Wall -g -m32 -D_WIN32_WINNT=0x0600
455+
CFG_GCCISH_CFLAGS_i686-w64-mingw32 := -Wall -Werror -g -m32 -D_WIN32_WINNT=0x0600
456456
CFG_GCCISH_CXXFLAGS_i686-w64-mingw32 := -fno-rtti
457457
CFG_GCCISH_LINK_FLAGS_i686-w64-mingw32 := -shared -g -m32
458458
CFG_GCCISH_DEF_FLAG_i686-w64-mingw32 :=
@@ -480,7 +480,7 @@ CFG_LIB_NAME_x86_64-w64-mingw32=$(1).dll
480480
CFG_STATIC_LIB_NAME_x86_64-w64-mingw32=$(1).lib
481481
CFG_LIB_GLOB_x86_64-w64-mingw32=$(1)-*.dll
482482
CFG_LIB_DSYM_GLOB_x86_64-w64-mingw32=$(1)-*.dylib.dSYM
483-
CFG_GCCISH_CFLAGS_x86_64-w64-mingw32 := -Wall -g -m64 -D_WIN32_WINNT=0x0600
483+
CFG_GCCISH_CFLAGS_x86_64-w64-mingw32 := -Wall -Werror -g -m64 -D_WIN32_WINNT=0x0600
484484
CFG_GCCISH_CXXFLAGS_x86_64-w64-mingw32 := -fno-rtti
485485
CFG_GCCISH_LINK_FLAGS_x86_64-w64-mingw32 := -shared -g -m64
486486
CFG_GCCISH_DEF_FLAG_x86_64-w64-mingw32 :=

mk/rt.mk

+7-2
Original file line numberDiff line numberDiff line change
@@ -192,6 +192,9 @@ LIBUV_LIB_$(1) := $$(RT_OUTPUT_DIR_$(1))/libuv/$$(LIBUV_NAME_$(1))
192192

193193
LIBUV_MAKEFILE_$(1) := $$(CFG_BUILD_DIR)$$(RT_OUTPUT_DIR_$(1))/libuv/Makefile
194194

195+
# libuv triggers a few warnings on some platforms
196+
LIBUV_CFLAGS_$(1) := $(subst -Werror,,$(CFG_GCCISH_CFLAGS_$(1)))
197+
195198
$$(LIBUV_MAKEFILE_$(1)): $$(LIBUV_DEPS)
196199
(cd $(S)src/libuv/ && \
197200
$$(CFG_PYTHON) ./gyp_uv.py -f make -Dtarget_arch=$$(LIBUV_ARCH_$(1)) \
@@ -205,14 +208,16 @@ $$(LIBUV_MAKEFILE_$(1)): $$(LIBUV_DEPS)
205208
ifdef CFG_WINDOWSY_$(1)
206209
$$(LIBUV_LIB_$(1)): $$(LIBUV_DEPS)
207210
$$(Q)$$(MAKE) -C $$(S)src/libuv -f Makefile.mingw \
208-
CC="$$(CC_$(1)) $$(CFG_GCCISH_CFLAGS_$(1)) $$(SNAP_DEFINES)" \
211+
LDFLAGS="$$(CFG_GCCISH_LINK_FLAGS_$(1))" \
212+
CC="$$(CC_$(1)) $$(LIBUV_CFLAGS_$(1)) $$(SNAP_DEFINES)" \
213+
CXX="$$(CXX_$(1))" \
209214
AR="$$(AR_$(1))" \
210215
V=$$(VERBOSE)
211216
$$(Q)cp $$(S)src/libuv/libuv.a $$@
212217
else
213218
$$(LIBUV_LIB_$(1)): $$(LIBUV_DEPS) $$(LIBUV_MAKEFILE_$(1))
214219
$$(Q)$$(MAKE) -C $$(@D) \
215-
CFLAGS="$$(CFG_GCCISH_CFLAGS_$(1)) $$(SNAP_DEFINES)" \
220+
CFLAGS="$$(LIBUV_CFLAGS_$(1)) $$(SNAP_DEFINES)" \
216221
LDFLAGS="$$(CFG_GCCISH_LINK_FLAGS_$(1))" \
217222
CC="$$(CC_$(1))" \
218223
CXX="$$(CXX_$(1))" \

src/rt/rust_upcall.c

-10
Original file line numberDiff line numberDiff line change
@@ -35,16 +35,6 @@ struct _Unwind_Exception;
3535
# endif
3636
#endif
3737

38-
#if __USING_SJLJ_EXCEPTIONS__
39-
void _Unwind_SjLj_Resume(struct _Unwind_Exception* exception_object);
40-
41-
void _Unwind_Resume(struct _Unwind_Exception* exception_object) {
42-
_Unwind_SjLj_Resume(exception_object);
43-
}
44-
#else
45-
extern void _Unnwind_Resume(struct _Unwind_Exception* exception_object);
46-
#endif
47-
4838
_Unwind_Reason_Code
4939
PERSONALITY_FUNC(int version,
5040
_Unwind_Action actions,

src/rt/rustrt.def.in

-1
Original file line numberDiff line numberDiff line change
@@ -49,4 +49,3 @@ rust_get_test_int
4949
rust_pthread_mutex_t_size
5050
rust_pthread_cond_t_size
5151
rust_crit_section_size
52-
_Unwind_Resume

0 commit comments

Comments
 (0)