svn commit: r352604 - head/sys/mips/cavium

[Kyle Evans]

On Sun, Sep 22, 2019 at 1:30 PM Kyle Evans <kevans at freebsd.org> wrote:
>
> Author: kevans
> Date: Sun Sep 22 18:30:19 2019
> New Revision: 352604
> URL: https://svnweb.freebsd.org/changeset/base/352604
>
> Log:
>   octeon1: suppress a couple of warnings under clang
>
>   These appear in octeon-sdk -- there are new releases, but they don't seem to
>   address the running issues in octeon-sdk. GCC4.2 is more than happy, but
>   clang is much less-so and most of them are fairly innocuous and perhaps a
>   by-product of their style guide, which may make some of the changes harder
>   to upstream (if this is even possible anymore).
>
> Modified:
>   head/sys/mips/cavium/std.octeon1
>
> Modified: head/sys/mips/cavium/std.octeon1
> ==============================================================================
> --- head/sys/mips/cavium/std.octeon1    Sun Sep 22 18:27:57 2019        (r352603)
> +++ head/sys/mips/cavium/std.octeon1    Sun Sep 22 18:30:19 2019        (r352604)
> @@ -7,3 +7,4 @@ cpu             CPU_CNMIPS
>  makeoptions CFLAGS_PARAM_INLINE_UNIT_GROWTH=10000
>  makeoptions CFLAGS_PARAM_LARGE_FUNCTION_GROWTH=100000
>  makeoptions CFLAGS_ARCH_PARAMS="--param max-inline-insns-single=10000"
> +makeoptions "CWARNFLAGS.clang"+="-Wno-parentheses-equality -Wno-pointer-sign"

I meant to mention this... the extra quoting of "CWARNFLAGS.clang" is
because config(8) doesn't see the dot as a valid character in an ID
token, so unquoting it leaves us with a config(8) error -- we could
resolve this by allowing periods in ID strings, but this doesn't seem
to have bothered us thus far so I wonder if it's worth it.