Sorry, I'm not sure I follow - what I meant is the patch needs (or should have, unless there's a reason this isn't testable, etc) a test case in Clang's regression suite. Probably in clang/test/CodeGen/debug-info-*
(you could try removing other targets from that conditional and see which tests fail - that should piont you to where to add tests for this platform) On Wed, Oct 15, 2014 at 4:44 AM, Alexander Eremin < [email protected]> wrote: > Hello David, > this is probably same issue as here: > https://gcc.gnu.org/ml/gcc-patches/2012-05/msg00445.html > > Without patch: > > # clang++ -v -o /tmp/test /tmp/test.cpp > clang version 3.5.0 (tags/RELEASE_350/final) > Target: i386-pc-solaris2.11 > Thread model: posix > "/usr/local/bin/clang" -cc1 -triple i386-pc-solaris2.11 -emit-obj > -mrelax-all -disable-free -disable-llvm-verifier -main-file-name test.cpp > -mrelocation-model static -mdisable-fp-elim -fmath-errno -masm-verbose > -mconstructor-aliases -target-cpu pentium4 -target-linker-version -v > -dwarf-column-info -resource-dir /usr/local/bin/../lib/clang/3.5.0 > -fdeprecated-macro -fdebug-compilation-dir > /tmp/build_root/developer_clang-3.5_pkg/usr -ferror-limit 19 > -fmessage-length 168 -mstackrealign -fobjc-runtime=gcc -fcxx-exceptions > -fexceptions -fdiagnostics-show-option -fcolor-diagnostics -o > /var/tmp/test-616bb1.o -x c++ /tmp/test.cpp > clang -cc1 version 3.5.0 based upon LLVM 3.5.0 default target > i386-pc-solaris2.11 > ignoring nonexistent directory "/opt/gcc4/include/c++/4.2.4" > ignoring nonexistent directory > "/opt/gcc4/include/c++/4.2.4/i386-pc-solaris2.11/" > ignoring nonexistent directory "/opt/gcc4/include/c++/4.2.4/backward" > #include "..." search starts here: > #include <...> search starts here: > /opt/gcc-4.8.1/include/c++/4.8.1 > /opt/gcc-4.8.1/include/c++/4.8.1//i386-pc-solaris2.11 > /opt/gcc-4.8.1/include/c++/4.8.1//backward > /usr/local/include > /usr/local/bin/../lib/clang/3.5.0/include > /opt/gcc-4.8.1/include (framework directory) > /usr/include > End of search list. > "/usr/bin/ld" -C -e _start -Bdynamic --dynamic-linker /lib/ld.so.1 -o > /tmp/test /usr/lib/crt1.o /usr/lib/crti.o /usr/lib/values-Xa.o > /opt/gcc-4.8.1/lib/gcc/i386-pc-solaris2.11/4.8.1/crtbegin.o -L/lib > -L/usr/lib -L/opt/gcc-4.8.1/lib/gcc/i386-pc-solaris2.11/4.8.1 > -L/usr/local/bin/../lib -L/usr/lib /var/tmp/test-616bb1.o -lstdc++ -lgcc_s > -lgcc -lc -lm /opt/gcc-4.8.1/lib/gcc/i386-pc-solaris2.11/4.8.1/crtend.o > /usr/lib/crtn.o > ld: fatal: unwind table: file /var/tmp/test-616bb1.o: section .eh_frame: > bad cie version 0: offset 0xfffffd7fffdff8a8 > > clang: error: linker command failed with exit code 1 (use -v to see > invocation) > > With patch: > > # clang++ -o /tmp/test /tmp/test.cpp > # > # /tmp/test > Hello World! > # > > > Alex > > 2014-10-15 0:37 GMT+04:00 David Blaikie <[email protected]>: > >> Test coverage? >> >> On Sat, Oct 11, 2014 at 1:53 PM, Alexander Eremin < >> [email protected]> wrote: >> >>> Hello >>> provided patch fixes issue with illumos/Solaris linker: Solaris has >>> default dwarf version 2 and linker now fails with error: unwind table: >>> section ._eh_frame: bad cie version 0. >>> >>> Cheers, >>> Alex >>> >>> _______________________________________________ >>> cfe-commits mailing list >>> [email protected] >>> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits >>> >>> >> >
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
