[vsipl++] [patch] Distributed transpose

Stefan Seefeld stefan at codesourcery.com
Thu Jun 5 19:06:13 UTC 2008


Jules Bergmann wrote:
> This patch fixes handling of distributed transposes of the form:
> 
>     A = B.transpose();
> 
> Ok to apply?


Jules,

the patch looks good.

There were a couple of changes of the form:

> -/* Copyright (c) 2005 by CodeSourcery, LLC.  All rights reserved. */
> +/* Copyright (c) 2005, 2008 by CodeSourcery, LLC.  All rights reserved. */
>  

where I'd like to suggest we remove the obsolete 'LLC.', while applying 
other changes. (I don't think this is worth a full sweep over the sources !)

Also, this patch reminds me of two issues, which I hope we will find the 
time to address over the coming months:

* We need to address the many header interdependencies, in particular 
between the serial and the parallel set. There is some circularity that 
is very hard to break out of.

* There are sets of templates that form a self-contained block of 
functionality that I think is important to document, both, from a low 
level as well as middel-level view. I'm in particular thinking of the 
Combine_return_type logic (which I have run across multiple times this 
week already)


Thanks,
		Stefan

-- 
Stefan Seefeld
CodeSourcery
stefan at codesourcery.com
(650) 331-3385 x718



More information about the vsipl++ mailing list