Re: [patch] Fix oddity in personality routine

Jack Howarth wrote:
> On Thu, Dec 03, 2009 at 09:25:11PM +0000, Andrew Haley wrote:
>> Boehm, Hans wrote:
>>>> From:  Jack Howarth
>>>> Do you think that  -Wl,-allow_stack_execute needs to be 
>>>> passed even more widely than on just ecjx_LDFLAGS? Any 
>>>> suggestions as to where I should be passing it? Perhaps on 
>>>> something like LIBJAVA_LDFLAGS_NOUNDEF at the toplevel of 
>>>> libjava? Or should we even be building libffi and boehm-gc 
>>>> with that as well?
>>>>
>>> I haven't been following this completely, but:
>>>
>>> The GC also has a compile-time flag NO_EXECUTE_PERMISSION, which affects heap executability.
>> I think that's probably not the cause, since libffi allocates memory itself.
>>
>> Andrew.
> 
> Andrew,
>    On reflection, shouldn't something like...
> 
> Index: libjava/Makefile.in
> ===================================================================
> --- libjava/Makefile.in	(revision 154965)
> +++ libjava/Makefile.in	(working copy)
> @@ -8500,8 +8500,7 @@
>  	$(am__append_28)
>  gij_SOURCES = 
>  gij_LDFLAGS = -rpath $(dbexecdir) -rpath $(toolexeclibdir) \
> -	-shared-libgcc $(THREADLDFLAGS) $(extra_ldflags) \
> -	$(extra_gij_ldflags) 
> +	-shared-libgcc $(THREADLDFLAGS) $(extra_ldflags)
>  
>  gij_LINK = $(GCJLINK) $(gij_LDFLAGS)
>  gij_LDADD = -L$(here)/.libs libgij.la
> Index: libjava/Makefile.am
> ===================================================================
> --- libjava/Makefile.am	(revision 154965)
> +++ libjava/Makefile.am	(working copy)
> @@ -294,7 +294,7 @@
>  
>  LTLDFLAGS = $(shell $(top_srcdir)/../libtool-ldflags $(LDFLAGS))
>  GCJLINK = $(LIBTOOL) --tag=GCJ $(LIBTOOLFLAGS) --mode=link $(GCJ) -L$(here) \
> -	  $(JC1FLAGS) $(LTLDFLAGS) -o $@
> +	   $(extra_gij_ldflags) $(JC1FLAGS) $(LTLDFLAGS) -o $@
>  GCJ_FOR_ECJX = @GCJ_FOR_ECJX@
>  GCJ_FOR_ECJX_LINK = $(GCJ_FOR_ECJX) -o $@
>  LIBLINK = $(LIBTOOL) --tag=CXX $(LIBTOOLFLAGS) --mode=link $(CXX) -L$(here) \
> @@ -1065,8 +1065,7 @@
>  ## need this because we are explicitly using libtool to link using the
>  ## `.la' file.
>  gij_LDFLAGS = -rpath $(dbexecdir) -rpath $(toolexeclibdir) \
> -	-shared-libgcc $(THREADLDFLAGS) $(extra_ldflags) \
> -	$(extra_gij_ldflags) 
> +	-shared-libgcc $(THREADLDFLAGS) $(extra_ldflags)
>  gij_LINK = $(GCJLINK) $(gij_LDFLAGS)
>  ## See jv_convert_LDADD.
>  gij_LDADD = -L$(here)/.libs libgij.la
> 
> ...be appropriate to make sure that all java executables on darwin are linked with
> -Wl,-allow_stack_execute?

No, because it will only affect the binaries in gcj, not things the
user compiles.  That's why I suggested SYSTEMSPEC.

Andrew.