Btw, this is a decent summary: http://gentoo-wiki.com/CFLAGS
I think you may have the order wrong now? It seems like -march implies -mtune, so does the -mtune need to be after the -march? Not sure... On Fri, Sep 26, 2008 at 4:56 PM, Dean McNamee <[EMAIL PROTECTED]> wrote: > There is nothing beyond O3, so O9 is kinda a bad joke. Also, any > guesses why GCC thinks those variables might go uninitialized? (I saw > the same problem). > > Seconds, I do not think we should -mtune=nocona for a few reasons. > nocona is a NetBurst chip, so we would be tuning towards p4. If > anyone we should tune towards Core Duo, however... > > We should not tune towards a specific chip, either 1) mtune=generic, > which will tune for a balance of all chips, or mtune=native, which > will issue a cpuid, and tune for whatever machine you are building on. > > On Fri, Sep 26, 2008 at 2:21 PM, <[EMAIL PROTECTED]> wrote: >> >> Let's try it out. Please revert you changes if this does not >> yield a measurable speedup. >> >> LGTM, >> Lars >> >> >> http://codereview.chromium.org/4298/diff/1/2 >> File src/objects.cc (right): >> >> http://codereview.chromium.org/4298/diff/1/2#newcode1694 >> Line 1694: uint32_t index = 0; >> Please add a comment why this assignment is necessary. >> >> http://codereview.chromium.org/4298/diff/1/2#newcode1741 >> Line 1741: uint32_t index = 0; >> Please move the assignment into AsArrayIndex in objects-inl.h. >> >> http://codereview.chromium.org/4298 >> >> >> >> > --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
