Skip to content

Commit

Permalink
druntime: Fix up cpp_demangle subtest of exceptions integration t…
Browse files Browse the repository at this point in the history
…ests (#14894)

* Don't run the test in release mode on Linux; just like for the other
  Posix targets.
* Include `-L--export-dynamic` as required dflag (added implicitly to cc
  linker cmdline by DMD, but not by LDC), which is required for symbol
  resolution in druntime exception backtraces.
  • Loading branch information
kinke committed Feb 19, 2023
1 parent 680cf33 commit a9eedfe
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions druntime/test/exceptions/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ ifeq ($(OS)-$(BUILD),linux-debug)
LINE_TRACE_DFLAGS:=-L--export-dynamic
endif
ifeq ($(OS),linux)
TESTS+=rt_trap_exceptions_drt_gdb cpp_demangle
TESTS+=rt_trap_exceptions_drt_gdb
endif
ifeq ($(OS)-$(BUILD),freebsd-debug)
TESTS+=line_trace line_trace_21656 long_backtrace_trunc cpp_demangle
Expand Down Expand Up @@ -129,7 +129,7 @@ $(ROOT)/long_backtrace_trunc: DFLAGS+=$(LINE_TRACE_DFLAGS)
$(ROOT)/rt_trap_exceptions: DFLAGS+=$(LINE_TRACE_DFLAGS)
$(ROOT)/rt_trap_exceptions_drt: DFLAGS+=-g
$(ROOT)/refcounted: DFLAGS+=-dip1008
$(ROOT)/cpp_demangle: DFLAGS+=-L-lstdc++
$(ROOT)/cpp_demangle: DFLAGS+=-L-lstdc++ $(LINE_TRACE_DFLAGS)

$(ROOT)/%: $(SRC)/%.d $(DMD) $(DRUNTIME)
$(QUIET)$(DMD) $(DFLAGS) -of$@ $<
Expand Down

0 comments on commit a9eedfe

Please sign in to comment.