Hi!

On Mon, Jan 31, 2022 at 07:43:32PM +0100, Jakub Jelinek wrote:
> As reported by Martin, while David has added OPTION_GLIBC define to aix
> and Iain to darwin, all the other non-linux targets now fail because
> rs6000.md macro isn't defined.
> 
> One possibility is to define this macro in option-defaults.h which on rs6000
> targets is included last, then we don't need to define it in aix/darwin
> headers and for targets using linux.h or linux64.h it will DTRT too.

That is okay for fallback definitions (like here), but not for "regular"
defaults, it is much nicer to have those explicit everywhere.

> +#ifndef OPTION_GLIBC
> +#define OPTION_GLIBC 0
> +#endif

So can you add a comment like that before this please?

Okay for trunk with that.  Thanks!


Segher

Reply via email to