Tobias Burnus wrote:
Build and regtested on x86-64-gnu-linux. I intent to commit the patch
soon.
Committed as Rev. 198623 + merged trunk into the branch (Rev. 198624).
Tobias
Uros,
The fix looks good to me.
Best regards.
Yuri.
2013/5/3 Uros Bizjak ubiz...@gmail.com:
Hello!
Apparently, the wrong insn is used when adding insn dependencies. The
last index gets updated to the found !NONDEBUG_INSN. Use first_arg
that points to the right insn instead.
2013-05-03
On 05/03/2013 10:24 PM, Jason Merrill wrote:
Last year Florian fixed the compiler to detect overflow in array new
size calculations and pass (size_t)-1 in that case. But C++11 specifies
that in case of overflow the program throws std::bad_array_new_length
It's at http://gcc.gnu.org/ml/gcc-patches/2013-04/msg01149.html
Thanks in advance.
--
Eric Botcazou
Hi,
I am re-submitting the patch to use html.css from the update_web_docs_svn
script present in the maintainer-scripts folder. The smallexample attributes
are defined to use a medium font size with a light blue background for
readability and clarity.
2013-05-06 Shakthi Kannan
On Fri, May 3, 2013 at 10:46 PM, Jeff Law l...@redhat.com wrote:
On 05/02/2013 01:55 AM, Richard Biener wrote:
On Wed, May 1, 2013 at 10:26 PM, Jeff Law l...@redhat.com wrote:
range_fits_type_p erroneously returns true in cases where the range has
overflowed. So for example, we might have
On Sun, May 5, 2013 at 10:45 PM, Bill Schmidt
wschm...@linux.vnet.ibm.com wrote:
When creating a phi-adjustment along an incoming edge, where the related
argument should use the hidden basis directly, I created a situation
where we double-count the subsequent adjustment based on the candidate's
Hello,
this patch is extracted from
http://gcc.gnu.org/ml/gcc-patches/2013-04/msg00783.html
When I copied this code from the C front-end and removed c_fully_fold, I
apparently also removed the save_expr, so I am putting it back in. I added
it after convert while the C front-end has it before
Hi,
This patch fixes G10 target support for interrupt functions in the recently
submitted patch, Improve interrupt function generation
http://gcc.gnu.org/ml/gcc-patches/2013-05/msg00011.html
The G10 target does not support the sel rbn insn as it has only bank 0.
Conditions have been added to
On 05/06/2013 05:46 AM, Florian Weimer wrote:
Nice, this is simpler than expected. However, it makes the call sites
even more bloated.
Hmm, perhaps the checking should be wrapped in an inline function, so
that the inliner can decide whether or not to expand it at the call site...
This
On 05/06/2013 02:39 PM, Jason Merrill wrote:
On 05/06/2013 05:46 AM, Florian Weimer wrote:
Nice, this is simpler than expected. However, it makes the call sites
even more bloated.
Hmm, perhaps the checking should be wrapped in an inline function, so
that the inliner can decide whether or not
Hi,
the issue is that we -Wunused-variable warn for:
const auto PI_3 = 3.1415926F;
The reason is that TREE_READONLY is not set (d instead r in the .o).
In turn, that's because the first and only time
cp_apply_type_quals_to_decl is called, at the end of grokdeclarator,
nothing happens
On Mon, 2013-05-06 at 07:56 +0300, Laurynas Biveinis wrote:
[...snip...]
The patch is OK with some minor formatting errors fixed:
[...snip...]
Thanks for reviewing this. BTW, is there an automated tool for checking
for formatting issues like this in patches? (contrib/check_GNU_style.sh
Jakub == Jakub Jelinek ja...@redhat.com writes:
Jakub 2013-05-04 Jakub Jelinek ja...@redhat.com
Jakub PR libgcj/57074
Jakub * class.c (emit_symbol_table): Use array type of the
Jakub right size for the_syms_decl and its DECL_INITIAL, instead
Jakub of symbols_array_type. Set TREE_TYPE
On Tue, Apr 30, 2013 at 10:35:21AM -0400, Jason Merrill wrote:
On 04/29/2013 02:32 PM, Jakub Jelinek wrote:
Should I copy the parser-omp_declare_simd_clauses vector pointer
say into cp_declarator structure so that grokfndecl could grab it from
there?
That sounds good.
Also, for the
This fixes an anonymous SSA name issue.
Bootstrapped and tested on x86_64-unknonw-linux-gnu, applied.
Richard.
2013-05-06 Richard Biener rguent...@suse.de
PR tree-optimization/57185
* tree-parloops.c (add_field_for_reduction): Handle anonymous
SSA names properly.
On 05/03/2013 04:46 PM, Teresa Johnson wrote:
On Fri, May 3, 2013 at 12:48 PM, Teresa Johnson tejohn...@google.com wrote:
Yes it will ICE on failure. What is the guideline on c.torture vs gcc.dg?
I don't think there's any general guidelines.
c-torture was an older framework that was
On 05/05/2013 04:18 AM, Steven Bosscher wrote:
On Fri, May 3, 2013 at 3:10 PM, Julian Brown wrote:
gcc/
* gcse.c (compute_ld_motion_mems): Invalidate non-simple mem refs
in REG_EQUAL notes.
Makes sense to me. Looking at REG_EQUAL notes in hash_scan_set is
something relatively
On Mon, May 06, 2013 at 09:24:28AM -0600, Jeff Law wrote:
On 05/03/2013 04:46 PM, Teresa Johnson wrote:
On Fri, May 3, 2013 at 12:48 PM, Teresa Johnson tejohn...@google.com wrote:
Yes it will ICE on failure. What is the guideline on c.torture vs gcc.dg?
I don't think there's any general
This removes lazy_create_slsr_reg and replaces uses of make_ssa_name
with make_temp_ssa_name, removing the need for a bunch of unnecessary
vars.
Bootstrapped and tested on powerpc64-unknown-linux-gnu with no new
regressions. Ok for trunk?
Thanks,
Bill
2013-05-06 Bill Schmidt
On 05/06/2013 08:46 AM, Florian Weimer wrote:
On 05/06/2013 02:39 PM, Jason Merrill wrote:
On 05/06/2013 05:46 AM, Florian Weimer wrote:
Nice, this is simpler than expected. However, it makes the call sites
even more bloated.
Hmm, perhaps the checking should be wrapped in an inline
On Thu, 2013-04-25 at 14:10 -0400, David Malcolm wrote:
On Thu, 2013-04-25 at 13:43 -0400, Diego Novillo wrote:
On 2013-04-25 12:30 , David Malcolm wrote:
diff --git a/contrib/ChangeLog b/contrib/ChangeLog
index 9f4505b..01afcb5 100644
--- a/contrib/ChangeLog
+++
On 05/06/2013 11:04 AM, Jakub Jelinek wrote:
isn't it too invasive to the C++ FE?
It is pretty invasive. Why not put the information in cp_declarator, as
you suggested in your earlier email? You could even put it in the
attributes field.
Jason
OK.
Jason
On 05/06/2013 07:46 AM, Marc Glisse wrote:
I added it after convert while the C front-end has it before convert, I
can put it before if you prefer.
Let's be consistent.
OK for trunk and 4.8.
Jason
On Mon, May 6, 2013 at 5:28 PM, Jeff Law wrote:
On 05/05/2013 04:18 AM, Steven Bosscher wrote:
On Fri, May 3, 2013 at 3:10 PM, Julian Brown wrote:
gcc/
* gcse.c (compute_ld_motion_mems): Invalidate non-simple mem refs
in REG_EQUAL notes.
Makes sense to me. Looking at
On 05/05/2013 09:57 AM, Dave Korn wrote:
This sounds like a bad idea to me, and not just because the locking
mechanism is dodgy. Is the problem more widespread than just your laptop?
Does it affect other host OSs? Linking multiple frontends at once doesn't
lock up my desktop PC, so I'd
Hi,
since
[patch] x86_64: CFI unwinding stop in _start
http://sourceware.org/ml/libc-alpha/2012-03/msg00573.html
there is a regression reproducible with gold:
http://sourceware.org/bugzilla/show_bug.cgi?id=15407
as .eh_frame is created before the __EH_FRAME_BEGIN__
On 05/06/2013 10:27 AM, Steven Bosscher wrote:
On Mon, May 6, 2013 at 5:28 PM, Jeff Law wrote:
On 05/05/2013 04:18 AM, Steven Bosscher wrote:
On Fri, May 3, 2013 at 3:10 PM, Julian Brown wrote:
gcc/
* gcse.c (compute_ld_motion_mems): Invalidate non-simple mem refs
in
On 05/03/2013 07:10 AM, Julian Brown wrote:
Hi,
This is a patch which fixes a latent bug in RTL GCSE/PRE, described
more fully in:
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57159
I haven't been able to reproduce the problem on mainline (nor on a
supported target). Maybe someone more
On 05/03/2013 06:48 PM, David Malcolm wrote:
+static int indent_amount = 0;
+static int had_recent_newline = 0;
Any clean way to do this without the global state? I have to ask given
you're generally working on removing global state. Seems to take a tiny
step backwards.
+
+static void
On Mon, May 06, 2013 at 12:23:07PM -0400, Jason Merrill wrote:
On 05/06/2013 11:04 AM, Jakub Jelinek wrote:
isn't it too invasive to the C++ FE?
It is pretty invasive. Why not put the information in
cp_declarator, as you suggested in your earlier email? You could
even put it in the
Hi,
Here is a 2nd attempt at enabling libsanitizer on ARM.
Compared with the previous version, this patch is more intrusive to
workaround some limitations with qemu:
* qemu does not work well with threads, so I chose to disable
clone-test-1.c and rlimit-mmap-test-1.c when running on a simulator.
Hi!
We ICE on the following testcase, because DECL_INITIAL contains
COMPOUND_LITERAL_EXPR, which expr.c assumes has been gimplified, but
initializers aren't gimplified. Fixed thusly, bootstrapped/regtested on
x86_64-linux and i686-linux, ok for trunk/4.8?
2013-05-06 Jakub Jelinek
Hi!
While these two sanitizing modes don't conflict on the compiler side, the
runtimes are incompatible with each other (each assumes different memory
layout). We'd need a special runtime library that would handle both at the
same time, probably not worth the hassle.
Ok for trunk/4.8?
On Mon, Apr 29, 2013 at 01:42:55PM +0200, Jakub Jelinek wrote:
On Mon, Apr 29, 2013 at 03:39:58PM +0400, Maksim Kuznetsov wrote:
2013/4/29 Jakub Jelinek ja...@redhat.com:
Also, why are you handling just %{ and %}, and
not also %| ? I mean, if you want to print say {|} into assembly for
On Sun, May 05, 2013 at 03:45:17PM -0500, Bill Schmidt wrote:
2013-05-05 Bill Schmidt wschm...@linux.vnet.ibm.com
* gimple-ssa-strength-reduction.c (slsr_process_phi): Re-enable.
(find_candidates_in_block): Re-enable slsr_process_phi.
(create_phi_basis): Fix double
On 05/06/2013 02:25 PM, Jakub Jelinek wrote:
I did it that way because if I understand the code well, often
grokdeclarator/start_decl/start_function/grokmethod/grokfield
is called with a chain of declarators, not just a single one.
True, perhaps adding it to declspecs-attributes would be
On Mon, 2013-05-06 at 21:25 +0200, Jakub Jelinek wrote:
On Sun, May 05, 2013 at 03:45:17PM -0500, Bill Schmidt wrote:
2013-05-05 Bill Schmidt wschm...@linux.vnet.ibm.com
* gimple-ssa-strength-reduction.c (slsr_process_phi): Re-enable.
(find_candidates_in_block): Re-enable
On Mon, May 6, 2013 at 12:01 PM, Jakub Jelinek ja...@redhat.com wrote:
On Mon, Apr 29, 2013 at 01:42:55PM +0200, Jakub Jelinek wrote:
On Mon, Apr 29, 2013 at 03:39:58PM +0400, Maksim Kuznetsov wrote:
2013/4/29 Jakub Jelinek ja...@redhat.com:
Also, why are you handling just %{ and %}, and
Hello!
Attached patch merges *sse4_1_pextrq and *sse4_1_pextrd with base
vec_select patterns. The patch splits instruction with zero selector
to plain movdi and movsi patterns. Please note that pextr $0,...
should only be generated for !TARGET_INTER_UNIT_MOVES_FROM_VEC targets
when -msse4 is
Hi
Here is the patch to fix the unordered containers profile mode.
There are a number of enhancements/fixes:
- Use inheritance to play code on instantiation/destruction. This way
some constructors and assignment operators can be defaulted like in the
normal implementation and then be
I checked it in for him with a small change in document. It
should be `|' instead of '@|'.
Thanks a lot!
--
Maxim Kuznetsov
On Mon, 2013-05-06 at 12:22 -0600, Jeff Law wrote:
On 05/03/2013 06:48 PM, David Malcolm wrote:
+static int indent_amount = 0;
+static int had_recent_newline = 0;
Any clean way to do this without the global state? I have to ask given
you're generally working on removing global state.
On Mon, 2013-05-06 at 17:05 -0400, David Malcolm wrote:
[...snip...]
Thanks for review; see attached rewrite of patch.
Sorry, I messed up the macro for generating the ChangeLog in the last
patch; version with fixed ChangeLog attached.
commit 566185b94449a90dc3213ea11643bfb3924a751d
Author: David
* Uses register bank 3 instead of register bank 0 inside the
handler.
I wonder if nested interrupts will cause problems, since the second
interrupt will occur with RB3 already selected.
Also, there are some hand-coded routines in libgloss/libgcc that
manually swap banks, they'd have
This fixes the following bug:
On Mon, 2013-05-06 at 21:25 +0200, Jakub Jelinek wrote:
On Sun, May 05, 2013 at 03:45:17PM -0500, Bill Schmidt wrote:
2013-05-05 Bill Schmidt wschm...@linux.vnet.ibm.com
* gimple-ssa-strength-reduction.c (slsr_process_phi): Re-enable.
On Mon, May 6, 2013 at 12:29 PM, Bill Schmidt
wschm...@linux.vnet.ibm.com wrote:
On Mon, 2013-05-06 at 21:25 +0200, Jakub Jelinek wrote:
On Sun, May 05, 2013 at 03:45:17PM -0500, Bill Schmidt wrote:
2013-05-05 Bill Schmidt wschm...@linux.vnet.ibm.com
* gimple-ssa-strength-reduction.c
On 06/05/2013 17:41, Jason Merrill wrote:
On 05/05/2013 09:57 AM, Dave Korn wrote:
This sounds like a bad idea to me, and not just because the locking
mechanism is dodgy. Is the problem more widespread than just your laptop?
Does it affect other host OSs? Linking multiple frontends at once
Applying under the obvious rule.
2013-05-07 Anton Blanchard an...@samba.org
* configure.ac (HAVE_LD_LARGE_TOC): Use correct linker emulation
for powerpc64 little endian.
* configure: Regenerate.
diff --git a/gcc/configure.ac b/gcc/configure.ac
index
After the refactoring has been checked in, the bug fixing part is simply
a moving a function call.
Tested by running ./buildit with both x86-64 and power64 targets.
The last time regression of tls-tests.c disappeared. So it is really flaky
in our testing environment.
thanks
Carrot
2013-05-02
On Mon, 2013-05-06 at 17:28 -0700, H.J. Lu wrote:
On Mon, May 6, 2013 at 12:29 PM, Bill Schmidt
wschm...@linux.vnet.ibm.com wrote:
On Mon, 2013-05-06 at 21:25 +0200, Jakub Jelinek wrote:
On Sun, May 05, 2013 at 03:45:17PM -0500, Bill Schmidt wrote:
2013-05-05 Bill Schmidt
The newlib support for Nios II ELF has just been approved and checked
in. However, there's no libgloss support; it's assumed that folks will
link with a BSP (like Altera's HAL) to provide the target-specific
startup code and I/O stubs. So, this patch adds libgloss to the
top-level
ok.
David
On Mon, May 6, 2013 at 6:03 PM, Carrot Wei car...@google.com wrote:
After the refactoring has been checked in, the bug fixing part is simply
a moving a function call.
Tested by running ./buildit with both x86-64 and power64 targets.
The last time regression of tls-tests.c
Most build_* functions have variadic versions, but build_constructor
doesn't as of yet, and it would be convenient for a patch I'm working
on. I decided to call it build_constructor_va, but am open to other
naming ideas.
Tested x86_64-pc-linux-gnu. OK for trunk?
commit
2013/5/6 Jeff Law l...@redhat.com:
On 05/03/2013 06:48 PM, David Malcolm wrote:
+static int indent_amount = 0;
+static int had_recent_newline = 0;
Any clean way to do this without the global state? I have to ask given
you're generally working on removing global state. Seems to take a
56 matches
Mail list logo
