Re: [PATCH] Add __builtin_stack_top

2015-08-19 Thread Segher Boessenkool
On Wed, Aug 19, 2015 at 03:18:46PM -0700, H.J. Lu wrote: > @deftypefn {Built-in Function} {void *} __builtin_argument_pointer (void) > This function is similar to @code{__builtin_frame_address} with an > argument of 0, but it returns the address of the incoming arguments to > the current function r

Re: [Patch/rtl-expand] Take tree range info into account to improve LSHIFT_EXP expanding

2015-08-19 Thread Jiong Wang
Jeff Law writes: >> +&& ! unsignedp > Don't you need to check that the conversion is actually a sign > extension. Oh, you're relying on the signedness of ops->type. That > should be sufficient. Exactly. >> +if (GET_MODE_SIZE (rmode) < GET_MODE_SIZE (mode) >> +

Re: [PATCH] PR66870 PowerPC64 Enable gold linker with split stack

2015-08-19 Thread Lynn A. Boger
Also, I don't think it is sufficient to add the option to enable the gold linker in gospec.c. That will only affect links when using gccgo. You also want to use the gold linker with gcc if -fsplit-stack is used. That is why I had to add it to a spec in linux64.h, so that -fuse-ld=gold is added i

Re: [PR64164] drop copyrename, integrate into expand

2015-08-19 Thread Alexandre Oliva
On Aug 19, 2015, Alexandre Oliva wrote: > I'm having some difficulty getting access to an ia64 box ATM, and for > ada bootstraps, a cross won't do, so... if you still have that build > tree around, any chance you could recompile par.o with both stage1 and > stage2, with -fdump-rtl-expand-details

C++ PATCH for c++/66957 (protected access)

2015-08-19 Thread Jason Merrill
The fix for bug 38579 was correct, but due to other bugs with our handling of protected access, it introduced bug 66957. The basic problem here was that [class.access.base] says, A member m is accessible at the point R when named in class N if -- m as a member of N is public, or -- m as a memb

Re: [C++ Patch] PR 67065 ("Missing diagnostics for ill-formed program with main variable instead of function")

2015-08-19 Thread Jason Merrill
OK. Jason

Re: [PATCH, libjava/classpath]: Fix overriding recipe for target 'gjdoc' build warning

2015-08-19 Thread Andrew Hughes
- Original Message - > Jeff> It's probably time for the occasional discussion WRT dropping > Jeff> gcj/libjava from the default languages and replace them with either > Jeff> Ada or Go. > > It's long past time to remove it. It's only had minimal maintenance for > years now. No one is wri

Re: [PATCH, libjava/classpath]: Fix overriding recipe for target 'gjdoc' build warning

2015-08-19 Thread Andrew Hughes
- Original Message - > On Fri, Aug 7, 2015 at 1:21 PM, Uros Bizjak wrote: > > > Attached patch fixes: > > > > Makefile:871: warning: overriding recipe for target 'gjdoc' > > Makefile:786: warning: ignoring old recipe for target 'gjdoc' > > > > build warning when compiling libjava. > > > >

[PATCH] Only accept BUILT_IN_NORMAL stringops for interesting_stringop_to_profile_p

2015-08-19 Thread Yangfei (Felix)
Hi, As DECL_FUNCTION_CODE is overloaded for builtin functions in different classes, so need to check builtin class before using fcode. Patch posted below. Bootstrapped on x86_64-suse-linux, OK for trunk? Thanks. Index: gcc/value-prof.c

Re: [PATCH, libjava/classpath]: Fix overriding recipe for target 'gjdoc' build warning

2015-08-19 Thread Tom Tromey
Andrew> No, it isn't. It's still a necessity for initial bootstrapping of Andrew> OpenJDK/IcedTea. Andrew Haley said the opposite here: https://gcc.gnu.org/ml/gcc-patches/2015-08/msg00537.html Tom

Re: Move some flag_unsafe_math_optimizations using simplify and match

2015-08-19 Thread Hurugalawadi, Naveen
Hi, Thanks again for your review and useful comments. >> I see. But I can't really help without a testcase that I can use to have a >> look >> (same for the above issue with the segfaults). The following testcase does not generate "x" as needed. double t (double x) { x =

Re: Move some flag_unsafe_math_optimizations using simplify and match

2015-08-19 Thread Marc Glisse
On Thu, 20 Aug 2015, Hurugalawadi, Naveen wrote: The following testcase does not generate "x" as needed. double t (double x) { x = sqrt (x) * sqrt (x); return x; } With -fno-math-errno, we CSE the calls to sqrt, so I would expect this to match: (m

Re: [PATCH, libjava/classpath]: Fix overriding recipe for target 'gjdoc' build warning

2015-08-19 Thread Uros Bizjak
On Thu, Aug 20, 2015 at 4:48 AM, Andrew Hughes wrote: > - Original Message - >> On Fri, Aug 7, 2015 at 1:21 PM, Uros Bizjak wrote: >> >> > Attached patch fixes: >> > >> > Makefile:871: warning: overriding recipe for target 'gjdoc' >> > Makefile:786: warning: ignoring old recipe for target

<    1   2