That is certainly a possibility. The original motivation was to
implement -fopt-info correctly. If there are other use cases, then I
can enhance the patch.
Thanks,
Sharad
On Mon, May 7, 2012 at 3:02 PM, Gabriel Dos Reis
g...@integrable-solutions.net wrote:
On Mon, May 7, 2012 at 4:58 PM,
Hello,
The attached patch is the same as in the PR comment #37.
Tested with
make -k check RUNTESTFLAGS=--target_board=sh-sim
\{-m2/-ml,-m2/-mb,-m2a/-mb,-m2a-single/-mb,-m4/-ml,-m4/-mb,
-m4-single/-ml,-m4-single/-mb,-m4a-single/-ml,-m4a-single/-mb}
and no new failures.
OK?
Cheers,
Oleg
Oleg Endo oleg.e...@t-online.de wrote:
The attached patch is the same as in the PR comment #37.
Tested with
make -k check RUNTESTFLAGS=--target_board=sh-sim
\{-m2/-ml,-m2/-mb,-m2a/-mb,-m2a-single/-mb,-m4/-ml,-m4/-mb,
-m4-single/-ml,-m4-single/-mb,-m4a-single/-ml,-m4a-single/-mb}
and no
Fix debug info of nested inline functions:
http://gcc.gnu.org/ml/gcc-patches/2012-03/msg00161.html
Emit variable as size attribute in debug info:
http://gcc.gnu.org/ml/gcc-patches/2012-04/msg00422.html
Implement static stack checking on IA-64:
Hi Richard,
Resurrecting the issue now that I have new data.
On Wed, 14 Dec 2011, Richard Sandiford wrote:
After some thinking I decided the simplest approach will be just emitting
the missing location directive in the context of the MIPS16 thunk being
built that will apply to the
On Mon, May 7, 2012 at 11:11 PM, H.J. Lu hongjiu...@intel.com wrote:
Hi,
I am preparing to update GLIBC longlong.h from GCC. This patch updates
GCC longlong.h to use a URL instead of an FSF postal address and replace
spaces with tab. OK to install?
Since I'd like to simply copy
On Tue, May 8, 2012 at 1:07 AM, Matthias Klose d...@ubuntu.com wrote:
The lto plugin is installed without x bits set, but gcc-ar.c still checks for
the execute bits. There is no need to have the lto plugin to have the x bits
set, so just check that it is readable.
Ok for the trunk and the 4.7
On Tue, May 8, 2012 at 5:39 AM, Hans-Peter Nilsson
hans-peter.nils...@axis.com wrote:
The problem was spotted while fixing PR53272, a target bug with
crisv32-* involving the error-prone notice_update_cc function.
When wrapping up the test-case to use as a run-test, adding main
and auxiliary
On Tue, May 8, 2012 at 12:02 AM, Gabriel Dos Reis
g...@integrable-solutions.net wrote:
On Mon, May 7, 2012 at 4:58 PM, Sharad Singhai sing...@google.com wrote:
This is the first patch for planned improvements to dump
infrastructure. Please reference the discussion in
On Tuesday, May 08, 2012 10:43:14 Richard Guenther wrote:
On Mon, May 7, 2012 at 11:11 PM, H.J. Lu hongjiu...@intel.com wrote:
Hi,
I am preparing to update GLIBC longlong.h from GCC. This patch
updates GCC longlong.h to use a URL instead of an FSF postal address
and replace spaces
Looks like a good idea.
Marc Glisse marc.gli...@inria.fr writes:
For the testsuite, since the patch is not in a particular target, it would
be better to have a generic test (in gcc.dg?), but I don't really know how
to write a generic one, so would a test in gcc.target/i386 that scans
the
On 08/05/12 10:04, Andreas Jaeger wrote:
On Tuesday, May 08, 2012 10:43:14 Richard Guenther wrote:
On Mon, May 7, 2012 at 11:11 PM, H.J. Lu hongjiu...@intel.com wrote:
Hi,
I am preparing to update GLIBC longlong.h from GCC. This patch
updates GCC longlong.h to use a URL instead of an FSF
Hi,
I have been investigating some failures in the MIPS16 GDB test suite that
appeared to me that was caused by some weird out-of place line
information.
For example we get this in gdb.base/break.exp, where a breakpoint is set
at main and the beginning of the function (after removing some
Hi,
On Mon, 7 May 2012, Mike Stump wrote:
On May 7, 2012, at 6:11 AM, Michael Matz wrote:
I'd like to retain the #if 0 code therein,
Can you structure this code as
#define DEBUG 0
if (DEBUG) ...
?
If so, that would be a preferable way to structure the code.
Sure, consider
On 04/30/2012 01:46 PM, Dodji Seketeli wrote:
Dodji Seketeli do...@redhat.com writes:
I am proposing a series of patches which is supposed to address the
remaining issues (I am aware of) preventing us from enabling the
-ftrack-macro-expansion by default.
The idea is to address each issue I
On Mon, May 07, 2012 at 05:23:33PM +0200, Olivier Hainque wrote:
(emit_frame_save): Don't handle reg+reg addressing.
introduces an assert on which we now trip compiling unwind-dw2.c for SPE
configurations. We now fall into the TARGET_SPE_ABI part of
/* Some cases that need
On Tuesday, May 08, 2012 11:59:34 Richard Earnshaw wrote:
On 08/05/12 10:04, Andreas Jaeger wrote:
On Tuesday, May 08, 2012 10:43:14 Richard Guenther wrote:
On Mon, May 7, 2012 at 11:11 PM, H.J. Lu hongjiu...@intel.com wrote:
Hi,
I am preparing to update GLIBC longlong.h from GCC.
Marc Glisse marc.gli...@inria.fr writes:
On Tue, 8 May 2012, Richard Sandiford wrote:
I know you said that generalising it could be done later,
and that's fine, but it looks in some ways like it would
be easier to go straight for the more general:
GET_CODE (trueop0) == VEC_CONCAT
This makes us fold (X * 8) 5 to zero and (X * 6) 5 to (X * 6) 4.
It amends the fix for PR52134 in that I noticed we fail to do some
simplifications on size expressions (later on SSA, CCP does the above
transform, though not (yet) changing of the constant).
Bootstrap and regtest ongoing on
This optimizes byte_from_pos and pos_from_bit by noting that we operate
on sizes whose computations have no intermediate (or final) overflow.
This is the single patch necessary to get Ada to bootstrap and test
with TYPE_IS_SIZETYPE removed. Rather than amending size_binop
(my original plan) I
This removes the TYPE_IS_SIZETYPE macro and all its uses (by
assuming it returns zero and applying trivial folding). Sizes
and bitsizes can still be treat specially by means of knowing
what the values represent and by means of using helper functions
that assume you are dealing with sizes (in
On Tue, 8 May 2012, Richard Guenther wrote:
This removes the TYPE_IS_SIZETYPE macro and all its uses (by
assuming it returns zero and applying trivial folding). Sizes
and bitsizes can still be treat specially by means of knowing
what the values represent and by means of using helper
On Tue, 8 May 2012, Richard Sandiford wrote:
Marc Glisse marc.gli...@inria.fr writes:
On Tue, 8 May 2012, Richard Sandiford wrote:
I know you said that generalising it could be done later,
and that's fine, but it looks in some ways like it would
be easier to go straight for the more general:
Marc Glisse marc.gli...@inria.fr writes:
Realised afterwards that both versions need to check
GET_MODE_NUNITS (mode) == 2, because we're requiring OP0 and OP1
to be scalar. Sorry for not noticing first time.
I thought that was a consequence of
XVECLEN (trueop1, 0) == 2
(in the lines
On 05/07/2012 11:28 PM, Paolo Carlini wrote:
error: could not convert ‘b.main()::lambda()()’ from ‘void’ to ‘bool’
It wouldn't say operator()?
I think I'd leave that alone; it is somewhat more informative (about
what b() expands to) and we're moving toward replacing %qE with caret
anyway.
On Tue, 8 May 2012, Matthias Klose wrote:
On 20.08.2011 21:51, Matthias Klose wrote:
Multiarch [1] is the term being used to refer to the capability of a system
to
install and run applications of multiple different binary targets on the
same
system.
please find attached an updated
On 05/08/2012 03:00 PM, Jason Merrill wrote:
On 05/07/2012 11:28 PM, Paolo Carlini wrote:
error: could not convert ‘b.main()::lambda()()’ from ‘void’ to ‘bool’
It wouldn't say operator()?
Nope, it says exactly the above. If you tell me what would be more
sensible which remaining informative,
This patch adds the microblaze-*-rtems* target to gcc.
OK to apply?
2012-05-07 Joel Sherrill joel.sherr...@oarcorp.com
* config.gcc (microblaze-*-rtems*): New target.
* config/microblaze/rtems.h: New file
--
Joel Sherrill, Ph.D. Director of Research Development
Sorry.. missed the attachment
This patch adds the microblaze-*-rtems* target to gcc.
OK to apply?
2012-05-07 Joel Sherrill joel.sherr...@oarcorp.com
* config.gcc (microblaze-*-rtems*): New target.
* config/microblaze/rtems.h: New file.
--
Joel Sherrill, Ph.D. Director of
Hi Honza,
I added L_gcov_reset and L_gcov_dump for the new interfaces, and also
added a description into the gcov man page. Let me know if it looks
ok now.
Bootstrapped and tested on x86_64-unknown-linux-gnu.
Thanks,
Teresa
2012-05-08 Teresa Johnson tejohn...@google.com
*
gcc/config/i386/freebsd.h: Likewise.
---
gcc/config/freebsd-spec.h |9 +++--
gcc/config/i386/freebsd.h |9 +++--
2 files changed, 6 insertions(+), 12 deletions(-)
diff --git a/gcc/config/freebsd-spec.h b/gcc/config/freebsd-spec.h
index 770a3d1..2808582 100644
---
For the record, there's a similar logic in FreeBSD's gcc:
http://svnweb.freebsd.org/base/head/contrib/gcc/config/freebsd-spec.h?revision=200038view=markup
Regards,
Alexis.
On Tue, May 8, 2012 at 6:32 AM, Alan Modra amo...@gmail.com wrote:
OK, the assert is doing its job. I wanted to minimize the number of
places that need temporary hard regs, so that tracking of which hard
reg is in use can all be done in rs6000_emit_prologue.
The problem is that the insns
On Sun, May 6, 2012 at 11:41 AM, Richard Sandiford
rdsandif...@googlemail.com wrote:
x32 uses a mixture of MEM address modes for the same address space.
Some MEMs have SImode addresses, some have DImode. This means that
the currently common idiom:
targetm.addr_space.address_mode
Hello,
This patch adds a flag to guard the optimization that optimize the
following code away:
free (malloc (4));
In some cases, we'd like this type of malloc/free pairs to remain in
the optimized code.
Tested with bootstrap, and no regression in the gcc testsuite.
Is it ok for mainline?
Hello Everyone,
This patch is for the Cilkplus branch affecting mostly the C compiler. This
patch will insert elemental functions for C programs.
Thanks,
Balaji V. Iyer.diff --git a/gcc/ChangeLog.cilk b/gcc/ChangeLog.cilk
index d6b28e2..4f01d31 100644
--- a/gcc/ChangeLog.cilk
+++
Tristan == Tristan Gingold ging...@adacore.com writes:
Tristan 2012-05-04 Tristan Gingold ging...@adacore.com
Tristan* expr.c (interpret_float_suffix): Add a guard.
Ok.
Tom
Hi,
On Fri, 4 May 2012, Richard Guenther wrote:
Well, as you touch all places that refer to header/footer why not
introduce macros to access them ...
Currently regstrapping on x86_64-linux, okay if that passes?
Ok with the above change.
For completeness, this is what I checked in
On Mon, May 7, 2012 at 10:40 PM, Uros Bizjak ubiz...@gmail.com wrote:
2012-05-07 Uros Bizjak ubiz...@gmail.com
PR target/53250
* config/i386/i386.c (ix86_set_reg_reg_cost): New function.
(ix86_rtx_costs): Handle SET.
Patch was tested on x86_64-pc-linux-gnu {,-m32}.
Hi DJ,
Make sure a match_dup will still match the generated pattern later,
I've had problems with match_dup not matching two rtx that
rtx_equals() says are the same but not physically the same.
I have tried, but failed, to find a way to trigger the use of the
bset_qi pattern. :-( I tried
To be clear, this flag is for malloc implementation (such as tcmalloc)
with side effect unknown to the compiler. Using -fno-builtin-xxx is
too conservative for that purpose.
David
On Tue, May 8, 2012 at 7:43 AM, Dehao Chen de...@google.com wrote:
Hello,
This patch adds a flag to guard the
On Mon, May 7, 2012 at 6:44 PM, Gabriel Dos Reis
g...@integrable-solutions.net wrote:
On Mon, May 7, 2012 at 7:10 PM, Sterling Augustine
saugust...@google.com wrote:
This is the second in the series of patches to make c decl pretty printing
more closely match the demangler. A full explanation
On Tue, May 8, 2012 at 11:20 AM, Sterling Augustine
saugust...@google.com wrote:
Do you have a suggestion for fixing the disagreement? I would love to
add this as a parameter somewhere, but the decision is very deep in
the internals of the pretty printer.
I disagree with your characterization
Hello Thomas,
below a very timely review - your patch is not even a month old
and was never pinged, besides, you have chosen an unlucky day.
(In other words: Sorry for the slow review.)
Thomas Koenig wrote on Fri, 13 Apr 2012:
this patch replaces a != '' with len_trim(a) != 0, to
speed up the
Here is a new version.
gcc/ChangeLog
2012-05-08 Marc Glisse marc.gli...@inria.fr
* simplify-rtx.c (simplify_binary_operation_1): Optimize shuffle
of concatenations.
gcc/testsuite/ChangeLog
2012-05-08 Marc Glisse marc.gli...@inria.fr
* gcc.target/i386/shuf-concat.c:
Hi,
This patch is for google-4_6 branch only.
It fixes a profile mismatch in streaming LIPO and recovers the performance loss
due to this.
Tested with google internal benchmarks.
Thanks,
2012-05-08 Rong Xu x...@google.com
* ipa-inline.c (fixed_arg_function_p):
Ok.
David
On Tue, May 8, 2012 at 9:53 AM, Rong Xu x...@google.com wrote:
Hi,
This patch is for google-4_6 branch only.
It fixes a profile mismatch in streaming LIPO and recovers the performance
loss
due to this.
Tested with google internal benchmarks.
Thanks,
2012-05-08 Rong Xu
Several /gcc.target/i386/hle-*.c tests are currently failing on Solaris
9/x86 with Sun as:
FAIL: gcc.target/i386/hle-add-acq-1.c scan-assembler lock[
\\n\\t]+(xacquire|.byte[ \\t]+0xf2)[ \\t\\n]+add
The .s file has
lock;
.byte 0xf2
but the scan-assembler regex currently
Maciej W. Rozycki ma...@codesourcery.com writes:
and the mips-linux-gnu configuration is not ready yet for MIPS16 testing.
Out of interest, what goes wrong? I've been testing -mabi=32/-mips16 on
mips64-linux-gnu for some time without difficulty.
I've thought some pieces are missing
Maciej W. Rozycki ma...@codesourcery.com writes:
gcc-mips16-gp-pseudo-loc.patch
Index: gcc-fsf-trunk-quilt/gcc/config/mips/mips.c
===
--- gcc-fsf-trunk-quilt.orig/gcc/config/mips/mips.c 2012-05-02
23:42:46.185566469 +0100
+++
Marc Glisse marc.gli...@inria.fr writes:
Here is a new version.
gcc/ChangeLog
2012-05-08 Marc Glisse marc.gli...@inria.fr
* simplify-rtx.c (simplify_binary_operation_1): Optimize shuffle
of concatenations.
OK, thanks. I'll leave an x86 maintainer to review the testcase,
but
#define q ((char *)0x1234)
foo(int x)
{
*q |= (1 (char)x);
}
$ m32c-elf-gcc -S -O3 nick.c
.global _foo
_foo:
mov.w r1,a0; 20
movhi_op/3
bset4660[a0] ; 11
On Fri, May 04, 2012 at 09:46:22PM +0800, Dehao Chen wrote:
Thanks for the prompt response. Attached is the updated patch.
Passed bootstrap and all regression tests.
All the new testcases fail for me, on both x86_64-linux and i686-linux,
apparently because of incorrectly committed patch (each
Hi!
The attached testcase loops endlessly, using more and more memory.
The problem is that the prev stmt iterator sometimes references stmts that
remove_prop_source_from_use decides to remove, and since Michael's
gimple seq changes that seems to be fatal.
Fixed by not keeping an iterator, but
On Tue, 8 May 2012, Richard Sandiford wrote:
Marc Glisse marc.gli...@inria.fr writes:
Here is a new version.
gcc/ChangeLog
2012-05-08 Marc Glisse marc.gli...@inria.fr
* simplify-rtx.c (simplify_binary_operation_1): Optimize shuffle
of concatenations.
OK, thanks. I'll
On Tue, May 8, 2012 at 10:37 AM, Jan Hubicka hubi...@ucw.cz wrote:
Hi Honza,
I added L_gcov_reset and L_gcov_dump for the new interfaces, and also
added a description into the gcov man page. Let me know if it looks
ok now.
Bootstrapped and tested on x86_64-unknown-linux-gnu.
On May 8, 2012, at 10:19 AM, Rainer Orth wrote:
Several /gcc.target/i386/hle-*.c tests are currently failing on Solaris
9/x86 with Sun as:
FAIL: gcc.target/i386/hle-add-acq-1.c scan-assembler lock[
\\n\\t]+(xacquire|.byte[ \\t]+0xf2)[ \\t\\n]+add
The .s file has
lock;
On Tue, May 8, 2012 at 10:37 AM, Jan Hubicka hubi...@ucw.cz wrote:
Hi Honza,
I added L_gcov_reset and L_gcov_dump for the new interfaces, and also
added a description into the gcov man page. Let me know if it looks
ok now.
Bootstrapped and tested on
On 05/07/12 19:11, Andrew MacLeod wrote:
On 05/07/2012 07:04 PM, Aldy Hernandez wrote:
Andrew suggested the correct fix was to add a new pass that was able
to do some ?? flow sensitive data flow analysis ?? that could discover
these unreachable paths and insert the 0 phis at the start of the
On 8 May 2012 18:20, Sterling Augustine saugust...@google.com wrote:
On Mon, May 7, 2012 at 6:44 PM, Gabriel Dos Reis
g...@integrable-solutions.net wrote:
On Mon, May 7, 2012 at 7:10 PM, Sterling Augustine
saugust...@google.com wrote:
This is the second in the series of patches to make c decl
Richard Earnshaw rearn...@arm.com writes:
FTR, this caused
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53278
Well, this really has been a brown-paper-bag patch. Fixed as below.
Tested on x86_64-linux-gnu and applied as obvious.
Richard
gcc/
PR rtl-optimization/53278
*
The return value of build_range_check can be NULL and integer_zerop
cannot handle that. Bootstrapped and regression tested. John David
Anglin tested that it fixes the ICE in hppa. Committed to mainline as
obvious.
2012-05-09 Manuel López-Ibáñez m...@gcc.gnu.org
PR c++/53261
*
On 6 May 2012 20:45, Joseph S. Myers jos...@codesourcery.com wrote:
One idea could be to have an additional auto_handle_option() that is
generated from the awk scripts and called after all other
handle_option functions. This function will populate a switch with
group options and the
On Wed, 9 May 2012, Manuel L?pez-Ib??ez wrote:
which looks correct to me. However, the build fails because now
options.h requires input.h which requires line-map.h, which is not
included when building for example libgcc. options.h is included by
tm.h, so it basically appears everywhere.
gcc/trans-mem.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/gcc/trans-mem.c b/gcc/trans-mem.c
index 2badf25..24073fa 100644
--- a/gcc/trans-mem.c
+++ b/gcc/trans-mem.c
@@ -4721,7 +4721,7 @@ ipa_tm_transform_clone (struct cgraph_node *node)
/* If this function makes
Found this while testing the C++ conversion for vec.[ch] on the
cxx-conversion branch. We do not build the build/*.o files with g++,
so I was getting lots of syntax errors while compiling build/vec.o.
I am not completely sure if the changes are correct. But it works for
me.
Tested on x86_64.
Hi,
this patch puts varpool more into control of what is output. Traditionally
frontends were outputting variables as they parsed them via
tree_rest_of_compilation or at the end of compilation via
wrapup_global_declarations that went through all the decls, output unused
warnings and called
Without this patch we generate calls to TM_GETTMCLONE for functions
called indirectly, but we don't actually store the clone mapping in
the clone table because we think the functions are not needed.
Compiles fine, dies at runtime. See GCC Bugzilla â Bug 53008
gcc/trans-mem.c | 14
Without this patch it is perfectly fine to assign non-transaction_safe
functions to function pointers marked as transaction_safe. Unpleasantness
happens at run time.
e.g.
__attribute__((transaction_safe)) long (*compare)(int, int);
compare = my_funky_random_function;
gcc/c-typeck.c |
On 12-05-08 15:46 , Diego Novillo wrote:
Found this while testing the C++ conversion for vec.[ch] on the
cxx-conversion branch. We do not build the build/*.o files with g++,
so I was getting lots of syntax errors while compiling build/vec.o.
I am not completely sure if the changes are correct.
On Tue, May 8, 2012 at 4:38 PM, Manuel López-Ibáñez
lopeziba...@gmail.com wrote:
A suggestion: Make dwarf_name call the demangler, and then a (new?) a
function that converts a mangled decl to a human-readable string. In
any case, the pretty-printer does a lot of stuff that is mostly
useless
A suggestion: Make dwarf_name call the demangler, and then a (new?) a
function that converts a mangled decl to a human-readable string. In
any case, the pretty-printer does a lot of stuff that is mostly
useless for just printing a declaration (translation, wrapping, etc.).
Bonus point if all
From: Richard Guenther richard.guent...@gmail.com
Date: Tue, 8 May 2012 10:50:43 +0200
On Tue, May 8, 2012 at 5:39 AM, Hans-Peter Nilsson
hans-peter.nils...@axis.com wrote:
The problem was spotted while fixing PR53272, a target bug with
crisv32-* involving the error-prone notice_update_cc
On 08.05.2012 15:20, Joseph S. Myers wrote:
On Tue, 8 May 2012, Matthias Klose wrote:
On 20.08.2011 21:51, Matthias Klose wrote:
Multiarch [1] is the term being used to refer to the capability of a system
to
install and run applications of multiple different binary targets on the
same
Sorry for the error. Here is a new patch to fix them:
gcc/testsuite/ChangeLog:
2012-05-08 Dehao Chen de...@google.com
* gcc.dg/predict-1.c: Remove the replicated text in this text.
* gcc.dg/predict-2.c: Likewise.
* gcc.dg/predict-3.c: Likewise.
*
Hi!
On Fri, 27 Apr 2012 10:29:06 +0200, Jakub Jelinek ja...@redhat.com wrote:
On Fri, Apr 27, 2012 at 12:42:41PM +0800, Thomas Schwinge wrote:
GET_MODE_BITSIZE (lmode)« (8 bits). (With the current sources, lmode is
VOIDmode.)
Is emmitting »BIT_FIELD_REF *common, 32, 0 255«
This fixes another statement-placement issue when reassociating
expressions with repeated factors. Multiplies feeding into
__builtin_powi calls were not getting placed properly ahead of them in
some cases.
Bootstrapped and tested on powerpc64-unknown-linux-gnu with no new
regressions. I've also
We never really should have been passing this Solaris compatability
option for specifying library paths in the first place. Let them
get set the right way when the compiler passes in -L options.
Committed to master.
gcc/
* config/sparc/linux.h (LINK_SPEC): Don't pass -Y option.
78 matches
Mail list logo