search

xulunfan / thrust

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

consider removing device-to-device copy workaround for absence of nvcc #449

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
IIRC we decided to eliminate this WAR because it violates the one-definition 
rule and there's no way to ensure we get the "good" version when both are 
present.

[1] 
http://code.google.com/p/thrust/source/browse/thrust/system/cuda/detail/copy_dev
ice_to_device.inl#41

Original issue reported on code.google.com by wnbell on 24 Jan 2012 at 4:23

GoogleCodeExporter commented 8 years ago 
I don't think we should modify that code unless we get a bug report

Original comment by jaredhoberock on 24 Jan 2012 at 7:25

GoogleCodeExporter commented 8 years ago 
I'll take a look at this later.  I can't say what the right approach is, but 
I'm concerned about the silent (and hard to diagnose) performance hazard that 
this introduces.

Original comment by wnbell on 24 Jan 2012 at 8:03

GoogleCodeExporter commented 8 years ago 
Forwarded to https://github.com/thrust/thrust/issues/58

Original comment by jaredhoberock on 7 May 2012 at 9:01