On Sat, Mar 23, 2019 at 10:20:16AM +0900, Michael Paquier wrote:
> On Fri, Mar 22, 2019 at 08:20:53PM -0400, Jeff Janes wrote:
> > PostgreSQL 12devel on aarch64-unknown-linux-gnu, compiled by gcc
> > (Ubuntu/Linaro 7.3.0-27ubuntu1~18.04) 7.3.0, 64-bit
>
> Adding Noah in CC as he has done the update of imath lately.
>
> > The attached patch adds PG_USED_FOR_ASSERTS_ONLY to silence it. Perhaps
> > there is a better way, given that we want to change imath.c as little as
> > possible from its upstream?
>
> Maybe others have better ideas, but marking the variable with
> PG_USED_FOR_ASSERTS_ONLY as you propose seems like the least invasive
> method of all.
That patch looks good. Thanks. The main alternative would be to pass
-Wno-unused for this file. Since you're proposing only one instance
PG_USED_FOR_ASSERTS_ONLY, I favor PG_USED_FOR_ASSERTS_ONLY over -Wno-unused.