https://bugzilla.novell.com/show_bug.cgi?id=324134
User [EMAIL PROTECTED] added comment https://bugzilla.novell.com/show_bug.cgi?id=324134#c39 --- Comment #39 from Paolo Molaro <[EMAIL PROTECTED]> 2008-10-06 03:33:16 MDT --- Andreas, your ppc-codegen.h changes are still full of irrelevant whitespace, reordering and formatting changes. I had asked you to provide just the needed changes for a reason: if your patchset was a working port, we could ignore the needless changes and just commit, but since it isn't we need to be able to see clearly which changes have been done and where the bugs are introduced. A few other notes. In ppc-codegen.h there is basically no need to check for ppc64 etc except in one place, all the others need to be removed as they just add clutter. It is a good idea to put the ppc64 specific macros at the end of the file, separated from the rest. A patch that just adds the needed stuff without all the formatting and reordering would be appreciated and it would be committed right away. arch/ppc/test.c changes can be committed. The tramp.c changes are needed for the interpreter, so they shouldn't be included as the interpreter is not working, so the changes must be completely untested as well. The atomic.h changes are fine, except for the gcc 4.1 parts which shouldn't be committed (they are also missing a check for __powerpc64__). The aot-compiler.c changes are ok, too, but before going on, please post a simple patch that introduces just a define __mono_ppc__ and __mono_ppc64__ and uses it in the proper places instead of the double check currently used. I'll review the rest of the changes after these cleanups. Thanks. -- Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the QA contact for the bug. _______________________________________________ mono-bugs maillist - [email protected] http://lists.ximian.com/mailman/listinfo/mono-bugs
