Re: [eigen] Totally missing /O2 for MSVC-Compiler

[ Thread Index | Date Index | More Archives ]

On Fri, Jan 30, 2009 at 8:34 PM, FMDSPAM <fmdspam@xxxxxxxxx> wrote:

>> I don't really understand what you meant here ? but FYI I'll make sure
>> the large_product test and lu test will be always compiled with /O2
>> for msvc (otherwise they take too long to run).
> OK, but why only these both testcases?
> a) I would consider to test all under the hardest conditions available.
> b) a general use of the compiler flags, causing the best possible
> performance is what we want to do in real use cases.

yes, of course, we should also run the tests in release mode. The goal
in making the debug mode the default one is to be able to debug broken
tests right away! Theoretically, using -DCMAKE_BUILD_TYPE=release
should do the job, but with MSVC I think you still have to choose
"release" from the UI. I don't know about NMake projects.

> I will take a look inside this and post valuable result, if any.
> Having in mind additionally test in regard of e.g. the /fp flag in terms of
> correctness and performance.
> I all is going fine, I consider to cdash several configuration on a regular
> basis. Objections ?

no pb

> FYI:
> Your original form "process initial unaligned coeffs" cachfriendlyproduct.h
> will to the job now perfectly, even with "complete optimization" (/Ox) flag
> set for all use cases. I have attached the revert patch. Maybe the your
> original form is cleaner and should be recreated?

ok, I'll do that.

Mail converted by MHonArc 2.6.19+