Hi again,
It seems like the BLAS standard allows negative stride, but judging from
all the implicit conversions happening, I don't think that it is well
supported in ViennaCL. In order to avoid loss of bits, it seems like my
cblas.h header uses signed int for both the strides and the offset/size.
Perhaps we should do the same thing?
2014-07-02 18:54 GMT+02:00 Philippe Tillet <phil.til...@gmail.com>:
> Hello,
>
> While fixing some warnings I have noticed an inconsistency in the type of
> strides.
>
> matrices/vectors use difference_type
> traits::stride use size_type
>
> This is a serious warnings because it leaves an ambiguity on negative
> strides.
> I think that negative strides should be allowed, as it allows to simply
> "reverse" a vector: (by using range(N, N), stride = -1). But I'm not sure
> our assertions (or the rest of the core) would actually properly support it.
>
> I think we should use difference type, anyway, since strides are
> conceptually differences.
>
> Philippe
>
------------------------------------------------------------------------------
Open source business process management suite built on Java and Eclipse
Turn processes into business applications with Bonita BPM Community Edition
Quickly connect people, data, and systems into organized workflows
Winner of BOSSIE, CODIE, OW2 and Gartner awards
http://p.sf.net/sfu/Bonitasoft
_______________________________________________
ViennaCL-devel mailing list
ViennaCL-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/viennacl-devel
