* SCM_STATIC_ALIGNED_ARRAY [not found] <E1NjgMt-0000xD-7j@vcs-noshell.in.savannah.gnu.org> @ 2010-03-01 16:33 ` Ludovic Courtès 2010-07-17 10:23 ` SCM_STATIC_ALIGNED_ARRAY Andy Wingo 0 siblings, 1 reply; 3+ messages in thread From: Ludovic Courtès @ 2010-03-01 16:33 UTC (permalink / raw) To: Andy Wingo; +Cc: guile-devel Hello Andy! "Andy Wingo" <wingo@pobox.com> writes: > commit cee1d22c3c10b1892c82a5758ef69cd6fc9aba31 > Author: Andy Wingo <wingo@pobox.com> > Date: Mon Feb 22 23:00:19 2010 +0100 > > actually capture partial continuations [...] > +#ifdef SCM_ALIGNED > +#define SCM_DECLARE_STATIC_ALIGNED_ARRAY(type, sym)\ > +static const type sym[] > +#define SCM_STATIC_ALIGNED_ARRAY(alignment, type, sym)\ > +static SCM_ALIGNED (alignment) const type sym[] > +#else > +#define SCM_DECLARE_STATIC_ALIGNED_ARRAY(type, sym)\ > +static type *sym > +#define SCM_STATIC_ALIGNED_ARRAY(alignment, type, sym) \ > +SCM_SNARF_INIT(sym = scm_malloc (sizeof(sym##__unaligned) + alignment - 1); \ ‘scm_gc_malloc_pointerless ()’ could be used here. It always returns 8-byte aligned areas [0], which should allow some of the alignment twiddling to be removed. What do you think? Also, the indentation, spacing, etc., are not GCS-conforming. Thanks, Ludo’. [0] http://www.hpl.hp.com/hosted/linux/mail-archives/gc/2006-May/001280.html ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: SCM_STATIC_ALIGNED_ARRAY 2010-03-01 16:33 ` SCM_STATIC_ALIGNED_ARRAY Ludovic Courtès @ 2010-07-17 10:23 ` Andy Wingo 2010-07-17 12:54 ` SCM_STATIC_ALIGNED_ARRAY Ludovic Courtès 0 siblings, 1 reply; 3+ messages in thread From: Andy Wingo @ 2010-07-17 10:23 UTC (permalink / raw) To: Ludovic Courtès; +Cc: guile-devel Heya, On Mon 01 Mar 2010 17:33, ludo@gnu.org (Ludovic Courtès) writes: > "Andy Wingo" <wingo@pobox.com> writes: > >> commit cee1d22c3c10b1892c82a5758ef69cd6fc9aba31 >> Author: Andy Wingo <wingo@pobox.com> >> Date: Mon Feb 22 23:00:19 2010 +0100 >> >> actually capture partial continuations > > [...] > >> +#ifdef SCM_ALIGNED >> +#define SCM_DECLARE_STATIC_ALIGNED_ARRAY(type, sym)\ >> +static const type sym[] >> +#define SCM_STATIC_ALIGNED_ARRAY(alignment, type, sym)\ >> +static SCM_ALIGNED (alignment) const type sym[] >> +#else >> +#define SCM_DECLARE_STATIC_ALIGNED_ARRAY(type, sym)\ >> +static type *sym >> +#define SCM_STATIC_ALIGNED_ARRAY(alignment, type, sym) \ >> +SCM_SNARF_INIT(sym = scm_malloc (sizeof(sym##__unaligned) + alignment - 1); \ > > ‘scm_gc_malloc_pointerless ()’ could be used here. It always returns > 8-byte aligned areas [0], which should allow some of the alignment > twiddling to be removed. What do you think? Done > Also, the indentation, spacing, etc., are not GCS-conforming. I tweaked it a little, but my instincts here are bad. If you still care, please commit a proper fix :) Andy -- http://wingolog.org/ ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: SCM_STATIC_ALIGNED_ARRAY 2010-07-17 10:23 ` SCM_STATIC_ALIGNED_ARRAY Andy Wingo @ 2010-07-17 12:54 ` Ludovic Courtès 0 siblings, 0 replies; 3+ messages in thread From: Ludovic Courtès @ 2010-07-17 12:54 UTC (permalink / raw) To: Andy Wingo; +Cc: guile-devel Hi, Andy Wingo <wingo@pobox.com> writes: > On Mon 01 Mar 2010 17:33, ludo@gnu.org (Ludovic Courtès) writes: [...] >> Also, the indentation, spacing, etc., are not GCS-conforming. > > I tweaked it a little, but my instincts here are bad. If you still care, > please commit a proper fix :) I’d prefer if we all cared. If in doubt, use ‘indent’. Ludo’. ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2010-07-17 12:54 UTC | newest] Thread overview: 3+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- [not found] <E1NjgMt-0000xD-7j@vcs-noshell.in.savannah.gnu.org> 2010-03-01 16:33 ` SCM_STATIC_ALIGNED_ARRAY Ludovic Courtès 2010-07-17 10:23 ` SCM_STATIC_ALIGNED_ARRAY Andy Wingo 2010-07-17 12:54 ` SCM_STATIC_ALIGNED_ARRAY Ludovic Courtès
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).