Hi!

On Wed, Sep 25, 2019 at 10:46:57PM +0200, Andreas Tobler wrote:
> --- gcc/config/rs6000/t-freebsd64     (revision 276090)
> +++ gcc/config/rs6000/t-freebsd64     (working copy)
> @@ -27,3 +27,6 @@
>  MULTILIB_EXCEPTIONS     =
>  MULTILIB_OSDIRNAMES  = ../lib32
>  
> +SECURE_PLT = $(if $(findstring TARGET_FREEBSD32_SECURE_PLT=1, 
> $(tm_defines)),msecure-plt)
> +
> +MULTILIB_EXTRA_OPTS += $(SECURE_PLT)

$(findstring) isn't super great, it looks for substrings, so it would
also match "TARGET_FREEBSD32_SECURE_PLT=123"; you can use $(filter) instead?

Looks fine to me either way.


Segher

Reply via email to