search

mayingzhen / nvidia-texture-tools

Automatically exported from code.google.com/p/nvidia-texture-tools
Other
0 stars 0 forks source link

Release mode projects for visual studio 2008 #62

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
The Visual Studio 2008 projects are have mismatched optimizations in
release mode, none of which are optimal.

All projects, Release mode, Optimizations should be set to Full Optimization 

I did comprehensive tests in all custom and preset optimization modes to
discover this was optimal for NVTT.

Original issue reported on code.google.com by andi.ire...@gmail.com on 25 Aug 2008 at 5:11

GoogleCodeExporter commented 8 years ago 
Thanks for the heads up. The vc9 projects were generated automatically from the 
vc8
projects. The Optimization option is set to default in vc8 and vc9, but I guess 
the
default might have changed between versions.

I don't have access to visual studio 2008 right now. Would you mind attaching a 
patch?

Thanks!

Original comment by cast...@gmail.com on 25 Aug 2008 at 8:48

GoogleCodeExporter commented 8 years ago 
1) where do i submit patch files.
2) is tortoise SVN 1.5.1 .patch file acceptable

Original comment by andi.ire...@gmail.com on 26 Aug 2008 at 7:51

GoogleCodeExporter commented 8 years ago 
You can attach patches to this thread. I suppose the tortoise svn patch should 
be fine.

Original comment by cast...@gmail.com on 26 Aug 2008 at 11:33

GoogleCodeExporter commented 8 years ago 
Path applied against root public SVN HEAD nvidia-texture-tools

Original comment by andi.ire...@gmail.com on 27 Aug 2008 at 2:17

Attachments:

GoogleCodeExporter commented 8 years ago 
oops the ( in the file name was supposed to be a 9 :P

Original comment by andi.ire...@gmail.com on 27 Aug 2008 at 2:18

GoogleCodeExporter commented 8 years ago 
I forgot there were a couple of projects that use Release(no cuda) i fixed the
optimizations in those too and tested perforance gains.

Note enabling SSE2 via the ARCH:/SSE2 compiler command slows down conversion in 
these
tools by 20% so i don't use that.

I tested all possible combination's of optimization and this is basically the 
best one

Original comment by andi.ire...@gmail.com on 27 Aug 2008 at 2:23

Attachments:

GoogleCodeExporter commented 8 years ago 
FIXED : i had messed with NVCC paths in my local projects and didn't want to 
have
that in the patch

Original comment by andi.ire...@gmail.com on 29 Aug 2008 at 2:15

Attachments:

GoogleCodeExporter commented 8 years ago 
This looks good! The only issue I found is that the projects reference the CUDA 
paths 
assuming it's been installed at C:\CUDA which is not always the case. I've 
changed 
that to use the environment variables that the CUDA installer registers.

Original comment by cast...@gmail.com on 11 Sep 2008 at 7:49