Been looking at this off and on, and I'm still not sure I entirely get it - sorry.

On 11/11/2016 10:15 PM, David Malcolm wrote:
Implementing an RTL frontend by using the RTL reader from read
-rtl.c
means that we now need a diagnostics subsystem on the *host* for
handling errors in RTL files, rather than just on the build
machine.

So, there are two things that bother me about this patch description:
 - The host already has the full diagnostic subsystem. The fact that
   you're commenting out some of the functions in errors.c suggests
   that errors.c is conflicting with the full one.
 - We already compile errors.c for both build and host.

Is there a problem with using both the full and the light errors system for read-rtl, as available? Mismatches in function signatures or something like this?

-#ifdef HOST_GENERATOR_FILE
-#include "config.h"
-#define GENERATOR_FILE 1
-#else
+/* This file is compiled twice: once for the generator programs
+   once for the compiler.  */
+#ifdef GENERATOR_FILE
 #include "bconfig.h"
+#else
+#include "config.h"
 #endif
 #include "system.h"
 #include "errors.h"

The Makefile still has a HOST_GENERATOR_FILE definition for errors.c after this.


Bernd

Reply via email to