From 47fca284227fa4b20e02b4ce5269b0330a7915f1 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Martin=20Storsj=C3=B6?= Date: Mon, 1 Oct 2018 14:44:32 +0300 Subject: [PATCH] Tweak the wording for the comment about exception issues with dwarf for i686 --- wrappers/clang-target-wrapper.sh | 13 ++++++++----- 1 file changed, 8 insertions(+), 5 deletions(-) diff --git a/wrappers/clang-target-wrapper.sh b/wrappers/clang-target-wrapper.sh index 3cce388..f28609f 100755 --- a/wrappers/clang-target-wrapper.sh +++ b/wrappers/clang-target-wrapper.sh @@ -11,11 +11,14 @@ ARCH=$(echo $TARGET | sed 's/-.*//') case $ARCH in i686) # Dwarf is the default for i686, but libunwind sometimes fails to - # to unwind correctly on i686. This issue might be related to cases - # of DW_CFA_GNU_args_size (which were adjusted in libunwind SVN r337312). - # The issue can be reproduced with test/exception-locale.cpp. - # The issue goes away if building libunwind/libcxxabi/libcxx and the - # test example with -mstack-alignment=16 -mstackrealign. + # to unwind correctly on i686. The issue can be reproduced with + # test/exception-locale.cpp. The issue might be related to + # DW_CFA_GNU_args_size, since it goes away if building + # libunwind/libcxxabi/libcxx and the test example with + # -mstack-alignment=16 -mstackrealign. (libunwind SVN r337312 fixed + # some handling relating to this dwarf opcode, which made + # test/hello-exception.cpp work properly, but apparently there are + # still issues with it). ARCH_FLAGS=-fsjlj-exceptions ;; x86_64)