Evan Cheng wrote:> Hi Edwin, >Hi Evan,> It works fine for me. Thanks! > > We prefer a flat directory structure. Is it possible for you to > separate it out to sqlite3 and lemon rather than having them as sub- > directories under SQLite?Ok. The new package is here: http://edwintorok.googlepages.com/sqlite_lemon_llvmtest.tar.gz Anything else I should change?> Also, can you increase lemon's test size? >Lemon doesn't have a testsuite (those 40000+ tests are all for SQLite), however I found some projects that use lemon (its syntax is not compatible with yacc), and copied their grammar files into lemon's directory. I also wrote a wrapper main function for lemon that forks/executes for each file (lemon keeps some state in global variables, so I have to fork). The difference isn't much, it now takes 0.12 seconds to run lemon.> Please file bug about the performance issue. Thanks! >Done, bug #2174. Best regards, --Edwin
On Mar 25, 2008, at 12:56 PM, Török Edwin wrote:> Evan Cheng wrote: >> Hi Edwin, >> > > Hi Evan, > >> It works fine for me. Thanks! >> >> We prefer a flat directory structure. Is it possible for you to >> separate it out to sqlite3 and lemon rather than having them as sub- >> directories under SQLite? > > Ok. > The new package is here: > http://edwintorok.googlepages.com/sqlite_lemon_llvmtest.tar.gz > Anything else I should change?Looks good. Please commit.> > >> Also, can you increase lemon's test size? >> > > Lemon doesn't have a testsuite (those 40000+ tests are all for > SQLite), > however I found some projects that use lemon (its syntax is not > compatible with yacc), and copied their grammar files into lemon's > directory. > I also wrote a wrapper main function for lemon that forks/executes for > each file (lemon keeps some state in global variables, so I have to > fork). > > The difference isn't much, it now takes 0.12 seconds to run lemon.Still much too small to be really useful as a benchmark. Can you just run it 10 times? Evan> > >> Please file bug about the performance issue. Thanks! >> > > Done, bug #2174. > > Best regards, > --Edwin > _______________________________________________ > LLVM Developers mailing list > LLVMdev at cs.uiuc.edu http://llvm.cs.uiuc.edu > http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev
On Wed, Mar 26, 2008 at 4:09 AM, Evan Cheng <evan.cheng at apple.com> wrote:> > On Mar 25, 2008, at 12:56 PM, Török Edwin wrote: > > The new package is here: > > http://edwintorok.googlepages.com/sqlite_lemon_llvmtest.tar.gz > > Anything else I should change? > > Looks good. Please commit.Should I speak with Chris about commit access? I made the change below for lemon: http://edwintorok.googlepages.com/sqlite_lemon_llvmtest_2.tar.gz> > > > > > The difference isn't much, it now takes 0.12 seconds to run lemon. > > Still much too small to be really useful as a benchmark. Can you just > run it 10 times? >Ok, now it takes ~1.3 seconds. Best regards, --Edwin