On 22 Oct 23:21, Ilya Verbin wrote:
On 22 Oct 10:54, Jakub Jelinek wrote:
On Tue, Oct 21, 2014 at 09:20:34PM +0400, Ilya Verbin wrote:
This patch contains liboffloadmic library.
It is used by ICC for offloading. The sources are imported from upstream
(
On 10/29/2014 03:07 PM, Ilya Enkovich wrote:
2014-10-29 17:01 GMT+03:00 Martin Liška mli...@suse.cz:
On 10/29/2014 02:45 PM, Ilya Enkovich wrote:
On 29 Oct 10:34, Richard Biener wrote:
On Tue, Oct 28, 2014 at 5:14 PM, Ilya Enkovich enkovich@gmail.com
wrote:
Hi,
This patch fixes
Hi all,
This fixes an arm build failure due to removing the 'enum' keyword from
machine_mode.
Since libgcc2 is compiled with C rather than C++ we need it there for
the definition of CUMULATIVE_ARGS.
Another place where machine_mode is used is machine_function which is
not needed for libgcc2
Ok. For the branch please wait until after 4.9.2 is out.
Thanks! Committed to trunk.
Hi all,
This is similar to the arm patch
(https://gcc.gnu.org/ml/gcc-patches/2014-10/msg03104.html),
it adds back the enum keyword to machine_mode in the definition of MACHMODE.
This allows the aarch64* build to succeed.
Ok for trunk?
Thanks,
Kyrill
2014-10-28 Kyrylo Tkachov
On 29 October 2014 17:31, Kyrill Tkachov kyrylo.tkac...@arm.com wrote:
Hi all,
This fixes an arm build failure due to removing the 'enum' keyword from
machine_mode.
Since libgcc2 is compiled with C rather than C++ we need it there for the
definition of CUMULATIVE_ARGS.
Another place where
I'm getting these test failures on m68k-linux:
FAIL: gcc.misc-tests/godump-1.c scan-file (?n)^type _tsbf_anon_pad1 struct { c
uint[0-9]*; }$
FAIL: gcc.misc-tests/godump-1.c scan-file (?n)^type _tsbf_anon_pad5 struct { c
uint[0-9]*; }$
FAIL: gcc.misc-tests/godump-1.c scan-file (?n)^type
On Wed, Oct 29, 2014 at 9:38 AM, Andreas Schwab sch...@suse.de wrote:
I'm getting these test failures on m68k-linux:
Can you send the file BUILDDIR/gcc/testsuite/gcc/godump-1.out?
Ian
On Tue, 28 Oct 2014, Martin Uecker wrote:
attached is a revised and extended patch. Changes with respect
to the previous patch are:
Thanks for the revised patch. I've moved this to gcc-patches as the more
appropriate mailing list for discussion of specific patches as opposed to
more
On 10/29/2014 05:47 AM, Thomas Preud'homme wrote:
From: Nathan Sidwell [mailto:nat...@codesourcery.com]
Sent: Thursday, October 09, 2014 2:30 PM
On 10/09/14 09:25, Jason Merrill wrote:
I would think we want to handle this up in the existing defaulted_int
block:
my thought was to at least put
= 5.0.0 20141029 (experimental)
const ___linux__ = 1
const ___SCHAR_MAX__ = 0x7f
const ___GLIBC_MINOR__ = 18
const ___ATOMIC_ACQ_REL = 4
const ___DEC64_MAX__ = 9.999E384
const _INT_FAST16_MIN = (-2147483647-1)
const __SVID_SOURCE = 1
const ___WORDSIZE = 32
const ___WCHAR_MIN__
On 29/10/14 16:34, Kyrill Tkachov wrote:
Hi all,
This is similar to the arm patch
(https://gcc.gnu.org/ml/gcc-patches/2014-10/msg03104.html),
it adds back the enum keyword to machine_mode in the definition of MACHMODE.
This allows the aarch64* build to succeed.
Ok for trunk?
I've committed
On Wed, Oct 29, 2014 at 4:31 PM, Kyrill Tkachov kyrylo.tkac...@arm.com wrote:
Hi all,
This fixes an arm build failure due to removing the 'enum' keyword from
machine_mode.
Since libgcc2 is compiled with C rather than C++ we need it there for the
definition of CUMULATIVE_ARGS.
Another place
On Sat, 25 Oct 2014, Marek Polacek wrote:
+ pp_c_ws_string (cpp, aka);
That should be _(aka), as it's an English word, not a C syntax
construct.
OK with that change.
--
Joseph S. Myers
jos...@codesourcery.com
On Wed, Oct 29, 2014 at 9:24 AM, Richard Biener
richard.guent...@gmail.com wrote:
Because only genmatch calls functions from libstdc++. Btw, why
would genmatch miscompile an empty function or the call to it?
I tried bootstrapping with libstdc++ built without the AIX ld -G
flag and that is
Thanks. Part of the problem is that the m68k max alignment is 16
bits, but the godump test expects it to be at least 64 bits. This is
BIGGEST_ALIGNMENT in config/m68k/m68k.h. Another part of the problem
seems to be that structs are sometimes aligned to 16 bits although
there is no obvious
Hi Julian!
On Tue, 28 Oct 2014 11:16:19 +, Julian Brown jul...@codesourcery.com
wrote:
This patch rationalises TLS support by moving all thread-local
variables into a single structure. Because this meant interfering with
how per-thread/per-device initialisation was done, I took the
On Wed, 29 Oct 2014, Kyrill Tkachov wrote:
Hi all,
This fixes an arm build failure due to removing the 'enum' keyword from
machine_mode.
Since libgcc2 is compiled with C rather than C++ we need it there for the
definition of CUMULATIVE_ARGS.
But why is CUMULATIVE_ARGS needed for libgcc?
The first patch adds the following from Library Fundamentals v1:
std::experimental::sample
std::experimental::search
std::experimental::default_searcher
std::experimental::make_default_searcher
std::experimental::boyer_moore_searcher
std::experimental::make_boyer_moore_searcher
As pointed out in http://stackoverflow.com/q/26543242/981959 our
std::function does more copying/moving than necessary. The solution is
to use perfect forwarding for the internal invoker functions, even if
functionR(Args...)::operator() and the target function take their
arguments by value.
gcc/ChangeLog.gimple-classes:
* gimple.h (gimple_omp_teams_clauses): Strengthen param from
const_gimple to const gomp_teams *.
(gimple_omp_teams_clauses_ptr): Strengthen param from gimple to
gomp_teams *.
* gimple-walk.c (walk_gimple_op): Add checked cast.
I've pushed the following ten patches to the git branch
dmalcolm/gimple-classes.
Successfully bootstrappedregrtested the combination of the ten
patches upon the branch on x86_64-unknown-linux-gnu (Fedora 20) -
same results relative to an unpatched control bootstrap of trunk's
r216746.
David
gcc/ChangeLog.gimple-classes:
* gimple.h (gimple_omp_task_clauses): Strengthen param from
const_gimple to const gomp_task *.
(gimple_omp_task_child_fn): Likewise.
(gimple_omp_task_data_arg): Likewise.
(gimple_omp_task_copy_fn): Likewise.
gcc/ChangeLog.gimple-classes:
* gimple.h (gimple_omp_target_clauses): Strengthen param from
const_gimple to const gomp_target *.
(gimple_omp_target_kind): Likewise.
(gimple_omp_target_clauses_ptr): Strengthen param from gimple to
gomp_target *.
*
gcc/ChangeLog.gimple-classes:
* gimple.h (gimple_eh_filter_types): Strengthen param from
const_gimple to const geh_filter *.
(gimple_eh_filter_types_ptr): Strengthen param from gimple to
geh_filter *.
(gimple_eh_filter_failure_ptr): Likewise.
gcc/ChangeLog.gimple-classes:
* gimple.h (gimple_omp_for_kind): Strengthen param from
const_gimple to const gomp_for *.
(gimple_omp_for_combined_p): Likewise.
(gimple_omp_for_combined_into_p): Likewise.
(gimple_omp_for_clauses): Likewise.
gcc/ChangeLog.gimple-classes:
* gimple.h (gimple_omp_sections_clauses): Strengthen param from
const_gimple to const gomp_sections *.
(gimple_omp_sections_control): Likewise.
(gimple_omp_sections_clauses_ptr): Strengthen param from gimple to
gomp_sections *.
gcc/ChangeLog.gimple-classes:
* gimple.h (gimple_omp_single_clauses): Strengthen param from
const_gimple to const gomp_single *.
(gimple_omp_single_clauses_ptr): Strengthen param from gimple to
gomp_single *.
* gimple-walk.c (walk_gimple_op): Add checked
gcc/ChangeLog.gimple-classes:
* gimple.h (struct gimple_statement_omp_return): Rename to...
(struct gomp_return): ...this.
(is_a_helper gimple_statement_omp_return *::test): Rename to...
(is_a_helper gomp_return *::test): ...this.
(is_a_helper const
Ping?
On Mon, Oct 20, 2014 at 10:31 AM, Cary Coutant ccout...@google.com wrote:
This patch adds support for the mips target in gold.
OK to commit?
-cary
2014-10-20 Cary Coutant ccout...@google.com
* configure (--enable-gold): Add mips*-*-*.
* configure.ac: Regenerate.
gcc/ChangeLog.gimple-classes:
* coretypes.h (struct gpredict): Add forward declaration.
* doc/gimple.texi (Class hierarchy of GIMPLE statements): Add
gpredict.
* gimple-pretty-print.c (pp_gimple_stmt_1): Within case
GIMPLE_PREDICT, add local predict_stmt via
gcc/ChangeLog.gimple-classes:
* gimple.h (gimple_omp_parallel_combined_p): Strengthen param from
const_gimple to const gomp_parallel *.
(gimple_omp_parallel_clauses): Likewise.
(gimple_omp_parallel_set_combined_p): Strengthen param from gimple
to
Hi!
On Tue, 28 Oct 2014 17:00:38 +0100, I wrote:
Committed in r216804:
commit 4f9566b3e2954218c0d9ce3c585e14e539f0c1af
Author: tschwinge tschwinge@138bc75d-0d04-0410-961f-82ee72b054a4
Date: Tue Oct 28 15:57:48 2014 +
libgomp: Don't refer to CUDA installation in
On Wed, Oct 29, 2014 at 1:28 PM, Evgeny Stupachenko evstu...@gmail.com wrote:
The test passes now. So let's remove xfail.
2014-10-29 Evgeny Stupachenko evstu...@gmail.com
gcc/testsuite
* gcc.target/i386/pr23098.c: Remove xfail.
OK.
Thanks,
Uros.
On 29 October 2014 10:03, Kyrill Tkachov kyrylo.tkac...@arm.com wrote:
Hi all,
This patch fixes an issue with the final_prescan workaround for the
Cortex-A53 erratum 835769
where calling recog_memoized could modify the recog data for the
multiply-accumulate instruction
when looking at a
-Original Message-
From: Matthew Fortune [mailto:matthew.fort...@imgtec.com]
Sent: Tuesday, October 28, 2014 1:13 PM
To: Moore, Catherine; 'gcc-patches@gcc.gnu.org' (gcc-
patc...@gcc.gnu.org); Eric Christopher (echri...@gmail.com)
Cc: Richard Sandiford; Rich Fuhler; Rozycki,
On 29 October 2014 10:05, Kyrill Tkachov kyrylo.tkac...@arm.com wrote:
Hi all,
This is the backport of the trunk patch posted at
https://gcc.gnu.org/ml/gcc-patches/2014-10/msg03019.html.
It is essentially the same content (only the diff context differs).
Jakub, this is a regression fix so,
On 29 October 2014 10:05, Kyrill Tkachov kyrylo.tkac...@arm.com wrote:
Hi all,
This is the 4.8 backport of the trunk patch
(https://gcc.gnu.org/ml/gcc-patches/2014-10/msg03019.html).
Tested similarly.
Ok for that branch?
OK once the 4.9 fix is committed. /Marcus
On Wed, Oct 29, 2014 at 07:57:46PM +, Marcus Shawcroft wrote:
On 29 October 2014 10:05, Kyrill Tkachov kyrylo.tkac...@arm.com wrote:
Hi all,
This is the backport of the trunk patch posted at
https://gcc.gnu.org/ml/gcc-patches/2014-10/msg03019.html.
It is essentially the same content
On Wed, Oct 29, 2014 at 11:45:51AM +0100, Jakub Jelinek wrote:
On Wed, Oct 29, 2014 at 11:37:26AM +0100, Paolo Bonzini wrote:
On 10/29/2014 11:31 AM, Jakub Jelinek wrote:
shouldn't libcc1 be in build_tools instead?
I mean, it is a library meant to be dlopened by gdb and gcc
plugin that
On 10/29/14 08:32, Phil Muldoon wrote:
On 29/10/14 14:26, Phil Muldoon wrote:
On 29/10/14 11:24, Phil Muldoon wrote:
On 29/10/14 10:31, Jakub Jelinek wrote:
It would be nice to have libcc1 built just once, not bootstrap it, but
it is a build module, is that possible?
In toplevel configure.ac
On 10/29/14 10:28, Martin Liška wrote:
PR63574.changelog
gcc/testsuite/ChangeLog:
2014-10-29 Martin Liskamli...@suse.cz
* g++.dg/ipa/pr63574.C: New test.
gcc/ChangeLog:
2014-10-29 Martin Liskamli...@suse.cz
* ipa-icf-gimple.c (func_checker::compare_variable_decl):
On 10/29/14 07:21, Wilco Dijkstra wrote:
This patch adjusts the spill cost of literal pool loads to reduce the chance of
them being
caller-saved (which is inefficient). Such loads should be rematerialized and
thus should not include
the cost of a spill store. This was done only on constants
On 10/29/14 04:46, James Greenhalgh wrote:
Hi,
This patch moves s390 to TARGET_MOVE_BY_PIECES_PROFITABLE_P.
I tried building a compiler and there were no fires, but otherwise,
I have no reasonable way to test this patch. If one of the s390
maintainers wants to pick it up and test it, that
On 10/29/14 04:48, James Greenhalgh wrote:
Hi,
This patch moves arc to TARGET_MOVE_BY_PIECES_PROFITABLE_P.
While I am there, arc defines a macro CAN_MOVE_BY_PIECES, which is
unused, so clean that up too.
I tried building a compiler but no amount of fiddling with target
strings got me to a
On 10/29/14 04:50, James Greenhalgh wrote:
Hi,
This patch moves mips to TARGET_MOVE_BY_PIECES_PROFITABLE_P.
I tried building a compiler and there were no fires, I don't have access
to any MIPS hardware, so if one of the MIPS maintainers wanted to pick
this up and test it, that would be very
On 10/29/14 04:50, James Greenhalgh wrote:
Hi,
This final patch gets rid of MOVE_BY_PIECES_P.
Bootstrapped on x86_64, ARM and AArch64.
Thanks,
James
---
gcc/
2014-10-28 James Greenhalgh james.greenha...@arm.com
* doc/tm.texi.in (MOVE_BY_PIECES_P): Remove.
* doc/tm.texi:
On 10/29/14 04:49, James Greenhalgh wrote:
Hi,
This patch moves sh to TARGET_MOVE_BY_PIECES_PROFITABLE_P.
I tried building a compiler and there were no fires, but otherwise,
I have no reasonable way to test this patch. If one of the sh
maintainers wants to pick it up and test it, that would
On 10/28/14 10:38, Ilya Enkovich wrote:
Hi,
After recent merge with trunk I found that new IPA pass ICF requires few
modifications for instrumented code:
- instrumentation thunk existence means we cannot merge function into another
one and should generate thunk instead
- thunk generation
On 10/28/14 08:49, Bernd Schmidt wrote:
On 10/22/2014 08:12 PM, Jeff Law wrote:
Yea, let's keep your approach. Just wanted to explore a bit since the
PA seems to have a variety of similar characteristics.
Here's an updated version of the patch. I experimented a little with ptx
calling
On 10/28/14 08:56, Bernd Schmidt wrote:
I have patches that expose all the address spaces to the middle-end
through a lower-as pass that runs early. The preliminary patches for
that ran into some resistance and into general brokenness of our address
space support, so I decided to rip all that
On Wed, Oct 29, 2014 at 5:50 PM, Joseph S. Myers
jos...@codesourcery.com wrote:
On Wed, 29 Oct 2014, Kyrill Tkachov wrote:
Hi all,
This fixes an arm build failure due to removing the 'enum' keyword from
machine_mode.
Since libgcc2 is compiled with C rather than C++ we need it there for the
On 09/11/2014 03:24 PM, Bernd Schmidt wrote:
The nvptx backend is somewhat unusual in that call insns set a pseudo.
The combiner is surprised by this and allows combining them into other
insns, which remain as INSN rather than CALL_INSN. Aborts ensue.
Bootstrapped and tested on x86_64-linux,
On 09/11/2014 05:58 PM, Steven Bosscher wrote:
On Thu, Sep 11, 2014 at 3:27 PM, Bernd Schmidt wrote:
It turns out that we're calling eliminate_regs for global variables which
can't possibly have eliminable regs in their decl. At that point,
reg_eliminate can be NULL. This patch avoids
James Greenhalgh james.greenha...@arm.com wrote:
This patch moves sh to TARGET_MOVE_BY_PIECES_PROFITABLE_P.
I tried building a compiler and there were no fires, but otherwise,
I have no reasonable way to test this patch. If one of the sh
maintainers wants to pick it up and test it, that
On 10/30/2014 12:35 AM, Jeff Law wrote:
A nit -- Richard S. recently removed the need to include the enum
for enum machine_mode. I believe he had a script to handle the
mundane parts of that change. Please make sure to update the nvptx port
to conform to that new convention, obviously feel
On Tue, 2014-10-21 at 15:24 -0400, David Malcolm wrote:
On Tue, 2014-10-21 at 18:15 +0200, Gerald Pfeifer wrote:
On Monday 2014-10-20 15:19, David Malcolm wrote:
+@item Sphinx (any working version)
As opposed to any non-working version? ;-) I'd just omit
working from this.
FWIW,
On 10/29/14 17:55, Bernd Schmidt wrote:
Thanks! I've pinged some of the preliminary patches that went unapproved
up to this point.
Thanks.
One leftover issue, discussed in the [0/11] mail - what amount of
documentation is appropriate for this, given that we don't want to
support using this
On 09/11/14 07:24, Bernd Schmidt wrote:
The nvptx backend is somewhat unusual in that call insns set a pseudo.
The combiner is surprised by this and allows combining them into other
insns, which remain as INSN rather than CALL_INSN. Aborts ensue.
Bootstrapped and tested on x86_64-linux,
On 10/29/14 17:48, Bernd Schmidt wrote:
On 09/11/2014 05:58 PM, Steven Bosscher wrote:
On Thu, Sep 11, 2014 at 3:27 PM, Bernd Schmidt wrote:
It turns out that we're calling eliminate_regs for global variables
which
can't possibly have eliminable regs in their decl. At that point,
reg_eliminate
On 10/28/14 06:23, Jakub Jelinek wrote:
On Tue, Oct 28, 2014 at 11:47:31AM +, Phil Muldoon wrote:
I think I have a solution. Though my automake fu is very weak. Does
this patch work for you? I'm really not sure how to deal with the
three possible versions of libiberty any other way.
On 10/27/14 14:47, Jakub Jelinek wrote:
Hi!
The following testcase is miscompiled in 4.8+ on x86_64 at -O2+,
because REE widens for ZERO_EXTEND mode on
(set (reg:QI ax) (const_int -1)) instruction to SImode, but doesn't
adjust REG_EQUAL note of (const_int -1) also to (const_int 0xff)
like it
For the first time in over a decade, I'm going to poke a bit at the PA
port. Basically I want to use Bin's work to do insn fusion to generate
fmpyadd and fmpysub instructions on the PA -- doing so ought to answer
most of my questions about the implementation.
Ultimately that'll be
On 10/29/14 04:28, Jakub Jelinek wrote:
On Tue, Oct 28, 2014 at 05:36:50PM +, Phil Muldoon wrote:
On 28/10/14 13:19, Joseph S. Myers wrote:
I'm seeing a different bootstrap failure from those already discussed:
In file included from
On 10/26/14 19:53, Zhenqiang Chen wrote:
Hi,
Function noce_try_store_flag_mask converts if (test) x = 0; to x =
-(test == 0);
But from code size view, x = -(test == 0); might have more instructions
than if (test) x = 0;. The patch checks the cost to determine the
conversion is valuable or not.
On 10/15/14 11:54, Ilya Enkovich wrote:
Thanks for review! I see no more patches not reviewed at all. I see
4 more patches requiring approve before I can start a merge.
Two of them are parts of split #14 (Passes): #14.3 (Helper functions)
and #14.16 (Reduce bounds lifetime)
These are fine.
101 - 166 of 166 matches
Mail list logo