General

Profile

_dim (Dimitry Andric)

  • Login: _dim
  • Email: dimitry@andric.com
  • Registered on: 12/07/2016
  • Last sign in: 05/16/2021

Issues

open closed Total
Assigned issues 0 0 0
Reported issues 0 2 2

Activity

11/24/2021

08:49 AM Ruby Revision 06aafeb4 (git): Fix clang -Wcompound-token-split-by-macro warning in ruby.h
Building certain ruby gem native extensions (such as thrift), with clang
12.0.0 or later fails, because they have -Werror in their CFLAGS,
resulting in complaints about the expansion of the `rb_intern()` macro:
```
current directory: /w...
_dim (Dimitry Andric)

05/16/2021

10:07 AM Ruby Bug #17865 (Closed): clang 12 -Wcompound-token-split-by-macro warning in ruby.h
As reported in [pull request \#4504](https://github.com/ruby/ruby/pull/4504) (originally via [FreeBSD PR 255910](https://bugs.freebsd.org/255910)), certain ruby gem native extensions (such as thrift, see [here](http://package22.nyi.freeb... _dim (Dimitry Andric)

12/08/2016

09:52 AM Ruby Bug #13014: Adding clang volatile fixes from FreeBSD and NetBSD
Can we please backport these fixes to 2.1, 2.2 and 2.3? I am willing to do the work for supplying the patches, as we already have them in the FreeBSD ports collection.
Does this require a new issue, or can we re-use this one?
_dim (Dimitry Andric)

12/07/2016

10:03 PM Ruby Bug #13014: Adding clang volatile fixes from FreeBSD and NetBSD
Ok, I also tested this on x86_64-freebsd12, and there it still got a segfault in `cont_capture()`. This was caused by the `stat` parameter being a pointer to a volatile int, while it should have been a volatile pointer to an int.
Here...
_dim (Dimitry Andric)
08:52 PM Ruby Bug #13014 (Closed): Adding clang volatile fixes from FreeBSD and NetBSD
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... _dim (Dimitry Andric)
08:54 PM Ruby Bug #12893: trunk fails on linux with clang on test with callcc
Shugo Maeda wrote:
> Yui NARUSE wrote:
> ...
Did you add it as `volatile rb_context_t *cont`, or as `rb_context_t *volatile cont`? Only the latter is correct.
Please refer to #13014 for the full patch set, including fix for `cont_n...
_dim (Dimitry Andric)

Also available in: Atom