Hi!
Committed to gomp-4_0-branch in r230214:
commit 265f04668a5b7dece82a35e2d75e8b51a1d75b69
Merge: 5e71838 b656be3
Author: tschwinge
Date: Thu Nov 12 07:40:36 2015 +
svn merge -r 230082:230169 svn+ssh://gcc.gnu.org/svn/gcc/trunk
git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc
On 11 November 2015 at 09:50, Robert Suchanek
wrote:
> Hi,
>
>> I guess this is ok to stop the failures for now, but you may want to
>> move the check to the point where we set terminated_this_insn. Also, as
>> I pointed out earlier, clearing terminated_this_insn should probably
>> happen earlier.
Hi Richard,
Thanks for the review.
>>>
>>> The basic "structure" thing still remains. You walk over all uses and
>>> defs in all stmts
>>> in promote_all_stmts which ends up calling promote_ssa_if_not_promoted on
>>> all
>>> uses and defs which in turn promotes (the "def") and then fixes up all
Oh, and we could also be more informative and print the size of an array,
or the number of elements, as clang does.
Yes, that's pretty nice. It helps but the diagnostic must point at
the right dimension. GCC often just points at the whole expression
or some token within it.
void* foo ()
{
e
On 11/10/2015 08:30 AM, Wilco Dijkstra wrote:
Ping of https://gcc.gnu.org/ml/gcc-patches/2014-12/msg00829.html:
This fixes a bug in register preferencing. When live range splitting creates
a new register from
another, it copies most fields except for the register preferences. The
preference GE
On 11 November 2015 at 12:10, Kyrill Tkachov wrote:
>
> On 11/11/15 12:08, Charles Baylis wrote:
>>
>> On 11 November 2015 at 11:22, Kyrill Tkachov
>> wrote:
>>>
>>> Hi Charles,
>>>
>>> On 08/11/15 00:26, charles.bay...@linaro.org wrote:
From: Charles Baylis
Charles Baylis
Another GC problem I noticed while looking at something else: when we
freed a duplicate typedef, we were leaving its type in the variants
list, with its TYPE_NAME still pointing to the now-freed TYPE_DECL,
leading to a crash.
Tested x86_64-pc-linux-gnu, applying to trunk.
commit e36d7607f157b5
Jakub,
The attached patch and ChangeLog reflect the updates from your
review: https://gcc.gnu.org/ml/gcc-patches/2015-11/msg01317.html.
Highlights
The following issue was handled by Dominique d'Humières
in: https://gcc.gnu.org/ml/gcc-patches/2015-11/msg01375.html
On 11/11/2015 02:32 AM, Ja
While looking into something else I noticed a GC problem with lambdas:
it is possible to collect after instantiating the closure type, which
implies instantiating the operator(), while we're in the middle of an
expression, leading to corruption. We deal with this sort of thing in
mark_used by
Hi Ilya,
The patch committed as r230098 has caused a number of ICEs on
powerpc64le-linux-gnu. The gcc/ChangeLog entry for that patch is:
2015-11-10 Ilya Enkovich
* expr.c (do_store_flag): Use expand_vec_cmp_expr for mask
results.
(const_vector_mask_from_tree): New.
(c
Earlier, graphite used to translate portions of the original program after
scop-detection in order to represent the SCoP into polyhedral model. This was
required because each basic block was represented as independent basic block in
the polyhedral model. So all the cross-basic-block dependencies w
On 12 November 2015 at 00:18, Jonathan Wakely wrote:
> So I think we want to define them again, independently, in
> , even though it might lead to ambiguities
Here. Tested again on Linux-PPC64.
2015-11-11 Ville Voutilainen
Implement D0013R2, logical type traits.
/libstdc++-v3
*
> > On 11/11/2015 10:21 AM, Richard Biener wrote:
> > >On Tue, 10 Nov 2015, Jan Hubicka wrote:
> > >>The reason is that TYPE_CANONICAL is initialized in get_alias_set that
> > >>may be
> > >>called before we finish all merging and then it is more fine grained than
> > >>what
> > >>we need here (i
On 12/11/15 00:06 +0200, Ville Voutilainen wrote:
--- a/gcc/testsuite/g++.dg/cpp0x/Wattributes1.C
+++ b/gcc/testsuite/g++.dg/cpp0x/Wattributes1.C
@@ -5,4 +5,4 @@
#include
__attribute__((visibility("hidden")))void*operator new(std::size_t); // { dg-warning
"visibility attribute ignored" }
-// {
> On 11/11/2015 10:21 AM, Richard Biener wrote:
> >On Tue, 10 Nov 2015, Jan Hubicka wrote:
> >>The reason is that TYPE_CANONICAL is initialized in get_alias_set that may
> >>be
> >>called before we finish all merging and then it is more fine grained than
> >>what
> >>we need here (i.e. TYPE_CANON
On 11/11/15 22:53 +0100, John Marino wrote:
On 11/11/2015 10:51 PM, Jonathan Wakely wrote:
On 16/10/15 11:21 +0200, John Marino wrote:
The only significant comment was:
e.g. `"de_DE" => "de_DE@ISO8859-15` should be `e.g. "de_DE" =>
"de_DE.ISO8859-15"` ? Since it's an encoding, @ is used for a
Tested on Linux-PPC64.
2015-11-11 Ville Voutilainen
Implement D0013R2, logical type traits.
/libstdc++-v3
* include/experimental/type_traits (conjunction_v, disjunction_v,
negation_v): New.
* include/std/type_traits (conjunction, disjunction, negation):
Likewise.
*
On 11/11/2015 10:51 PM, Jonathan Wakely wrote:
> On 16/10/15 11:21 +0200, John Marino wrote:
>
> The only significant comment was:
>
> e.g. `"de_DE" => "de_DE@ISO8859-15` should be `e.g. "de_DE" =>
> "de_DE.ISO8859-15"` ? Since it's an encoding, @ is used for a modifier.
> FYI, https://www.ietf.o
On 16/10/15 11:21 +0200, John Marino wrote:
There are a few issues with the locales portion of the libstdc++
testsuite. These issues stem from the fact that much about locales are
not standardized, such as the names of the encodings, how to handle
abbreviated versions of named locales, and modif
On 09.11.15 13:41, Andreas Tobler wrote:
Hi all,
any objections when I apply the below patch to gcc-4.9?
committed as: 230193
Andreas
2015-11-09 Andreas Tobler
Backport from mainline
2015-03-04 Andreas Tobler
* config/rs6000/t-freebsd64: Remove 32-bit soft-f
On 09.11.15 13:38, Andreas Tobler wrote:
Hi,
any objections that I apply this patch to gcc-4.9?
It is FreeBSD only.
committed as: 230192
Andreas
2015-11-09 Andreas Tobler
Backport from mainline
2015-05-18 Andreas Tobler
* config/freebsd-spec.h (FBSD_STARTFIL
On 09/04/2015 11:36 AM, Ajit Kumar Agarwal wrote:
diff --git a/gcc/passes.def b/gcc/passes.def
index 6b66f8f..20ddf3d 100644
--- a/gcc/passes.def
+++ b/gcc/passes.def
@@ -82,6 +82,7 @@ along with GCC; see the file COPYING3. If not see
NEXT_PASS (pass_ccp);
/* After CCP we re
On 09/11/15 19:33, Tom de Vries wrote:
On 09/11/15 16:35, Tom de Vries wrote:
Hi,
this patch series for stage1 trunk adds support to:
- parallelize oacc kernels regions using parloops, and
- map the loops onto the oacc gang dimension.
The patch series contains these patches:
1Insert
GCC warns for class members that are initialized with themselves,
and with c++/64667 fixed, this includes references.
Unfortunately, the diagnostic that's printed is inconsistent between
pointers and references, and other members. For the first two kinds,
it's missing the caret, and for others it
The first patch is the work of Fan You, who implemented the
"polymorphic memory resources" feature from the C++ Library
Fundamentals TS as a Google Summer of Code project. Thanks very much
to Fan You for contributing this work, and to Tim Shen for mentoring
the work, and Google for sponsoring it.
On Wed, Nov 11, 2015 at 10:34 AM, Jim Wilson wrote:
> I had to disable the cortex-a57 fma steering pass in the aarch64 port
> while testing the patch. A bootstrap for aarch64 configured
> --with-cpu=cortex-a57 gives multiple ICEs while building the stage1
> libstdc++. The ICEs are in scan_rtx_re
I’ve updated the original patch, which was built against 5.1.0 on
x64_64-apple-darwin13, this patch is against 6.0.0-20151101 on
x86_64-apple-darwin15.
--
If the RTS in use is "configurable" (I believe this is the same in this context
as "restricted") and includes finalization, gna
> > "Would be really nice" means that you're asking us to work on and resolve
> > PR68271 before installing this patch?
>
> Dominique has committed a quick hack for this, so it is not urgent, but
> would be nice to get it resolved. If somebody from Mentor gets to that,
> perfect, otherwise I (or s
---
gcc/testsuite/gcc.dg/graphite/fuse-1.c | 10 +++---
gcc/testsuite/gcc.dg/graphite/fuse-2.c | 4 +---
gcc/testsuite/gcc.dg/graphite/graphite.exp | 2 ++
gcc/testsuite/gcc.dg/graphite/isl-ast-gen-blocks-1.c| 3 ---
gcc/testsuite/gcc.dg/gr
The patch builds the original schedule based on the now optimized scattering
dimension instead of building one based on the loop index only.
The implementation is simpler and catches more cases where the original schedule
and the transformed schedule are the same, such as the one below:
for (i =
On 11/11/2015 10:34 AM, Steve Kargl wrote:
> This probably falls under the "obviously correct" moniker.
> It has been built and tested on i386-*-freebsd. OK to commit?
>
> The patch removes a gfc_internal_error(). I suspect that it
> was originally put into gfortran to cover "correctly written
>
On 11/10/2015 06:41 AM, Dominique d'Humières wrote:
> Index: gcc/testsuite/ChangeLog
> ===
> --- gcc/testsuite/ChangeLog (revision 229793)
> +++ gcc/testsuite/ChangeLog (working copy)
> @@ -1,3 +1,9 @@
> +2015-11-10 Dominique d'Hu
On 11/11/2015 13:25, "Richard Biener" wrote:
>On Wed, Nov 11, 2015 at 1:22 PM, Alan Hayward
>wrote:
>> Hi,
>> I hoped to post this in time for Monday’s cut off date, but
>>circumstances
>> delayed me until today. Hoping if possible this patch will still be able
>> to go in.
>>
>>
>> This patch
This adds an option for the Qualcomm server parts, qdf24xx, just
optimizing like a cortex-a57 for now, same as how the initial Samsung
exynos-m1 support worked.
This was tested with armv8 and aarch64 bootstraps and make check.
I had to disable the cortex-a57 fma steering pass in the aarch64 port
This probably falls under the "obviously correct" moniker.
It has been built and tested on i386-*-freebsd. OK to commit?
The patch removes a gfc_internal_error(). I suspect that it
was originally put into gfortran to cover "correctly written
valid Fortran code cannot possibly ever hit this line
On 11/11/15 15:22, Claudiu Zissulescu wrote:
Please find attached a patch that fixes the ARC backend ICE on pr29921-2 test
from gcc.dg (dg.exp).
The patch will allow generating conditional move also outside expand scope. The
error was triggered during if-conversion.
Ok to apply?
OK.
On Nov 10, 2015, Jeff Law wrote:
>> * function.c (assign_parm_setup_block): Right-shift
>> upward-padded big-endian args when bypassing the stack slot.
> Don't you need to check the value of BLOCK_REG_PADDING at runtime?
> The padding is essentially allowed to vary.
Well, yeah, it's the result o
On 11/11/15 18:17 +0100, Dominique d'Humières wrote:
Revision r230175
2015-11-10 Ville Voutilainen
LWG 2510, make the default constructors of library tag types
explicit.
* include/bits/mutex.h (defer_lock_t, try_lock_t,
adopt_lock_t): Add an explicit default constructor.
On Wed, Nov 11, 2015 at 11:08:21AM +0100, Thomas Schwinge wrote:
> Hi!
>
> On Wed, 11 Nov 2015 09:32:33 +0100, Jakub Jelinek wrote:
> > On Mon, Nov 09, 2015 at 05:11:44PM -0600, James Norris wrote:
> > > diff --git a/gcc/c-family/c-pragma.h b/gcc/c-family/c-pragma.h
> > > index 953c4e3..c6a2981 1
Jakub,
this patch implements default data attribute determination. The current
behaviour defaults to 'copy' and ignores 'default(none)'. The patch corrects that.
1) We emit a diagnostic when 'default(none)' is in effect. The fortran FE emits
some artificial decls that it doesn't otherwise a
Revision r230175
> 2015-11-10 Ville Voutilainen
>
> LWG 2510, make the default constructors of library tag types
> explicit.
> * include/bits/mutex.h (defer_lock_t, try_lock_t,
> adopt_lock_t): Add an explicit default constructor.
> * include/bits/stl_pair.h (piecewise_const
Hi!
On Wed, 11 Nov 2015 08:59:17 -0500, Nathan Sidwell wrote:
> On 11/11/15 07:06, Bernd Schmidt wrote:
> > On 11/10/2015 11:33 PM, Nathan Sidwell wrote:
> >> I've been unable to introduce a testcase for this.
(But you still committed an update to gcc/testsuite/ChangeLog.)
You'll need to put su
This fixes part of PR 60421 by looping in this_thread::sleep_for when
it is interrupted by a signal, and looping in this_thread::sleep_until
to handle clock adjustments.
There are still problems with integer overflow/wrapping in sleep_for,
which still need to be addressed somehow. Maybe using the
Hi,
This patch fixes an obscure cross-testing problem that crashed (OOMed) our
boards at Linaro. Several tests in libstdc++ (e.g., [1]) limit themselves to
some reasonable amount of RAM and then try to allocate 32 gigs. Unfortunately,
the configure test that checks presence of setrlimit is ra
Hi!
On Mon, Oct 19, 2015 at 10:47:54PM +0300, Ilya Verbin wrote:
> So, here is what I have for now. Attached target-29.c testcase works fine
> with
> MIC emul, however I don't know how to (and where) properly check for
> completion
> of async execution on target. And, similarly, where to do un
On Wed, Nov 11, 2015 at 09:56:42AM -0500, Rich Felker wrote:
> > > I'm actually
> > > trying to prepare a simpler FDPIC patch for other gcc versions we're
> > > interested in that's not so invasive, and for now I'm just having
> > > function_symbol replace SFUNC_STATIC with SFUNC_GOT on TARGET_FDPI
On Wed, Nov 11, 2015 at 07:27:51PM +0300, Alexander Monakov wrote:
> > Alex reported to me privately that with the OpenMP 4.5 handling of
> > array section bases (that they are firstprivate instead of mapped)
> > we unnecessarily mark the pointers addressable and that result
> > in less efficient w
On Wed, 11 Nov 2015, Jakub Jelinek wrote:
> Hi!
>
> Alex reported to me privately that with the OpenMP 4.5 handling of
> array section bases (that they are firstprivate instead of mapped)
> we unnecessarily mark the pointers addressable and that result
> in less efficient way of passing them as s
2015-11-11 Uros Bizjak
* config/alpha/alpha-protos.h (print_operand): Remove.
(print_operand_address): Remove.
* config/alpha/alpha.h (PRINT_OPERAND): Remove.
(PRINT_OPERAND_ADDRESS): Remove.
(PRINT_OPERAND_PUNCT_VALID_P): Remove.
* config/alpha/alpha.c (TARGET_PRINT_OPE
Hi!
Alex reported to me privately that with the OpenMP 4.5 handling of
array section bases (that they are firstprivate instead of mapped)
we unnecessarily mark the pointers addressable and that result
in less efficient way of passing them as shared to inner constructs.
They don't need to be made
Hi all,
The attached testcase ICEs when compiled with -march=armv6k -mthumb -Os or any
march
for which -mthumb gives Thumb1:
error: unrecognizable insn:
}
^
(insn 13 12 14 5 (set (reg:SI 116 [ x ])
(unspec:SI [
(mem:SI (reg/v/f:SI 112 [ s ]) [0 MEM[(unsigned char
*)s_
Please find attached a patch that fixes the ARC backend ICE on pr29921-2 test
from gcc.dg (dg.exp).
The patch will allow generating conditional move also outside expand scope. The
error was triggered during if-conversion.
Ok to apply?
Claudiu
ChangeLog:
2015-11-11 Claudiu Zissulescu
On 11/11/15 15:04 +, Szabolcs Nagy wrote:
yes, non-experimental
(since you were talking about libstdc++ changes to
complete the support for the TS).
The TS is experimental. That's the nature of all C++ TSs.
Completing the TS support does not mean anything is non-experimental.
Hi.
Following patch adds support for global variables seen by
an HSAIL executable. The HSA runtime can link a name of a global variable
with pointer to the variable used by host.
Installed to HSA branch.
Martin
>From de58711a6ddbb1e4558a9454d7aeb6d2b33861de Mon Sep 17 00:00:00 2001
From: marxin
On 10/11/15 18:29, Torvald Riegel wrote:
On Tue, 2015-11-10 at 17:26 +, Szabolcs Nagy wrote:
On 09/11/15 00:19, Torvald Riegel wrote:
Hi,
I'd like to summarize the current state of support for the TM TS, and
outline the current plan for the work that remains to complete the
support.
...
On 11/11/15 09:50, Cesar Philippidis wrote:
On 11/11/2015 05:40 AM, Nathan Sidwell wrote:
On 11/10/15 18:08, Cesar Philippidis wrote:
On 11/10/2015 12:35 PM, Nathan Sidwell wrote:
I've committed this to gomp4. In preparing the reworked firstprivate
patch changes for gomp4's gimplify.c I disc
On 11/10/2015 10:35 PM, Abe wrote:
I wrote:
What I'm saying is I don't see a reason for a "definitely always
unsafe" state.
Why would any access not be made safe if a scratchpad is used?
Because the RTL if-converter doesn`t "know" how to convert
{everything that can be made safe using a scratc
On Wed, Nov 11, 2015 at 11:36:26PM +0900, Oleg Endo wrote:
> On Tue, 2015-11-10 at 15:07 -0500, Rich Felker wrote:
>
> > > The way libcalls are now emitted is a bit unhandy. If more special
> > > -ABI
> > > libcalls are to be added in the future, they all have to do the jsr
> > > vs.
> > > bsrf h
On Wed, Nov 11, 2015 at 01:42:04PM +0100, Bernd Schmidt wrote:
> On 11/11/2015 01:31 PM, Marek Polacek wrote:
>
> >Certainly I'm in favor of sharing code between C and C++ FEs, though in
> >this case it didn't seem too important/obvious, because of the extra !=
> >error_mark_node check + I don't r
On 11/11/2015 05:40 AM, Nathan Sidwell wrote:
> On 11/10/15 18:08, Cesar Philippidis wrote:
>> On 11/10/2015 12:35 PM, Nathan Sidwell wrote:
>>> I've committed this to gomp4. In preparing the reworked firstprivate
>>> patch changes for gomp4's gimplify.c I discovered these testcases were
>>> pass
On Tue, Nov 10, 2015 at 04:48:13PM -0700, Jeff Law wrote:
> Someone (I can't recall who) suggested the overflow check ought to be
> shared, I agree. Can you factor out that check, shove it into c-family/ and
> call it from the C & C++ front-ends?
>
> Approved with that change. Please post it her
On Tue, 2015-11-10 at 15:07 -0500, Rich Felker wrote:
> > The way libcalls are now emitted is a bit unhandy. If more special
> > -ABI
> > libcalls are to be added in the future, they all have to do the jsr
> > vs.
> > bsrf handling (some potential candidates for new libcalls are
> > optimized
> >
This saves some more compile-time avoiding vector size iteration for
trivial fails. It also improves time spent by not giving up completely
for all SLP instances if one fails to vectorize because of alignment
issues. And it sneaks in a correctness fix for a previous change.
Bootstrapped on x86_
On 11/11/2015 02:59 PM, Nathan Sidwell wrote:
That's not the problem. How to conditionally enable the test is the
difficulty. I suspect porting something concerning accel_compiler from
the libgomp testsuite is needed?
Maybe a check_effective_target_offload_nvptx which tries to see if
-fofflo
On Wed, Nov 11, 2015 at 03:10:37PM +0100, Dominique d'Humières wrote:
> Is the following OK?
>
> Index: gcc/ChangeLog
> ===
> --- gcc/ChangeLog (revision 230162)
> +++ gcc/ChangeLog (working copy)
> @@ -1,3 +1,10 @@
> +2015-11
On 10/11/15 22:01 +0200, Ville Voutilainen wrote:
LWG 2510, make the default constructors of library tag types
explicit.
* include/bits/mutex.h (defer_lock_t, try_lock_t,
adopt_lock_t): Add an explicit default constructor.
* include/bits/stl_pair.h (piecewise_construct_t): Likewise
On Tue, Nov 10, 2015 at 10:04:30PM +0100, Bernd Schmidt wrote:
> On 11/10/2015 06:44 PM, Segher Boessenkool wrote:
>
> >Yes I know. All the rest of the code around is it like this though.
> >Do you want this written in a saner way?
>
> I won't object to leaving it as-is for now, but in the futur
Is the following OK?
Index: gcc/ChangeLog
===
--- gcc/ChangeLog (revision 230162)
+++ gcc/ChangeLog (working copy)
@@ -1,3 +1,10 @@
+2015-11-11 Dominique d'Humieres
+
+ PR bootstrap/68271
+ * cp/parser.h (cp
Hello.
Following patch has been just applied to HSA branch and is responsible
for correct emission of builtins. As HSAIL can support approximation
for builtins like 'sin', we emit these if unsafe_math_optimization flag
is enabled. Otherwise direct call instructions are emitted.
I would like to in
On 11/11/15 07:06, Bernd Schmidt wrote:
On 11/10/2015 11:33 PM, Nathan Sidwell wrote:
I've committed this patch to trunk. It implements a partitioning
optimization for a loop partitioned over both vector and worker axes.
We can elide the inner vector partitioning state propagation, if there
are
I've committed this patch to gomp4 to remove the openacc-specific enums from
gimplify_omp_ctx. Instead extending the existing omp_region_type enum. A
similar patch will shortly be applied to trunk, now Jakub.s approved it.
If you had patches relying on the old scheme, you'll need to update
On 11/11/15 03:04, Jakub Jelinek wrote:
On Tue, Nov 10, 2015 at 09:12:55AM -0500, Nathan Sidwell wrote:
+ /* Create a local object to hold the instance
+ value. */
+ tree inst = create_tmp_var
+ (TREE_TYPE (TREE_TYPE (
SPARC doesn't have vector support in this testcase and no integer
multiplication. The general scalarization support fails to fold
generated stmts so the following just does what other parts of
the lowering do - factor in constants/constructors.
On another note I noticed a tree sharing issue (mit
Committed as r230160.
Thanks,
Simon
> -Original Message-
> From: Moore, Catherine [mailto:catherine_mo...@mentor.com]
> Sent: 28 October 2015 14:00
> To: Simon Dardis; Matthew Fortune
> Cc: gcc-patches@gcc.gnu.org
> Subject: RE: [PATCH, Mips] Compact branch/delay slot optimization.
>
>
On 11/10/15 18:08, Cesar Philippidis wrote:
On 11/10/2015 12:35 PM, Nathan Sidwell wrote:
I've committed this to gomp4. In preparing the reworked firstprivate
patch changes for gomp4's gimplify.c I discovered these testcases were
passing by accident, and lacked a data clause.
It used to be i
On 11/10/15 17:45, Ilya Verbin wrote:
I've been unable to introduce a testcase for this. The difficulty is we want
to check an rtl dump from the acceleration compiler, and there doesn't
appear to be existing machinery for that in the testsuite. Perhaps
something to be added later?
I haven't tr
On Wed, Nov 11, 2015 at 1:22 PM, Alan Hayward wrote:
> Hi,
> I hoped to post this in time for Monday’s cut off date, but circumstances
> delayed me until today. Hoping if possible this patch will still be able
> to go in.
>
>
> This patch builds upon the change for PR65947, and reduces the amount
On Wed, Nov 11, 2015 at 3:48 AM, Rainer Orth
wrote:
> Ian Lance Taylor writes:
>
>> On Sun, Nov 8, 2015 at 9:21 AM, Rainer Orth
>> wrote:
>>>
>>> There were two remaining problems:
>>>
>>> * Before Solaris 12, sendfile only lives in libsendfile. This lead to
>>> link failures in gotools.
>>>
On Wed, Nov 11, 2015 at 02:11:38PM +0100, Dominique d'Humières wrote:
> The following patch restore bootstrap on darwin
>
> --- ../_clean/gcc/cp/parser.h 2015-11-10 01:54:44.0 +0100
> +++ gcc/cp/parser.h 2015-11-11 12:10:28.0 +0100
> @@ -48,7 +48,7 @@ struct GTY (()) cp_token {
>
Richard,
What we should do to cope with this problem (structure size increasing)?
Should we return to vector comparison version?
Thanks.
Yuri.
2015-11-11 12:18 GMT+03:00 Richard Biener :
> On Tue, Nov 10, 2015 at 3:56 PM, Ilya Enkovich wrote:
>> 2015-11-10 17:46 GMT+03:00 Richard Biener :
>>> O
The following patch restore bootstrap on darwin
--- ../_clean/gcc/cp/parser.h 2015-11-10 01:54:44.0 +0100
+++ gcc/cp/parser.h 2015-11-11 12:10:28.0 +0100
@@ -48,7 +48,7 @@ struct GTY (()) cp_token {
/* Token flags. */
unsigned char flags;
/* Identifier for the pragm
On 11/11/2015 10:21 AM, Richard Biener wrote:
On Tue, 10 Nov 2015, Jan Hubicka wrote:
The reason is that TYPE_CANONICAL is initialized in get_alias_set that may be
called before we finish all merging and then it is more fine grained than what
we need here (i.e. TYPE_CANONICAL of pointers to two
On 11/11/2015 01:31 PM, Marek Polacek wrote:
Certainly I'm in favor of sharing code between C and C++ FEs, though in
this case it didn't seem too important/obvious, because of the extra !=
error_mark_node check + I don't really like the new function getting *type
and setting it there.
Make it
On 11/11/2015 01:20 PM, Richard Biener wrote:
> On Wed, Nov 11, 2015 at 12:18 PM, Martin Liška wrote:
>> Hi.
>>
>> There's a fix for fallout of r230027.
>>
>> Patch can bootstrap and survives regression tests on x86_64-linux-gnu.
>
> Hmm, but only the new elements are zeroed so this still is diff
On Tue, Nov 10, 2015 at 12:40:49PM -0700, Martin Sebor wrote:
> On 11/10/2015 09:36 AM, Marek Polacek wrote:
> >While both C and C++ FEs are able to reject e.g.
> >int a[__SIZE_MAX__ / sizeof(int)];
> >they are accepting code such as
> >int (*a)[__SIZE_MAX__ / sizeof(int)];
> >
> >As Joseph pointed
On 11/11/2015 01:31 PM, Eric Botcazou wrote:
Yes, it probably should, thanks for spotting it, revised patch attached.
PR target/67265
* ira.c (ira_setup_eliminable_regset): Do not necessarily create the
frame pointer for stack checking if non-call exceptions aren't us
This patch is committed.
Thanks Joern,
Claudiu
> -Original Message-
> From: Joern Wolfgang Rennecke [mailto:g...@amylaar.uk]
> Sent: Tuesday, November 10, 2015 3:02 PM
> To: Claudiu Zissulescu; gcc-patches@gcc.gnu.org
> Cc: Francois Bedard; jeremy.benn...@embecosm.com
> Subject: Re: [PATC
> This piece of code along doesn't tell me exactly why the frame pointer
> is needed. I was looking for an explicit use, but I now guess that if
> you have multiple adjusts of the [stack] pointer you can't easily undo
> them in the error case (the function behaves as-if using alloca). Is
> that it?
This patch is committed (without the gen_compare_reg change).
Thanks Joern,
Claudiu
> Apart from the gen_compare_reg change, the patch is OK.
> If the v2 support mostly works like support for the other subtargets, you may
> check it in without the gen_compare_reg change.
> If that change is requ
On Tue, Nov 10, 2015 at 06:38:31PM +0100, Paolo Carlini wrote:
> Hi,
>
> On 11/10/2015 05:36 PM, Marek Polacek wrote:
> >+
> >+/* Did array size calculations overflow or does the array
> >+ cover more than half of the address-space? */
> >+if (COMPLETE_TYPE_
Hi,
I hoped to post this in time for Monday’s cut off date, but circumstances
delayed me until today. Hoping if possible this patch will still be able
to go in.
This patch builds upon the change for PR65947, and reduces the amount of
code produced in a vectorized condition reduction where operand
On 04/11/15 09:45, Jiong Wang wrote:
> As discussed at the bugzilla
>
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67305
>
> neon_vector_mem_operand is broken. As the comments says
> "/* Reject eliminable registers. */", the code block at the head
> of this function which checks eliminabl
On Wed, Nov 11, 2015 at 12:18 PM, Martin Liška wrote:
> Hi.
>
> There's a fix for fallout of r230027.
>
> Patch can bootstrap and survives regression tests on x86_64-linux-gnu.
Hmm, but only the new elements are zeroed so this still is different
from previous behavior.
Note that the previous .cre
On 11/11/15 12:08, Charles Baylis wrote:
On 11 November 2015 at 11:22, Kyrill Tkachov wrote:
Hi Charles,
On 08/11/15 00:26, charles.bay...@linaro.org wrote:
From: Charles Baylis
Charles Baylis
* config/arm/neon.md (neon_vld1_lane): Remove error for
invalid
lane numbe
On 04/11/15 09:45, Jiong Wang wrote:
As discussed at the bugzilla
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67305
neon_vector_mem_operand is broken. As the comments says
"/* Reject eliminable registers. */", the code block at the head
of this function which checks eliminable registers i
On 11 November 2015 at 11:22, Kyrill Tkachov wrote:
> Hi Charles,
>
> On 08/11/15 00:26, charles.bay...@linaro.org wrote:
>>
>> From: Charles Baylis
>>
>> Charles Baylis
>>
>> * config/arm/neon.md (neon_vld1_lane): Remove error for
>> invalid
>> lane number.
>> (neon_v
On 11/10/2015 11:33 PM, Nathan Sidwell wrote:
I've committed this patch to trunk. It implements a partitioning
optimization for a loop partitioned over both vector and worker axes.
We can elide the inner vector partitioning state propagation, if there
are no intervening instructions in the worke
On 11/11/2015 12:38 PM, Eric Botcazou wrote:
this is an ICE on an asm statement requiring a lot of registers, when compiled
in 32-bit mode on x86/Linux with -O -fstack-check -fPIC:
pr67265.c:10:3: error: 'asm' operand has impossible constraints
The issue is that, since stack checking defines ST
Hi!
Committed to gomp-4_0-branch in r230154:
commit 1fe1fa3a7b9d4286630cd286e0a52abe2d11e955
Merge: 02d9df1 76e711b
Author: tschwinge
Date: Wed Nov 11 11:43:09 2015 +
svn merge -r 230048:230082 svn+ssh://gcc.gnu.org/svn/gcc/trunk
git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc
Ian Lance Taylor writes:
> On Sun, Nov 8, 2015 at 9:21 AM, Rainer Orth
> wrote:
>>
>> There were two remaining problems:
>>
>> * Before Solaris 12, sendfile only lives in libsendfile. This lead to
>> link failures in gotools.
>>
>> * Solaris 12 introduced a couple more types that use _in6_ad
1 - 100 of 128 matches
Mail list logo
