Hi,
For ILP32 on AARCH64, we have ptr_mode != Pmode (we have ptr_mode
being SImode while Pmode is DImode and POINTER_SIZE is 32). This
breaks ipa-polymorphic-call assumption that Pmode is the correct mode
for pointers. Right now before this patch we get many testcase
failures in the C++
On Wed, Nov 19, 2014 at 6:32 PM, Ilya Tocar tocarip.in...@gmail.com wrote:
Hi,
New revision of Intel ISA reference [1] has new instructions:
Clwb, pcommit and new flavors of AVX512. Patch bellow adds them.
I understand that stage 1 is closed, however those changes shouldn't
affect anything
On 19/11/14 09:29, Yangfei (Felix) wrote:
Sorry for missing the point. It seems to me that 't2' here will conflict with
condition of the pattern *movhi_insn_arch4:
TARGET_ARM
arm_arch4
(register_operand (operands[0], HImode)
|| register_operand (operands[1],
-Original Message-
From: Kyrill Tkachov [mailto:kyrylo.tkac...@arm.com]
Sent: Tuesday, November 18, 2014 11:08 PM
To: Terry Guo; gcc-patches@gcc.gnu.org
Cc: ger...@pfeifer.com
Subject: Re: [PATCH][wwwdocs] Update 5.0 changes.html with Thumb1 UAL
On 18/11/14 02:48, Terry Guo
On 19/11/14 09:29, Yangfei (Felix) wrote:
Sorry for missing the point. It seems to me that 't2' here will
conflict with
condition of the pattern *movhi_insn_arch4:
TARGET_ARM
arm_arch4
(register_operand (operands[0], HImode)
|| register_operand
Hi Philipp,
I don't mind it being in config/arm if you plan to wire it up later,
good to know.
Another comment inline
Thanks,
Kyrill
On 19/11/14 21:42, Philipp Tomsich wrote:
Here's an updated patch with Kyrill's and Andrew's comments integrated.
I left the file in the
On 19/11/14 09:29, Yangfei (Felix) wrote:
Sorry for missing the point. It seems to me that 't2' here will
conflict with
condition of the pattern *movhi_insn_arch4:
TARGET_ARM
arm_arch4
(register_operand (operands[0], HImode)
|| register_operand
When moving tree-ssa-forwprop.c:associate_plusminus to match.pd patterns
a single-use restriction escaped my eye. It is indeed important
for non-simplifications like
(ptr p+ off1) p+ off2 - ptr p+ (off1 + off2)
to not un-CSE. The association is most useful to enable later
re-association as
On 19/11/14 18:22, Sandra Loosemore wrote:
On 11/13/2014 10:47 AM, Andrew Pinski wrote:
On Thu, Nov 13, 2014 at 9:42 AM, Sandra Loosemore
san...@codesourcery.com wrote:
On 11/13/2014 10:27 AM, Richard Earnshaw wrote:
On 13/11/14 17:05, Ramana Radhakrishnan wrote:
On Thu, Nov 13, 2014 at
Hi,
r217646 enhances ifcvt to handle cbranchcc4 instruction. But ifcvt does not
strictly check the dependence before moving instructions before IF. Then
some instructions, which clobber CC, are inserted before the cbranchcc4
instruction.
For the case in the patch, ifcvt transfers code from
The compiler silently skips the generation of code to perform the
conversion of an access type whose designated type is a class-wide
interface type, thus causing unexpected problems at runtime in
dispatching calls to the target object. After this patch the
following test compiles and executes
This change lifts the arbitrary limitation on the number of iterations that
can be executed between loading of the inlined bodies and instantiation of
the generic bodies of external units when inter-unit inlining is activated.
It was previously limited to 1 but this may be not sufficient in some
Kyrill,
I don't mind it being in config/arm if you plan to wire it up later, good to
know.
Another comment inline….
I’ll clean up the missing xgene1_ and the mistyped xgene_ prefix and resubmit.
+(define_insn_reservation div 2
+ (and (eq_attr tune xgene1)
+ (eq_attr type
Hi Philipp,
On 20/11/14 10:47, Dr. Philipp Tomsich wrote:
Kyrill,
I don't mind it being in config/arm if you plan to wire it up later, good to
know.
Another comment inline….
I’ll clean up the missing xgene1_ and the mistyped xgene_ prefix and resubmit.
+(define_insn_reservation div 2
+
This change makes is so that Following_Address_Clause is invoked only if this
is really necessary from Analyze_Object_Declaration. This saves about 1% of
the compilation time at low optimization levels. No functional changes.
Tested on x86_64-pc-linux-gnu, committed on trunk
2014-11-20 Eric
An operator that is called in functional notation is rewritten as an operator
so that its operands can be properly resolved. ASIS needs the semantic info
to be available on the original node, so in ASIS mode the resolved operands
are linked back to the original call. This patch takes into account
If the left-hand side of an assignment is an Ada 2012 generalized indexing
with an implicit derenference, the compiler must verify that the type of
the access discriminant that provides the implicit dereference is not an
access_to_constant.
Compiling ada_test.adb must yield:
The following changes are importants:
- It is possible to have multiple tasks waiting for a child process
to terminate.
- When a child terminates, a single wait call will receive the
corresponding process id.
- A call to wait will handle new incoming child processes.
Tested on
The patch modifies the expansion of attributes 'Old and 'Update to ensure that
the tag of a tagged prefix is not modified as a result attribute evaluation.
-- Source --
-- types.ads
package Types is
type Root is tagged record
X : Integer;
end record;
On Wed, Nov 19, 2014 at 9:42 PM, Philipp Tomsich
philipp.toms...@theobroma-systems.com wrote:
Here's an updated patch with Kyrill's and Andrew's comments integrated.
I left the file in the config/arm-directory, as XGene-family is capable of
executing ARMv7 and we will wire this into the 32bit
This patch the following SPARK rule (the part about 'Loop_Entry, 'Old, 'Update)
If the Extensions_Visible aspect is False for a subprogram, then certain
restrictions are imposed on the use of any parameter of the subprogram which
is of a specific tagged type. Such a parameter shall not
While the library of formal maps/sets correctly set SPARK_Mode on spec
(On) and private part / body (Off), it was not the case for lists and
vectors, thus causing some errors in GNATprove when instantiating such
formal containers because bodies contain non-SPARK features (e.g. access
types in
Range checks on empty ranges typically correspond to deactivated code
based on a given configuration (say, dead code inside a loop over the
empty range). In GNATprove mode, instead of issuing an error message
(which would stop analysis), enable the range check so that GNATprove
will issue a
Like other similar pragmas, we should disallow duplicate pragma or
aspect Linker_Section for non-overloadable entities (for the case
of overloading, the pragma only applies to previous entities which
do not have such a pragma).
The following should compile with the given error:
1. package
Ok to commit?
gcc/
* config/mips/mips.c (mips_process_sync_loop): Place a nop in the
delay slot of the branch likely instruction.
With an updated ChangeLog to account for the changes in the callers, OK.
Matthew
Hi,
The patch expand even/odd permutation using:
and, and, pack in odd case
shift, shift, pack in even case
instead of current pshufb, pshufb, or or big set of unpack insns.
AVX2/CORE bootstrap and make check passed.
expensive tests are in progress
Is it ok for trunk?
Evgeny
2014-11-20
On Thu, Nov 20, 2014 at 02:36:26PM +0300, Evgeny Stupachenko wrote:
+ /* Only V8HI, V16QI, V16HI and V32QI modes are more profitable than general
+ shuffles. */
I think switch (d-vmode) would be more readable.
+ op = gen_reg_rtx (d-vmode);
+ t = gen_reg_rtx (V4DImode);
+
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. F.i., the asm file may contain an insn:
On platform where the switch is allowed, the gcc driver, when called with
-Gnnn (nnn is a non negative number) invokes the compiler (gnat1) with
-G nnn. This patch skips the argument nnn after -G, so that it is not
taken as a source file name.
Tested on x86_64-pc-linux-gnu, committed on trunk
On Wed, Nov 19, 2014 at 9:59 PM, Uros Bizjak ubiz...@gmail.com wrote:
Hello!
libcpp/lex.c includes ../gcc/config/i386/cpuid.h, and is picked up
by the system compiler during stage1. Recently, cpuid.h was changed to
account for %ebx changes and now uses b asm constraint for i686 even
with
On Tue, Nov 18, 2014 at 11:14 AM, Michael Matz m...@suse.de wrote:
Hi,
On Mon, 17 Nov 2014, Richard Biener wrote:
This means I can no longer interrupt a compile that is running too long?
No, that's not what it means, cc1 will also get the SIGINT.
You should instead debug the actual
On Thu, Nov 20, 2014 at 1:48 AM, Zhenqiang Chen zhenqiang.c...@arm.com wrote:
Hi,
r217646 enhances ifcvt to handle cbranchcc4 instruction. But ifcvt does not
strictly check the dependence before moving instructions before IF. Then
some instructions, which clobber CC, are inserted before the
On Thu, Nov 20, 2014 at 12:36 PM, Evgeny Stupachenko evstu...@gmail.com wrote:
Hi,
The patch expand even/odd permutation using:
and, and, pack in odd case
shift, shift, pack in even case
instead of current pshufb, pshufb, or or big set of unpack insns.
AVX2/CORE bootstrap and make check
On 11/13/2014 05:06 AM, Jan Hubicka wrote:
this patch adds infrastructure for proper streaming and merging of
TREE_TARGET_OPTION.
This breaks the offloading path via LTO since it introduces an
incompatibility in LTO format between host and offload machine.
A very quick patch to fix it is
Now that I've managed to put together and test all the submitted OpenACC
patches I found there was one piece missing. The problem is that omp-low
on the host likes to generate function names like _main._omp_fn. On
ptx, the dot is not allowed in identifiers, so we have to rewrite this
to use a
On 11/20/2014 07:52 AM, Jakub Jelinek wrote:
On Thu, Nov 20, 2014 at 03:19:11AM +0100, Bernd Schmidt wrote:
Thomas had apparently already pointed out an issue with the new gomp_target
class (there are multiple similar types of statements we want to handle with
OpenACC, they have different codes
On Thu, Nov 20, 2014 at 12:05 AM, Andrew MacLeod amacl...@redhat.com wrote:
On 11/19/2014 05:24 PM, David Malcolm wrote:
On Wed, 2014-11-19 at 22:36 +0100, Richard Biener wrote:
On November 19, 2014 10:09:56 PM CET, Andrew MacLeod
amacl...@redhat.com wrote:
On 11/19/2014 03:43 PM, Richard
On Wed, Nov 19, 2014 at 11:24 PM, David Malcolm dmalc...@redhat.com wrote:
On Wed, 2014-11-19 at 22:36 +0100, Richard Biener wrote:
On November 19, 2014 10:09:56 PM CET, Andrew MacLeod amacl...@redhat.com
wrote:
On 11/19/2014 03:43 PM, Richard Biener wrote:
On November 19, 2014 8:26:23 PM
On Thu, Nov 20, 2014 at 1:18 AM, Mike Stump mikest...@comcast.net wrote:
On Nov 14, 2014, at 2:26 AM, Richard Biener richard.guent...@gmail.com
wrote:
On Fri, Nov 14, 2014 at 2:10 AM, Jeff Law l...@redhat.com wrote:
On 11/13/14 12:37, Mike Stump wrote:
I was doing a merge, and it failed to
On Thu, Nov 20, 2014 at 4:21 AM, Patrick Palka patr...@parcs.ath.cx wrote:
VRP may simplify a conditional like i = 5 to i == 5 if it is known that
the lower bound of i's range is 5, e.g. [5, +INF]. But if the upper
bound of i's range is also overflow infinity, i.e. [5, +INF(OVF)] then
this
On Thu, 20 Nov 2014, Bernd Schmidt wrote:
On 11/13/2014 05:06 AM, Jan Hubicka wrote:
this patch adds infrastructure for proper streaming and merging of
TREE_TARGET_OPTION.
This breaks the offloading path via LTO since it introduces an incompatibility
in LTO format between host and
Running the testsuite after bootstrap-ubsan on gcc112 shows several issues. See
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63426 for the full list.
This patch fixes several of them.
Tested on powerpc64-unknown-linux-gnu.
OK for trunk?
Thanks.
2014-11-20 Markus Trippelsdorf
Hello Jan-Benedict,
Hi!
This patch updates the files taken from Automake. Committed.
MfG, JBG
the updated version of missing will confuse the gmp-4.3.2 configure
script if it is installed in-tree with contrib/download_prerequisites
and flex is not installed:
...
checking readline
Thank you.
Patch with proposed fixes:
diff --git a/gcc/config/i386/i386.c b/gcc/config/i386/i386.c
index 085eb54..09c0057 100644
--- a/gcc/config/i386/i386.c
+++ b/gcc/config/i386/i386.c
@@ -48322,6 +48322,120 @@ expand_vec_perm_vpshufb2_vpermq_even_odd
(struct expand_vec_perm_d *d)
return
On 11/20/2014 08:08 AM, Richard Biener wrote:
On Thu, Nov 20, 2014 at 12:05 AM, Andrew MacLeod amacl...@redhat.com wrote:
On 11/19/2014 05:24 PM, David Malcolm wrote:
On Wed, 2014-11-19 at 22:36 +0100, Richard Biener wrote:
On November 19, 2014 10:09:56 PM CET, Andrew MacLeod
On Thu, Nov 20, 2014 at 8:27 AM, Markus Trippelsdorf
mar...@trippelsdorf.de wrote:
Running the testsuite after bootstrap-ubsan on gcc112 shows several issues.
See
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63426 for the full list.
This patch fixes several of them.
Tested on
On 11/20/2014 02:20 PM, Richard Biener wrote:
On Thu, 20 Nov 2014, Bernd Schmidt wrote:
On 11/13/2014 05:06 AM, Jan Hubicka wrote:
this patch adds infrastructure for proper streaming and merging of
TREE_TARGET_OPTION.
This breaks the offloading path via LTO since it introduces an
Hello.
As I reimplemented fibheap to C++ template, Honza told me that replace_key
method actually
supports just decrement operation. Old implementation suppress any feedback if
we try to increase key:
fibheap.c:
...
/* If we wanted to, we could actually do a real increase by redeleting and
My mistake yesterday. I thought I'd tested both x86_64 -m64/-m32, but not so.
Anyway, as the comment says, the backend keeps querying the static chain, and
if you don't early out, it sets ix86_static_chain_on_stack, at which point the
setting is permanent and affects prologue generation, and not
This patch fixes two errors in the handling of extension aggregates for limited
types: Ancestor part of extension aggregate can itself be an extension aggregate
as well as a function call that is rewritten as a reference.
The following must compile quietly:
gcc -c p2.adb
gcc -c
This enables inter-unit inlining of expression functions with -gnatn1, or more
simply with -O1/-O2 -gnatn. These functions are automatically candidates for
inlining, but there were actually inlined across units only with -gnatn2, or
more simply -O3 -gnatn.
The following program must compile
This patch modifies the mechanism which creates a subtype from an arbitrary
expression. The mechanism now captures the bounds of all index constraints
when the expression is of an array type.
-- Source --
-- pack.ads
with Ada.Finalization; use Ada.Finalization;
On 11/20/2014 12:36 PM, Evgeny Stupachenko wrote:
+ /* Required for pack. */
+ if (!TARGET_SSE4_2 || d-one_operand_p)
+return false;
Why the SSE4_2 check here when...
+
+ /* Only V8HI, V16QI, V16HI and V32QI modes are more profitable than general
+ shuffles. */
+ if (d-vmode
The compiler suppresses debugging information on predefined units that are
inlined in the code, because stepping into run-time units often complicates
debugging activity. We make an exception for calls that appear in the source,
when the unit is part of the Ada hierarchy, to facilitate monitoring
On 14/11/14 17:33, Marcus Shawcroft wrote:
On 14 November 2014 08:12, Tejas Belagod tejas.bela...@arm.com wrote:
2014-11-14 Tejas Belagod tejas.bela...@arm.com
gcc/
* config/aarch64/aarch64-protos.h (aarch64_classify_symbol):
Fixup prototype.
*
Hi,
On Thu, 20 Nov 2014, Patrick Palka wrote:
-wrapper is specifically also for invoking cc1 with gdb from the
driver (that's the usecase documented with -wrapper), so it better
should work as intended. I don't know what problems Patrick had with
that, though. For me gcc -wrapper
On Wed, 2014-11-19 at 23:09 -0800, Mike Stump wrote:
On Nov 19, 2014, at 10:23 PM, David Malcolm dmalc...@redhat.com
wrote:
It's not clear to me if I can approve my own patches to the jit
So, to answer that, we look at MAINTAINERS, and look up your name:
Various
This patch avoids issuing a warning for a missing component clause
for a discriminant in an unchecked union, and also avoids printing
a line for such a component in the -gnatR2 output.
The following program:
1. with Interfaces;
2. procedure Test_Union is
3. type Test_Type (Flag
On 11/19/2014 08:56 PM, H.J. Lu wrote:
On Wed, Nov 19, 2014 at 10:04 AM, Jakub Jelinek ja...@redhat.com wrote:
On Wed, Nov 19, 2014 at 03:58:50PM +0100, Richard Henderson wrote:
As opposed to always being a decl. This is a prerequisite
to allowing the static chain to be loaded for indirect
Good point! gen_shift also requires only SSE2.
That way we can optimize out interleave sequence for V16QI mode in
expand_vec_perm_even_odd_1.
Thanks!
Evgeny
Updated patch:
diff --git a/gcc/config/i386/i386.c b/gcc/config/i386/i386.c
index 085eb54..054089b 100644
--- a/gcc/config/i386/i386.c
+++
On 11/17/14 18:52, Chen Gang wrote:
What you said sound reasonable to me.
I shall try to send patch v2 within this week (use pretty_printer).
Thanks.
On 11/17/14 16:15, Marek Polacek wrote:
On Mon, Nov 17, 2014 at 08:38:19AM +0100, Jakub Jelinek wrote:
I think easiest would be to
Oh, sorry, after ran more than 10 days, the qemu crashed :-(
After checked the output log, and compare with the original log, we know
we have finished more than 90% test, and it is OK (no any new issues).
I guess the reason is I started too many other things on this machine.
Next, I shall try
Hi all,
As Joseph reminded, new -mcpu options should be documented in
invoke.texi. This adds the documentation for the cortex-a17 and
cortex-a17.cortex-a7 values.
Ok to go in if the corresponding support patches posted earlier are
accepted?
Thanks,
Kyrill
2014-11-19 Kyrylo Tkachov
Hi,
On Thu, 20 Nov 2014, Richard Biener wrote:
I don't think this API will make the non-C++-fans happier; I think the
objection to the work I just merged is that it's adding more C++ than
those people are comfortable with.
How so? It's already super-ugly in those views. We decided to
An sh compiler fails to build on systems that have plain char unsigned.
Fix that.
2014-11-20 Segher Boessenkool seg...@kernel.crashing.org
gcc/
PR target/60111
* config/sh/sh.c: Use signed char for signed field.
---
gcc/config/sh/sh.c | 2 +-
1 file changed, 1 insertion(+),
On Thu, 2014-11-20 at 07:41 -0800, Segher Boessenkool wrote:
An sh compiler fails to build on systems that have plain char unsigned.
Fix that.
Ouch. Thanks for spotting this. OK for trunk, 4.9 and 4.8.
Cheers,
Oleg
Two units, one in a multi-source file and one in another source with
the same base file name do not have the same object file name.
No error during processing of the following project file should be
reported:
project Prj is
package Naming is
for Spec (foo_bar) use foo_bar.ads at 2;
Hi Bernd,
On Thu, 2014-11-20 14:34:00 +0100, Bernd Edlinger bernd.edlin...@hotmail.de
wrote:
This patch updates the files taken from Automake. Committed.
the updated version of missing will confuse the gmp-4.3.2 configure
script if it is installed in-tree with
On Thu, Nov 20, 2014 at 02:27:52PM +0100, Markus Trippelsdorf wrote:
2014-11-20 Markus Trippelsdorf mar...@trippelsdorf.de
* emit-rtl.c (const_wide_int_htab_hash): Likewise.
* loop-iv.c (determine_max_iter): Likewise.
(iv_number_of_iterations): Likewise.
*
On 20/11/14 15:31, Kyrill Tkachov wrote:
Hi all,
As Joseph reminded, new -mcpu options should be documented in
invoke.texi. This adds the documentation for the cortex-a17 and
cortex-a17.cortex-a7 values.
Ok to go in if the corresponding support patches posted earlier are
accepted?
On Wed, Nov 19, 2014 at 9:02 PM, David Malcolm dmalc...@redhat.com wrote:
On Wed, 2014-11-19 at 09:59 -0700, Jeff Law wrote:
On 11/19/14 03:46, David Malcolm wrote:
Fix this leak:
160 bytes in 5 blocks are definitely lost in loss record 154 of 228
at 0x4A0645D: malloc (in
On Thu, Nov 20, 2014 at 4:34 PM, Michael Matz m...@suse.de wrote:
Hi,
On Thu, 20 Nov 2014, Richard Biener wrote:
I don't think this API will make the non-C++-fans happier; I think the
objection to the work I just merged is that it's adding more C++ than
those people are comfortable with.
Hi all,
This is a backport for gcc-4_9-branch of the patch [PR63762]GCC
generates UNPREDICTABLE STR with Rn = Rt for arm posted in:
https://gcc.gnu.org/ml/gcc-patches/2014-11/msg02253.html
arm-none-eabi has been test on the model, no new issues. bootstrapping
and regression tested on x86,
Fixing version number according to new GCC naming scheme.
PR ada/63931
* gnatvsn.ads (Library_Version): Switch to 5.
Index: gnatvsn.ads
===
--- gnatvsn.ads (revision 217874)
+++
On 11/20/14 09:01, Richard Biener wrote:
Is there a governance distinction here, between patch review vs
decisions of the steering committee? i.e. do changes to the maintainers
part of the MAINTAINERS file require higher-level approval?
Yes, reviewers and maintainers are appointed by the
On 09-05-14 16:47, Diego Novillo wrote:
I would probably use git config directly here. It would work with both
git and svn checkouts (if you have a global .git configuration). But
testing for .git is fine with me as well.
I like Peter's idea of having a ~/.mklog file to override. This would
Bootstrap / make check passed with updated patch.
Is it still ok?
It looks like we don't need expand_vec_perm_vpshufb2_vpermq_even_odd
any more with the patch.
However the clean up will be in the separate patch after appropriate testing.
Modified ChangeLog:
2014-11-20 Evgeny Stupachenko
On Wed, Nov 19, 2014 at 9:19 PM, Andrew Stubbs a...@codesourcery.com wrote:
On 19/11/14 16:39, Marek Polacek wrote:
On Wed, Nov 19, 2014 at 04:32:43PM +, Andrew Stubbs wrote:
+if (warning_at (gimple_location (elt-stmt),
+
2014-11-20 Alex Velenko alex.vele...@arm.com
*MAINTAINERS (write-after-approval): Add myself.
diff --git a/MAINTAINERS b/MAINTAINERS
index 11a28ef..eada4e9 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -566,6 +566,7 @@ David Ung
dav...@mips.com
Neil
On 20 November 2014 07:49, Marcus Shawcroft marcus.shawcr...@gmail.com wrote:
On 19 November 2014 19:05, Charles Baylis charles.bay...@linaro.org wrote:
PR target/63870
* config/aarch64/aarch64-builtins.c (aarch64_simd_expand_args): Pass
expression to
Hello.
Following patch fixes ICE in IPA ICF. Problem was that number of non-debug
statements in a BB can
change (for instance by IPA split), so that the number is recomputed.
Patch can bootstrap on x86_64-linux-pc and no regression has been seen.
Ready for trunk?
Thanks,
Martin
gcc/ChangeLog:
Hi,
MPX runtime checks some feature bits in order to check MPX is fully supported.
Runtime does it by cpuid calls but there is a __builtin_cpu_supports which may
be used for that. Unfortunately currently it doesn't support required bits.
Will it be OK to add them for trunk?
Thanks,
Ilya
--
On Thu, Nov 20, 2014 at 5:30 PM, Martin Liška mli...@suse.cz wrote:
Hello.
Following patch fixes ICE in IPA ICF. Problem was that number of non-debug
statements in a BB can
change (for instance by IPA split), so that the number is recomputed.
Huh, so can it get different for both candidates?
On Thu, Nov 20, 2014 at 07:36:03PM +0300, Ilya Enkovich wrote:
Hi,
MPX runtime checks some feature bits in order to check MPX is fully
supported. Runtime does it by cpuid calls but there is a
__builtin_cpu_supports which may be used for that. Unfortunately
currently it doesn't support
Hi all,
This patch makes the arm_issue_rate function lookup the issue rate of
the process from the tuning structs.
This makes it look more like the aarch64 mechanism and centralises a
processor-specific construct to the
tuning structs, thus not forcing us to remember to update the
On Thu, Nov 20, 2014 at 05:22:20PM +0100, Tom de Vries wrote:
+my $conf = $ENV{HOME}/.mklog;
+if (-f $conf) {
+open (CONF, $conf)
+ or die Could not open file '$conf' for reading: $!\n;
+while (CONF) {
+ if (m/^\s*NAME\s*=\s*(.*)\s*$/) {
The final \s* never matches anything
On 20 November 2014 14:33, Tejas Belagod tejas.bela...@arm.com wrote:
The same patch applies cleanly to 4.9. OK to commit?
Thanks,
Tejas.
Provided it regresses ok, yes.
/Marcus
I should say that the patch context depends on the macro fusion hook
implementation posted here:
https://gcc.gnu.org/ml/gcc-patches/2014-11/msg00958.html
Kyrill
On 20/11/14 16:43, Kyrill Tkachov wrote:
Hi all,
This patch makes the arm_issue_rate function lookup the issue rate of
the process
This patch fixes a bunch of ICEs related to C++14 constexprs and
-fsanitize=undefined. We should ignore ubsan internal functions
and ubsan builtins in constexpr functions in cxx_eval_call_expression.
Also add proper printing of internal functions into the C++ printer.
Bootstrapped/regtested on
On Thu, Nov 20, 2014 at 06:14:52PM +0100, Marek Polacek wrote:
This patch fixes a bunch of ICEs related to C++14 constexprs and
-fsanitize=undefined. We should ignore ubsan internal functions
and ubsan builtins in constexpr functions in cxx_eval_call_expression.
Also add proper printing of
This patch resolves a bootstrap failure in gomp-4_0-branch, which I
probably introduced after I switched the cache error message from a
gfc_error to a sorry. The code parameter isn't being used anymore by
resolve_oacc_cache, so I've explicitly marked it as unused.
I've applied this patch to
On Wed, Nov 19, 2014 at 02:23:47PM -0800, Mike Stump wrote:
On Nov 19, 2014, at 1:57 PM, Jakub Jelinek ja...@redhat.com wrote:
Though, following patch is just fine for me too, I don't think it will
make a significant difference:
This version is fine by me.
Richard, are you ok with that
Hi!
This patch fixes ICEs if a non-lvalue vector (say cast of one vector
to another vector type) was subscripted and used as lhs.
The following patch, if *vecp is not lvalue, will copy it to a temporary
variable which can be made addressable for the subscription, and afterwards
wrap it into a
Hi,
On Mon, Nov 03, 2014 at 10:46:49PM +0100, Marc Glisse wrote:
On Mon, 3 Nov 2014, Marc Glisse wrote:
On Mon, 3 Nov 2014, Martin Jambor wrote:
I just applied your patch on top of trunk revision 217032 on my
Ah, that explains it, thanks. This patch is a follow-up to
r217034. Still, I
On 14 November 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 2 and covers all the aarch64 changes.
When combined with the first patch, It fixes up movoi/ci/xi for Big
Endian, so that we end up with the lab of a
Hi!
Before the r211956 changes, the only places that set si-stmt
were required to check that stpcpy has been declared (with the right
prototype) to signal the strlen pass that it can use stpcpy for
optimization. But r211956 sets si-stmt also for malloca call,
which isn't in any way related to
On 13 November 2014 10:09, David Sherwood david.sherw...@arm.com wrote:
gcc/:
2014-11-13 David Sherwood david.sherw...@arm.com
* config/aarch64/aarch64-protos.h (aarch64_simd_attr_length_rglist,
aarch64_reverse_mask): New decls.
*
Hi!
Seems the gcc.target/ia64/small-addr-1.c testcase is failing on ia64 since
r210262 but clearly has been failing for much longer if compiled with C++
(just there is insufficient testsuite coverage).
The problem is that for the model attribute (and apparently common_object on
VMS too), the
On 11/20/2014 10:48 AM, Zhenqiang Chen wrote:
+/* Check X clobber CC reg or not. */
+
+static bool
+clobber_cc_p (rtx x)
+{
+ RTX_CODE code = GET_CODE (x);
+ int i;
+
+ if (code == CLOBBER
+ REG_P (XEXP (x, 0))
+ (GET_MODE_CLASS (GET_MODE (XEXP (x, 0))) == MODE_CC))
+
On 2014.11.20 at 08:59 -0500, David Edelsohn wrote:
On Thu, Nov 20, 2014 at 8:27 AM, Markus Trippelsdorf
mar...@trippelsdorf.de wrote:
Running the testsuite after bootstrap-ubsan on gcc112 shows several issues.
See
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63426 for the full list.
1 - 100 of 135 matches
Mail list logo