From: usa@... Date: 2016-12-08T10:11:41+00:00 Subject: [ruby-core:78545] [Ruby trunk Bug#13014] Adding clang volatile fixes from FreeBSD and NetBSD Issue #13014 has been updated by Usaku NAKAMURA. Backport changed from 2.1: UNKNOWN, 2.2: UNKNOWN, 2.3: UNKNOWN to 2.1: WONTFIX, 2.2: REQUIRED, 2.3: REQUIRED Dimitry Andric wrote: > Can we please backport these fixes to 2.1, 2.2 and 2.3? 2.1 won't be fixed because it's in the phase of security maintenance. But 2.2 and 2.3 will be fixed. > Does this require a new issue, or can we re-use this one? You don't have to do anything now. We stable branch maintainers are tracking this issue. Thank you! ---------------------------------------- Bug #13014: Adding clang volatile fixes from FreeBSD and NetBSD https://bugs.ruby-lang.org/issues/13014#change-61930 * Author: Dimitry Andric * Status: Closed * Priority: Normal * Assignee: * ruby -v: ruby 2.4.0dev (2016-12-08 trunk 57019) [i386-freebsd12.0] * Backport: 2.1: WONTFIX, 2.2: REQUIRED, 2.3: REQUIRED ---------------------------------------- Referring to bug #12893, where segfaults were observed when compiling with recent versions of clang. In particular in cont.c, about which Shugo Maeda noted that he could reproduce this on FreeBSD. These segfaults are most likely caused by missing volatile specifiers on certain variable declarations, in combination with the use of setjmp(). In https://bugs.freebsd.org/206111 (for ruby 2.3), we have applied a number of fixes for this, taken mostly from NetBSD's pkgsrc. These add a few volatile specifiers in critical places. I have also added one additional volatile specifier in cont_capture() in cont.c, and removed the special optnone handling for clang 3.8.0. For me, on FreeBSD 12 with clang 3.9.1, and optimizing at -O3 for -march=haswell, this successfully completes "gmake test-all": Finished tests in 461.890990s, 36.8312 tests/s, 4849.7937 assertions/s. 17012 tests, 2240076 assertions, 0 failures, 0 errors, 40 skips ---Files-------------------------------- ruby-trunk-add-setjmp-volatile-1.diff (4.61 KB) ruby-trunk-add-setjmp-volatile-2.diff (5 KB) -- https://bugs.ruby-lang.org/ Unsubscribe: