rom3r4 / thrust

Automatically exported from code.google.com/p/thrust
Apache License 2.0
0 stars 0 forks source link

Thrust should compile warning-free #55

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Some algorithms, particularly stable_merge_sort, produce spurious warnings.

e.g. thrust\sorting\detail\device\cuda\stable_merge_sort.inl(577): Warning: 
Cannot tell what pointer points to, assuming global memory space

Original issue reported on code.google.com by jaredhoberock on 23 Nov 2009 at 10:28

GoogleCodeExporter commented 9 years ago

Original comment by jaredhoberock on 26 Apr 2010 at 9:43

GoogleCodeExporter commented 9 years ago
With VS 2010 you get the following warning if /doc option is set:

c:\cuda\include\driver_types.h(119): warning C4635: XML document comment 
applied to 'cudaErrorTextureFetchFailed': badly-formed XML: Whitespace is not 
allowed at this location.

If /clr option is set then Thrust fails to compile. "namespace generic" 
conflicts with the "generic" keyword used by C++/CLI. I suspect this is by 
design (PPL etc don't support /CLR either) but it isn't called out in the docs 
anywhere and is an obviousl thing to try it you want to glue .NET into CUDA via 
Thrust.

Original comment by ade.s.mi...@gmail.com on 3 Sep 2010 at 5:48

GoogleCodeExporter commented 9 years ago
.NET support has been added as Thrust issue 206.

Thrust compiles cleanly with -Wall on Linux now with a nightly nvcc 3.2.  Pure 
CUDA causes many warnings on windows so it's not worth trying to 
comprehensively fix this issue for 1.3.

The unit tester builds with -Wall by default (but not -Werror).  We should 
enable -Werror, but testing the deprecated functions defeats that.  Perhaps we 
should quarantine the deprecated testing to testing/deprecated which would not 
inherit -Werror.

Original comment by jaredhoberock on 4 Sep 2010 at 12:24

GoogleCodeExporter commented 9 years ago
[deleted comment]
GoogleCodeExporter commented 9 years ago
I'm getting the following warning over and over when compiling on a mac

/thrust/detail/device/cuda/detail/b40c/radixsort_scanscatter_kernel.h:198: 
warning: integer constant is too large for "unsigned long" type

Original comment by andrew.c...@gmail.com on 31 Dec 2010 at 12:17

GoogleCodeExporter commented 9 years ago
[deleted comment]
GoogleCodeExporter commented 9 years ago

Original comment by jaredhoberock on 4 Feb 2011 at 11:04

GoogleCodeExporter commented 9 years ago

Original comment by jaredhoberock on 31 Aug 2011 at 12:12

GoogleCodeExporter commented 9 years ago

Original comment by jaredhoberock on 4 Feb 2012 at 12:14

GoogleCodeExporter commented 9 years ago
Forwarded to github:

https://github.com/thrust/thrust/issues/31

Original comment by jaredhoberock on 7 May 2012 at 8:29