On Thu, Jan 19, 2006 at 03:13:30PM +0100, Stefan Manegold wrote:
[...]
Moreover, if I understand the comments of Jan & Jens (separate mail discussion between Wouter, Jens, Jan & me) correctly, the current burK implementation/integration does cause some significant(?) performance penalty in the burG generated code, which will harm the performance of XQuery translation significantly(?).
I don't have any numbers, but I think the performance penalty is not "significant". Don't worry.
[...]
"Even" for the development trunk, I'd suggest to try an use the macro BURKOWSKI (and an AC_CONDITIONAL that we should add), that is/are set with the --enable-burk configure switch, to remove ALL Burkowski related code from the compilation; i.e., (if possible) also the declarative parts in compiler/algebra/core2alg.brg, compiler/core/coreopt.brg, compiler/core/simplify.brg, compiler/semantics/typecheck.brg, as well as the changes in compiler/parser/parser.y, compiler/parser/scanner.l, & runtime/pathfinder.mx. In fact, Jan & Jens suggest to even reduce the changes in (at least) compiler/algebra/core2alg.brg to a minimum. Maybe, Jan & Jens can help with the *.brg file (also optimizing the indices), I could help with runtime/pathfinder.mx, and the AC_CONDITIONAL to avoid compilation of pf_burk unless configure with --enable-burk.
I think Wouter did the best we can do here already. BURG does not provide any preprocessing features, so we cannot skip any declarations based on ./configure options. The same argument holds for the Yacc grammar as well. Wouter did use preprocessor macros in the action code in both cases (right?), so I think the impact of the Burkowski stuff with --disable-burk is as small as we can get. (Please don't come up with ideas such as feeding .brg and .y files into other preprocessors first. I really don't want to have that trouble in our code.) Regards, Jens
ps: the release must be ready by Sunday Jan 22, 23:59:59 [...]
Is Peter still in Argentina? Due to the time lag that could save us some hours. :-) -- Jens Teubner Technische Universitaet Muenchen, Department of Informatics D-85748 Garching, Germany Tel: +49 89 289-17259 Fax: +49 89 289-17263 Computer Scientist, n.: A device that turns coffee into programs.