In the course of working on unit tests in the 'parallel' branch, I came
across this inline comment in config/gen/makefiles.pm:

    # Why is this here?  I'd think this information belongs
    # in the CFLAGS.in file. -- A.D.  March 12, 2004
    if ( $conf->data->get('cpuarch') =~ /sun4|sparc64/ ) {

        # CFLAGS entries must be left-aligned.
        print {$CFLAGS} <<"EOF";
src/jit_cpu.c -{-Wcast-align}        # lots of noise!
src/nci.c     -{-Wstrict-prototypes} # lots of noise!
EOF
    }

My guess is that this code is not found in the CFLAGS.in template file
because it is platform-specific.  There is no platform- or OS-specific
code in CFLAGS.in.

So, is there any reason why we shouldn't delete the comment?

Thank you very much.
kid51

Reply via email to