On 20-11-14 12:54, Tom de Vries wrote:
Richard,
This patch fixes PR63718, which currently breaks Thumb1 bootstrap.
The problem is that in Thumb1 mode, we emit the epilogue in RTL, but the last
insn - epilogue_insns - does not accurately model the corresponding insns
emitted in the asm file.
On Mon, 1 Dec 2014, Marc Glisse wrote:
On Mon, 1 Dec 2014, Richard Biener wrote:
The following fixes PR64126.
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied.
Richard.
2014-12-01 Richard Biener rguent...@suse.de
PR middle-end/64126
* match.pd:
On Mon, 1 Dec 2014, Richard Biener wrote:
On Mon, 1 Dec 2014, FX wrote:
Your change is OK (we don’t want to use the type of the result, but the
type of the argument indeed).
Index: gcc/fortran/trans-intrinsic.c
On Mon, 1 Dec 2014, Joseph Myers wrote:
On Mon, 1 Dec 2014, Richard Biener wrote:
+/* Combine two successive divisions. */
+(for div (trunc_div ceil_div floor_div round_div exact_div)
This doesn't seem correct for all kinds of division and signedness of
arguments.
TRUNC_DIV_EXPR
involving INT_MIN from overflow handling.
* gcc.dg/torture/20141202-1.c: New testcase.
Index: gcc/match.pd
===
--- gcc/match.pd(revision 218260)
+++ gcc/match.pd(working copy)
@@ -140,7 +140,9
On Tue, 2 Dec 2014, Jakub Jelinek wrote:
Hi!
Here is an attempt to adjust gimple_build_assign* documentation.
Ok for trunk?
Ok.
Thanks,
Richard.
Note, apparently the documentation has not been adjusted for
the gimple - gassign * etc. changes, David, can you please work on
adjusting
2014-12-02 Richard Biener rguent...@suse.de
Merge from trunk r217561 through r218261.
Hello!
As mentioned in the PR, we need accurate gp live analysis to split
__tls_get_addr call. Move split from post-reload splitter to peephole2
pass, as is the case with other call insns.
2014-12-02 Uros Bizjak ubiz...@gmail.com
PR target/64113
* config/alpha/alpha.md
On Mon, Dec 1, 2014 at 10:06 PM, Jeff Law l...@redhat.com wrote:
On 11/25/14 14:16, Sebastian Pop wrote:
Sebastian Pop wrote:
I will bootstrap and regression test this patch on x86_64-linux and
powerpc64-linux. I will also run it on our internal benchmarks,
coremark, and
the llvm
On Mon, Dec 1, 2014 at 11:48 PM, Oleg Endo oleg.e...@t-online.de wrote:
On Mon, 2014-12-01 at 12:09 +0100, Richard Biener wrote:
On Mon, Dec 1, 2014 at 8:21 AM, Oleg Endo oleg.e...@t-online.de wrote:
Hi,
When running the testsuite on a sh-elf configuration, some test cases
fail due to
Hello Tobias,
a long time ago, I posted this patch, but it got forgotten.
However the described problem is still unsolved,
so I thought my patch should be re-posted now.
Boot-strapped and regression-tested on arm-linux-gnueabihf.
OK for trunk?
Thanks
Bernd.
On Tue, 15 Apr 2014 13:49:37,
On 01/12/14 22:42 +0100, François Dumont wrote:
Hi
Here is another proposal that consider all your remarks except
one. I finally prefer to go with std::vector of pointers. Dynamically
allocating Catalog_info allow to avoid numerous copies of locale when
we find this pointer from the
On Tue, 2014-12-02 at 11:22 +0100, Richard Biener wrote:
On Mon, Dec 1, 2014 at 11:48 PM, Oleg Endo oleg.e...@t-online.de wrote:
On Mon, 2014-12-01 at 12:09 +0100, Richard Biener wrote:
On Mon, Dec 1, 2014 at 8:21 AM, Oleg Endo oleg.e...@t-online.de wrote:
Hi,
When running the
On 05 Nov 11:00, Uros Bizjak wrote:
On Wed, Nov 5, 2014 at 10:35 AM, Ilya Enkovich enkovich@gmail.com wrote:
Hi,
Having stage1 close to end, may we make some decision regarding this
patch? Having a couple of working variants, may we choose and use one
of them?
I propose to wait
On Tue, Dec 2, 2014 at 12:08 PM, Ilya Enkovich enkovich@gmail.com wrote:
Having stage1 close to end, may we make some decision regarding this
patch? Having a couple of working variants, may we choose and use one
of them?
I propose to wait for Vlad for an update about his plans on
Jeff Law wrote:
OK with the appropropriate ChangeLog entires. THe original for
ira-costs.c was fine, so you just need the trivial one for the testcase.
ChangeLog below - Jiong, could you commit for me please?
2014-12-02 Wilco Dijkstra wdijk...@arm.com
* gcc/ira-costs.c
On 20/11/14 11:54, Tom de Vries wrote:
Richard,
This patch fixes PR63718, which currently breaks Thumb1 bootstrap.
The problem is that in Thumb1 mode, we emit the epilogue in RTL, but the last
insn - epilogue_insns - does not accurately model the corresponding insns
emitted in the asm file.
On 29/11/14 06:50, Chen Shanyao wrote:
I've backported this fix to 4.8 4.9 branch.
These patches have been tested for armeb-none-eabi-gcc/g++ with qemu,
and both the test results were ok.
The Changelog should mention all authors of the original patches i.e.
include my name.
Otherwise
On 21/11/2014 14:08, Alan Hayward alan.hayw...@arm.com wrote:
On 14/11/2014 16:48, Alan Hayward alan.hayw...@arm.com wrote:
This is a new version of my BE patch from a few weeks ago.
This is part 1 and covers rtlanal.c. The second part will be aarch64
specific.
When combined with the second
On Fri, Nov 21, 2014 at 10:59 AM, Martin Jambor mjam...@suse.cz wrote:
Hi,
PR 63814 is caused by cgraph_edge_brings_value_p misidentifying an
edge to an expanded artificial thunk as an edge to the original node,
which then leads to crazy double-cloning and doubling the thunks along
the call.
Hello,
as discussed in the PR, this patch removes an invalid ENABLE_CHECKING
sanity check in rs6000_delegitimize_address, fixing the ICE.
Tested on powerpc64-linux.
OK for mainline / 4.9 / 4.8?
Bye,
Ulrich
ChangeLog:
PR target/64115
* config/rs6000/rs6000.c
On 2 December 2014 at 07:50, Tobias Burnus bur...@net-b.de wrote:
Hi Manuel,
Manuel López-Ibáñez wrote:
This patch actually does not touch linemap but I will appreciate
Dodji's comments about the approach. The problem is that in case of
long lines, the column hint of 120 might be too small,
On Mon, 2014-12-01 at 14:31 -0700, Jeff Law wrote:
On 11/25/14 15:57, Mark Wielaard wrote:
This implements the DWARFv5 noreturn proposal:
http://dwarfstd.org/ShowIssue.php?issue=140331.1
TREE_THIS_VOLATILE on a FUNCTION_DECL node means the function does not
return normally. This catches
On Sat, Nov 29, 2014 at 3:00 PM, H.J. Lu hjl.to...@gmail.com wrote:
When searching for an usable algorithm with -minline-all-stringops,
decide_alg stops when it sees libcall even if there is a usable
algorithm. It goes into an infinite loop. This patch changes
decide_alg to stop searching
Update the docs. Committed to trunk.
commit 388d7943d85f505381f6e818d2de07978d022498
Author: Jonathan Wakely jwak...@redhat.com
Date: Tue Dec 2 13:13:33 2014 +
* doc/xml/manual/status_cxx2011.xml: Update.
* doc/xml/manual/status_cxx2014.xml: Update.
*
On Mon, Dec 1, 2014 at 4:53 PM, Yuri Rumyantsev ysrum...@gmail.com wrote:
Hi Richard,
I resend you patch1 and patch2 with minor changes:
1. I renamed flag_force_vectorize to aggressive_if_conv.
2. Use static cast for the first argument of gimple_phi_arg_edge.
I also very sorry that I sent
Manuel López-Ibáñez lopeziba...@gmail.com writes:
+/* Assert that becomes a conditional expression when not checking.
For the sake of clarity towards newcomers, I'd say:
Assert that becomes a conditional expression when checking is
disabled at compilation time.
+ Use this for
Hi,
Currently bndret is generated each time we need to get bounds for returned
pointer. It causes bndret generated for not instrumented calls incuding
builtin function calls. Troubles appear when such builtin call is optimized
out - bndret needs to be handled appropriately. Since we don't
Hi,
For strchr calls bounds of the first argument are considered as returned which
is wrong because NULL may be returned. This patch fixes that. Bootstrapped
and tested on x86_64-unknown-linux-gnu. OK for trunk?
Thanks,
Ilya
--
2014-12-02 Ilya Enkovich ilya.enkov...@intel.com
*
On Tue, Dec 2, 2014 at 7:56 AM, Ulrich Weigand uweig...@de.ibm.com wrote:
Hello,
as discussed in the PR, this patch removes an invalid ENABLE_CHECKING
sanity check in rs6000_delegitimize_address, fixing the ICE.
Tested on powerpc64-linux.
OK for mainline / 4.9 / 4.8?
Bye,
Ulrich
On Tue, Dec 2, 2014 at 3:34 AM, Wilco Dijkstra wdijk...@arm.com wrote:
Jeff Law wrote:
OK with the appropropriate ChangeLog entires. THe original for
ira-costs.c was fine, so you just need the trivial one for the testcase.
ChangeLog below - Jiong, could you commit for me please?
2014-12-02
Hi,
Currently cgraph nodes merge may break instrumented_version references. It
depends on order nodes are read and merged and therefore problem is not nicely
reproducible. I couldn't make a small reproducer. This patch fixes problem
and 253.perlbmk benchmark build with '-O3 -flto
On Tue, Dec 2, 2014 at 5:51 AM, H.J. Lu hjl.to...@gmail.com wrote:
On Tue, Dec 2, 2014 at 3:34 AM, Wilco Dijkstra wdijk...@arm.com wrote:
Jeff Law wrote:
OK with the appropropriate ChangeLog entires. THe original for
ira-costs.c was fine, so you just need the trivial one for the testcase.
On 23/09/14 09:27, James Greenhalgh wrote:
On Mon, Sep 15, 2014 at 11:56:03AM +0100, Andrew Stubbs wrote:
On 15/09/14 10:46, Richard Earnshaw wrote:
Hmm, I wonder if arm_override_options should reject neon + (arch 7).
Is this more to your taste?
Is this really such a good idea? It causes
On Tue, Dec 2, 2014 at 5:25 AM, Uros Bizjak ubiz...@gmail.com wrote:
On Sat, Nov 29, 2014 at 3:00 PM, H.J. Lu hjl.to...@gmail.com wrote:
When searching for an usable algorithm with -minline-all-stringops,
decide_alg stops when it sees libcall even if there is a usable
algorithm. It goes into
On Tue, Dec 2, 2014 at 3:03 PM, H.J. Lu hjl.to...@gmail.com wrote:
On Tue, Dec 2, 2014 at 5:25 AM, Uros Bizjak ubiz...@gmail.com wrote:
On Sat, Nov 29, 2014 at 3:00 PM, H.J. Lu hjl.to...@gmail.com wrote:
When searching for an usable algorithm with -minline-all-stringops,
decide_alg stops when
+
+ PR target/64108
+ * gcc.target/i386/memset-strategy-2.c: New test.
+
2014-12-02 Richard Biener rguent...@suse.de
* gcc.dg/torture/20141202-1.c: New testcase.
diff --git a/gcc/testsuite/gcc.target/i386/memset-strategy-2.c
b/gcc/testsuite/gcc.target/i386/memset-strategy-2.c
new file mode 100644
On Fri, Nov 28, 2014 at 09:28:03AM -0500, Jason Merrill wrote:
I was thinking even more detailed: one diagnostic for negative count, one
diagnostic for count larger than the precision of the lhs, and then a third
for overflow.
Alright, done.
+ /* For signed x y the following:
+
Hello Manuel, Tobias,
Manuel López-Ibáñez lopeziba...@gmail.com writes:
This patch actually does not touch linemap but I will appreciate
Dodji's comments about the approach.
Thanks :-)
The problem is that in case of long lines, the column hint of 120
might be too small, thus we do not have
Hi,
Currently symbol names privatization doesn't work for instrumentation clones
because clones assembler name is transparent alias and therefore alias target
should be privatized instead. This patch does it. Bootstrapped and tested on
x86_64-unknown-linux-gnu. OK for trunk?
Thanks,
Ilya
This is a (tiny) piece of fixing PR14541 which basically complains
that most math function simplification doesn't happen on GIMPLE.
This is of course because nobody re-builds the nested GENERIC
call expressions that would be necessary to trigger the various
simplifications implemneted in
On 12/02/2014 09:14 AM, Marek Polacek wrote:
On Fri, Nov 28, 2014 at 09:28:03AM -0500, Jason Merrill wrote:
I was thinking even more detailed: one diagnostic for negative count, one
diagnostic for count larger than the precision of the lhs, and then a third
for overflow.
Alright, done.
Thanks Richard for your quick reply!
1. I agree that we can combine predicate_extended_ and
predicate_arbitrary_ to one function as you proposed.
2. What is your opinion about using more simple decision about
insertion point - if bb has use of phi result insert phi predication
before it and at
On Tue, Dec 02, 2014 at 09:59:26AM -0500, Jason Merrill wrote:
Thanks. These errors also need to be conditional on (!ctx-quiet), and if
we see one of these conditions we need to set *non_constant_p.
Ah, sorry. Fixed in the following.
Regtested on ppc64-linux, bootstrap in progress.
Backport these header changes to the 4.9 branch. No regression, but reported to
the Debian BTS. Safe to backport according to Jonathan Wakely. No regressions
in the testsuite on x86_64-linux-gnu. Ok for the branch?
libstdc++-v3/
2014-12-02 Matthias Klose d...@ubuntu.com
PR
On 02/12/14 16:24 +, Jonathan Wakely wrote:
On 02/12/14 17:20 +0100, Matthias Klose wrote:
Backport these header changes to the 4.9 branch. No regression, but
reported to the Debian BTS. Safe to backport according to Jonathan
Wakely. No regressions in the testsuite on x86_64-linux-gnu. Ok
Hi all,
Back in July this deprecated option was removed.
This patch adds a note to changes.html for that.
Looked at the result in firefox, looked ok to me.
Ok to commit?
Thanks,
Kyrill
Index: htdocs/gcc-5/changes.html
===
RCS
On 02/12/14 17:20 +0100, Matthias Klose wrote:
Backport these header changes to the 4.9 branch. No regression, but
reported to the Debian BTS. Safe to backport according to Jonathan
Wakely. No regressions in the testsuite on x86_64-linux-gnu. Ok for
the branch?
You might as well include the
On Wed, 2014-11-26 at 10:41 -0700, Jeff Law wrote:
On 11/25/14 18:39, David Malcolm wrote:
I suspect this is papering over a real problem, but I've been
applying this workaround locally to keep my valgrind output clean.
gcc/ChangeLog:
PR/64003
* final.c (shorten_branches):
On 27/11/14 11:27, Renlin Li wrote:
Hi all,
We have the following code in aarch64_override_options() function.
/* The selected cpu may be an architecture, so lookup tuning by core
ID. */
if (!selected_tune)
selected_tune = all_cores[selected_cpu-core];
However, the logic here is
OK, thanks.
Jason
Hello!
2014-12-02 Uros Bizjak ubiz...@gmail.com
* gcc.target/i386/avx512ifma-vpmaddhuq-2.c: Define AVX512IFMA.
* gcc.target/i386/avx512ifma-vpmaddluq-2.c: Ditto.
* gcc.target/i386/avx512vbmi-vpermb-2.c: Define AVX512VBMI.
* gcc.target/i386/avx512vbmi-vpermi2b-2.c: Ditto.
*
Hi,
Underlying pthread_setspecific can return non-zero with ENOMEM or EINVAL.
2014-12-02 Ryan Mansfield rmansfi...@qnx.com
* emutls.c (__emutls_get_address): Check __gthread_setspecific
returns.
OK?
Regards,
Ryan Mansfield
Index: libgcc/emutls.c
Hi,
another simple issue: this one argues that the warning is overeager when
anonymous enums are involved, which often are just used as named
constants. Details: maybe write the conditional in a different way;
maybe even use instead of ||. Tested x86_64-linux.
Thanks,
Paolo.
Hi,
The following patch adds support for instrumentation of globals for
Linux kernel (-fsanitize=kernel-address). Kernel only supports
constructors with default priority, but the rest works fine.
OK for trunk?
https://codereview.appspot.com/176570043
Index: gcc/ChangeLog
Ping.
On Mon, Nov 10, 2014 at 3:22 PM, Sriraman Tallam tmsri...@google.com wrote:
Ping.
On Mon, Oct 6, 2014 at 1:43 PM, Sriraman Tallam tmsri...@google.com wrote:
Ping.
On Mon, Sep 29, 2014 at 10:57 AM, Sriraman Tallam tmsri...@google.com
wrote:
Ping.
On Fri, Sep 19, 2014 at 2:11 PM,
On Tue, Dec 02, 2014 at 09:56:36PM +0400, Dmitry Vyukov wrote:
--- gcc/ChangeLog (revision 218280)
+++ gcc/ChangeLog (working copy)
@@ -1,3 +1,8 @@
+2014-12-02 Dmitry Vyukov dvyu...@google.com
+
+ * asan.c: (asan_finish_file): Use default priority for constructors
+ in kernel mode.
Seems
On Tue, Dec 2, 2014 at 9:04 PM, Jakub Jelinek ja...@redhat.com wrote:
On Tue, Dec 02, 2014 at 09:56:36PM +0400, Dmitry Vyukov wrote:
--- gcc/ChangeLog (revision 218280)
+++ gcc/ChangeLog (working copy)
@@ -1,3 +1,8 @@
+2014-12-02 Dmitry Vyukov dvyu...@google.com
+
+ * asan.c:
Hi, this patch adds a new plugin event PLUGIN_START_PARSE_FUNCTION and
PLUGIN_FINISH_PARSE_FUNCTION that are invoked at start_function and
finish_function respectively in the C and C++ frontends.
PLUGIN_START_PARSE_FUNCTION is called before parsing the function body.
On Tue, Dec 02, 2014 at 10:12:30PM +0400, Dmitry Vyukov wrote:
Shall we backport it to 4.9 branch too?
If it's doable, it would be nice. Thanks.
Bet the same patch will just apply there.
When do we expect next 4.9 release?
Probably in March/April timeframe or so, 4.9.2 has been released
On Nov 30, 2014, at 11:21 PM, Oleg Endo oleg.e...@t-online.de wrote:
When running the testsuite on a sh-elf configuration, some test cases
fail due to multiple definitions of the function '_init'. This is
because on sh-elf every function is automatically prefixed with a '_'
char. When
On Tue, Dec 2, 2014 at 9:17 PM, Jakub Jelinek ja...@redhat.com wrote:
On Tue, Dec 02, 2014 at 10:12:30PM +0400, Dmitry Vyukov wrote:
Shall we backport it to 4.9 branch too?
If it's doable, it would be nice. Thanks.
Bet the same patch will just apply there.
Do I need to do anything
On Tue, 2 Dec 2014, Richard Biener wrote:
I'm not sure if these forms of division actually occur in places where
this could cause a problem, but it does look like Ada may enable you to
generate ROUND_DIV_EXPR.
Hmm. I thought I was following what extract_muldiv_1 does (but of course
On Mon, Sep 8, 2014 at 3:19 PM, Sriraman Tallam tmsri...@google.com wrote:
On Tue, Sep 2, 2014 at 1:40 PM, Richard Henderson r...@redhat.com wrote:
On 06/20/2014 05:17 PM, Sriraman Tallam wrote:
Index: config/i386/i386.c
===
---
On Tue, 2 Dec 2014, Joseph Myers wrote:
(y), 1)) is not the right condition for rounding away). The following Ada
testcase test_round_div.adb will generate a ROUND_DIV_EXPR which is
I should add that I'm not sure if Ada requires correct rounding for
fixed-point division converted to
Hello!
Ping.
Ping.
Ping.
Ping.
It would probably help reviewers if you pointed to actual path
submission [1], which unfortunately contains the explanation in the
patch itself [2], which further explains that this functionality is
currently only supported with gold, patched with [3].
[1]
On Tue, Dec 2, 2014 at 11:19 AM, Uros Bizjak ubiz...@gmail.com wrote:
Hello!
Ping.
Ping.
Ping.
Ping.
It would probably help reviewers if you pointed to actual path
submission [1], which unfortunately contains the explanation in the
patch itself [2], which further explains that this
On Tue, Dec 2, 2014 at 11:19 AM, Uros Bizjak ubiz...@gmail.com wrote:
Hello!
Ping.
Ping.
Ping.
Ping.
It would probably help reviewers if you pointed to actual path
submission [1], which unfortunately contains the explanation in the
patch itself [2], which further explains that this
On Tue, Dec 2, 2014 at 8:40 PM, H.J. Lu hjl.to...@gmail.com wrote:
On Tue, Dec 2, 2014 at 11:19 AM, Uros Bizjak ubiz...@gmail.com wrote:
Hello!
Ping.
Ping.
Ping.
Ping.
It would probably help reviewers if you pointed to actual path
submission [1], which unfortunately contains the
Hi,
thanks for the comments.
On Mon, Dec 01, 2014 at 11:40:59PM +0100, Jan Hubicka wrote:
2014-11-19 Martin Jambor mjam...@suse.cz
* ipa-prop.h (ipa_alignment): New type.
(ipa_jump_func): New field alignment.
(ipcp_transformation_summary) New type.
On 12/01/14 16:30, Oleg Endo wrote:
On Mon, 2014-12-01 at 10:38 -0700, Jeff Law wrote:
On 11/27/14 18:44, Segher Boessenkool wrote:
Currently reg_dead_at_p returns 0 for registers that are set but never
used. This patch repairs that oversight. This fixes PR59278.
2014-11-27 Segher
On 12/02/14 03:15, Richard Biener wrote:
I'm a bit worried about compile-time impacts of the all the recursion, but
I'm willing to wait and see if it turns out to be a problem in practice.
Please consider restricting it to -fexpensive-optimizations (-O2+).
Yea, let's go ahead and do that.
On Tue, 2014-12-02 at 08:49 +0100, Jakub Jelinek wrote:
Hi!
Here is an attempt to adjust gimple_build_assign* documentation.
Ok for trunk?
Note, apparently the documentation has not been adjusted for
the gimple - gassign * etc. changes, David, can you please work on
adjusting gimple.texi
On Tue, Dec 02, 2014 at 12:16:09PM -0800, H.J. Lu wrote:
IMO, nobody will use this highly special x86_64-only option. It would
be best for gnu-ld to reach feature parity with gold as far as this
functionality is concerned. In this case, the optimization would be
auto-configured, and would
Hi,
On Mon, Dec 01, 2014 at 10:43:19PM +0100, Jan Hubicka wrote:
On Fri, Nov 21, 2014 at 08:18:12PM +0100, Martin Jambor wrote:
Hi,
when debugging PR 63814 I noticed that when cgraph_node::create_clone
was using redirect_edge_duplicating_thunks to redirect two edges to a
thunk
Hi,
apparently it is necessary to check that type sizes match before
attempting to fold-V_C_E them in evaluate_conditions_for_known_args.
So this patch does this.
It passes bootstrap and testing on x86_64-linux and I have verified
with a cross compiler that the reported bug is fixed (the
On 12/02/14 09:20, David Malcolm wrote:
In short, I believe the problem occurs with a *jcc_1 insn that jumps
forwards, but the full details are in the bug.
My first thought is that something must be creating a new insn after
shorten_branches is complete or an existing insn that was not on the
My memories of why I did MeP the way I did are... vague. I recall it
had to do with getting the attributes to apply to C++ objects
correctly, since C++ objects tend to be complicated and gcc didn't
always pass me what I expected.
think they are suppose to. They build, but I cant test them...
On Tue, Dec 2, 2014 at 12:01 PM, Uros Bizjak ubiz...@gmail.com wrote:
On Tue, Dec 2, 2014 at 8:40 PM, H.J. Lu hjl.to...@gmail.com wrote:
On Tue, Dec 2, 2014 at 11:19 AM, Uros Bizjak ubiz...@gmail.com wrote:
Hello!
Ping.
Ping.
Ping.
Ping.
It would probably help reviewers if you pointed to
Hi,
apparently it is necessary to check that type sizes match before
attempting to fold-V_C_E them in evaluate_conditions_for_known_args.
So this patch does this.
It passes bootstrap and testing on x86_64-linux and I have verified
with a cross compiler that the reported bug is fixed (the
The attached patch fixes the regression in the
gcc/testsuite/gcc.dg/graphite/isl-codegen-loop-dumping.c testcase
caused by the accidental removal of -fgraphite-identity from
dg-options at r217315. Okay for gcc trunk?
Jack
2014-12-01 Jack Howarth howa...@bromo.med.uc.edu
Looks good to me. Thank you.
2014-12-03 2:09 GMT+05:00 Jack Howarth howarth.at@gmail.com:
The attached patch fixes the regression in the
gcc/testsuite/gcc.dg/graphite/isl-codegen-loop-dumping.c testcase
caused by the accidental removal of -fgraphite-identity from
dg-options at
On 12/02/14 09:20, David Malcolm wrote:
I've spent some time trying to track this down, and I've added detailed
notes to the bug.
In short, I believe the problem occurs with a *jcc_1 insn that jumps
forwards, but the full details are in the bug.
My first thought is that something must be
On Tuesday 2014-12-02 16:34, Kyrill Tkachov wrote:
Back in July this deprecated option was removed.
This patch adds a note to changes.html for that.
Looked at the result in firefox, looked ok to me.
Ok to commit?
Yep.
And I very comfortable for you to commit changes like this
without asking
On Tue, Dec 2, 2014 at 2:01 PM, Kyrill Tkachov kyrylo.tkac...@arm.com wrote:
On 23/09/14 09:27, James Greenhalgh wrote:
On Mon, Sep 15, 2014 at 11:56:03AM +0100, Andrew Stubbs wrote:
On 15/09/14 10:46, Richard Earnshaw wrote:
Hmm, I wonder if arm_override_options should reject neon + (arch
On 12/01/14 14:02, Cary Coutant wrote:
[+cc Michael Eager]
Rather than having to lobby to keep it unchanged because we jumped the gun,
can we lobby to get the number assigned in the near future rather than in
the potentially far future? That feels more cooperative to me :-)
Would that make
On Tue, 2 Dec 2014, Richard Biener wrote:
Joseph may have more experience with how targets should setup
USER_LABEL_PREFIX to avoid this situation.
See e.g. config/arm/lib1funcs.S:
#define SYM(x) CONCAT1 (__USER_LABEL_PREFIX__, x)
(and the associated macro definition of CONCAT1 that uses,
See my comment on the PR; I think the testcase illustrates why we still
want this warning for anonymous enums.
Jason
On Tue, Dec 2, 2014 at 12:19 PM, Jakub Jelinek ja...@redhat.com wrote:
On Tue, Dec 02, 2014 at 12:16:09PM -0800, H.J. Lu wrote:
IMO, nobody will use this highly special x86_64-only option. It would
be best for gnu-ld to reach feature parity with gold as far as this
functionality is
Hi,
On 12/02/2014 11:01 PM, Jason Merrill wrote:
See my comment on the PR; I think the testcase illustrates why we
still want this warning for anonymous enums.
Ok... Thus, barring further discussion, I will simply close the Bug as
invalid.
Thanks!
Paolo.
On Fri, Nov 21, 2014 at 6:44 PM, Philipp Tomsich
philipp.toms...@theobroma-systems.com wrote:
The following patch-series adds optimized support for the APM X-Gene 1
by providing a cost-model and pipeline-model. The pipeline-model has a
few long reservation-chains, but looking at the stats for
On Fri, Nov 21, 2014 at 6:44 PM, Philipp Tomsich
philipp.toms...@theobroma-systems.com wrote:
To keep this change separately buildable from the pipeline model,
this patch directs the APM XGene-1 to use the generic scheduling
model.
---
gcc/ChangeLog| 8 +++
On Fri, Nov 21, 2014 at 6:44 PM, Philipp Tomsich
philipp.toms...@theobroma-systems.com wrote:
---
gcc/ChangeLog | 6 +
gcc/config/aarch64/aarch64.md | 3 +-
gcc/config/arm/xgene1.md | 532
++
3 files changed, 540
CCing release maintainers as well as they need to approve this
backport if Vlad is happy with it.
Vlad - is this ok to go back as it fixes a bug for ARM in the 4.9 tree
that came up in building bits of debian.
Ramana
On Mon, Dec 1, 2014 at 5:24 PM, Renlin Li renlin...@arm.com wrote:
On
Committed as Rev. 218294, now that ISL 0.14 is supported by GCC 5 and
ISL 0.14 is in the infrastructure directory.
Tobias
Index: contrib/ChangeLog
===
--- contrib/ChangeLog (Revision 218293)
+++ contrib/ChangeLog (Arbeitskopie)
@@
So here is another proposal with all your remarks implemented.
2014-12-02 François Dumont fdum...@gcc.gnu.org
DR libstdc++/13631
* include/bits/codecvt.h (codecvtchar, char, mbstate_t): friend class
std::messageschar.
(codecvtwchar_t, char, mbstate_t): friend class
On Tue, Nov 11, 2014 at 11:55 AM, Kyrill Tkachov kyrylo.tkac...@arm.com wrote:
Hi all,
This is the arm implementation of the macro fusion hook.
It tries to fuse movw+movt operations together. It also tries to take lo_sum
RTXs into account since those generate movt instructions as well.
On Tue, 2 Dec 2014, Manuel López-Ibáñez wrote:
2014-12-02 Manuel López-Ibáñez m...@gcc.gnu.org
* diagnostic.c (diagnostic_color_init): New.
* diagnostic.h: Declare.
* gcc.c (driver::global_initializations): Use it.
(driver_handle_option): Handle -fdiagnostics-color_.
libcpp uses diagnostic-override_column to give a custom column number
to diagnostics. This is taken into account when building the prefix,
but it was missing when placing the caret.
Before:
/home/manuel/override_column.c:1:4: warning: /* within comment [-Wcomment]
/* /* */
^
After:
On Tue, Dec 2, 2014 at 2:14 PM, H.J. Lu hjl.to...@gmail.com wrote:
On Tue, Dec 2, 2014 at 12:19 PM, Jakub Jelinek ja...@redhat.com wrote:
On Tue, Dec 02, 2014 at 12:16:09PM -0800, H.J. Lu wrote:
IMO, nobody will use this highly special x86_64-only option. It would
be best for gnu-ld to reach
1 - 100 of 112 matches
Mail list logo