On Mon, Aug 18, 2014 at 5:15 PM, Jan Hubicka hubi...@ucw.cz wrote:
Hi,
Firefox FDO build fails with undefined reference to __builtin_unreachable.
THis is caused by a bug
in gimple-fold that introduces the refernece in the cases it hits impossible
devirtualization.
We use
On Tue, Aug 19, 2014 at 7:20 AM, lin zuojian manjian2...@gmail.com wrote:
Hi,
Here patch v2. Move the function as Andrew instructed.
* tree-inline.c (setup_one_parameter): Add strict aliasing check.
* c-family/c-common.c (strict_aliasing_warning): Move to alias.c.
*
Hello,
On 12 Aug 10:58, Andrey Turetskiy wrote:
All remarks from
https://gcc.gnu.org/ml/gcc-patches/2014-06/msg02166.html are fixed.
Updated version of liboffloadmic you can find in GIT branch
kyukhin/gomp4-offload:
Hi Richard,
Generally I don't think we want to expand the use of the IMHO broken
strict_aliasing_warning code.
If you have read my test code, you must understand it is the
programmer who responsible for this undefined behavior, instead of
the compiler. Like PR 60546 concluded.
On 19 August 2014 06:58, Marek Polacek pola...@redhat.com wrote:
On Mon, Aug 18, 2014 at 10:57:58PM +0200, Manuel López-Ibáñez wrote:
On 18 August 2014 22:04, Marek Polacek pola...@redhat.com wrote:
+void
+maybe_warn_bool_compare (location_t loc, enum tree_code code, tree op0,
+
Here is the warning after my patch:
1.cpp: In function 'int foo(int, int, int)':
1.cpp:29:70: warning: dereferencing type-punned pointer will break
strict-aliasing rules [-Wstrict-aliasing]
return hash2(static_castconst unsigned short*(change1), len / 2);
On Mon, Aug 18, 2014 at 6:48 PM, VandeVondele Joost
joost.vandevond...@mat.ethz.ch wrote:
ping ?
https://gcc.gnu.org/ml/fortran/2014-05/msg00162.html
Ok, again. If Dominique comes up with some reason why the patch should
be modified, it can be done then. AFAICT your patch is better than the
Thanks, can somebody with svn write access commit ?
Unfortunately the Alpha box I used last time is no longer available and so I've
not been able to try the linker fix. I've not had any responses from anyone
saying they use Ada on Alpha (let alone on VMS - where the original bug was
reported - and Ada/VMS support has now been retired). If none
Unfortunately the Alpha box I used last time is no longer available
and so I've not been able to try the linker fix. I've not had any
responses from anyone saying they use Ada on Alpha (let alone on VMS
- where the original bug was reported - and Ada/VMS support has now
been retired). If none
handle_alias_pairs contains the following thinko.
I believe it should be not a virtual function.
Bootstrapped/regtested on x86_64-linux, ok for trunk?
2014-08-19 Marek Polacek pola...@redhat.com
PR c/61271
* cgraphunit.c (handle_alias_pairs): Fix condition.
diff --git
On Tue, Aug 19, 2014 at 1:07 PM, VandeVondele Joost
joost.vandevond...@mat.ethz.ch wrote:
Thanks, can somebody with svn write access commit ?
Please get yourself write-after-approval access per instructions at
https://gcc.gnu.org/svnwrite.html . You can use me as your sponsor.
Thanks.
--
On 18 Aug 18:24, Bernd Schmidt wrote:
I think I'd be happier if the function was called init_offload_table
and also collected functions, rather than leaving that to
expand_omp_target. I think the patch would be ok with that change.
For the functions it's not so easy to identify which of them
Vector comparisons are sometimes generated with needless 'not' instructions, and
'cmtst' is generally not output at all. This patch makes
gen_aarch64_vcond_internal more intelligent with regard to swapping the operands
to both the comparison and the conditional move, such that not is avoided
Hi all,
I just retested this patch. The crash it fixes is still there,
and the patch still fixes it. Is this ok to commit?
Cheers,
Gary
Andrew Burgess wrote:
In two places when a struct demangle_component is of type
DEMANGLE_COMPONENT_FIXED_TYPE we fall back to accessing the default
On Mon, 18 Aug 2014, Oleg Endo wrote:
On Mon, 2014-08-18 at 16:57 -0400, Hans-Peter Nilsson wrote:
On Mon, 18 Aug 2014, Oleg Endo wrote:
On Sun, 2014-08-17 at 16:56 -0400, Hans-Peter Nilsson wrote:
On Fri, 15 Aug 2014, Oleg Endo wrote:
How about the attached .html as a replacement
On Mon, 2014-07-07 at 11:57 +0200, Richard Biener wrote:
Btw, why doesn't it succeed with LTO? I suspect it's because
we drop the unused variables - try adding __attribute__((used)) to
them.
You are right, this makes the whole new test PASS also with LTO:
2014-08-19 Mark Wielaard
This fixes Honzas previous commit to not build integer_zero_node (what?).
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied.
Richard.
2014-08-19 Richard Biener rguent...@suse.de
* gimple-fold.c (fold_gimple_assign): Properly build a
null-pointer constant when
On Tue, Aug 19, 2014 at 12:25 PM, Marek Polacek pola...@redhat.com wrote:
handle_alias_pairs contains the following thinko.
I believe it should be not a virtual function.
Bootstrapped/regtested on x86_64-linux, ok for trunk?
Ok. Is only trunk affected?
Thanks,
Richard.
2014-08-19 Marek
On Tue, Aug 19, 2014 at 1:05 PM, Mark Wielaard m...@redhat.com wrote:
On Mon, 2014-07-07 at 11:57 +0200, Richard Biener wrote:
Btw, why doesn't it succeed with LTO? I suspect it's because
we drop the unused variables - try adding __attribute__((used)) to
them.
You are right, this makes the
On Tue, Aug 19, 2014 at 01:22:20PM +0200, Richard Biener wrote:
On Tue, Aug 19, 2014 at 12:25 PM, Marek Polacek pola...@redhat.com wrote:
handle_alias_pairs contains the following thinko.
I believe it should be not a virtual function.
Bootstrapped/regtested on x86_64-linux, ok for trunk?
On Tue, Aug 19, 2014 at 1:31 PM, Marek Polacek pola...@redhat.com wrote:
On Tue, Aug 19, 2014 at 01:22:20PM +0200, Richard Biener wrote:
On Tue, Aug 19, 2014 at 12:25 PM, Marek Polacek pola...@redhat.com wrote:
handle_alias_pairs contains the following thinko.
I believe it should be not a
Makes sense, I've attached new patch (retested as usual).
Well, you probably did not test with -m32:
Running target unix/-m32
FAIL: c-c++-common/asan/pr62089.c -O0 output pattern test, is
=
FAIL: c-c++-common/asan/pr62089.c
So the negative version is -Wno-use-no-only? That sounds weird.
What about -Wuse-without-only?
I can also confirm that the test succeeds without the ' ! { dg-bogus has no
ONLY qualifier }'.
Dominique
So the negative version is -Wno-use-no-only? That sounds weird.
What about -Wuse-without-only?
Would be fine with me. Approved with this change ?
On Tue, Aug 19, 2014 at 01:35:20PM +0200, Richard Biener wrote:
On Tue, Aug 19, 2014 at 1:31 PM, Marek Polacek pola...@redhat.com wrote:
On Tue, Aug 19, 2014 at 01:22:20PM +0200, Richard Biener wrote:
On Tue, Aug 19, 2014 at 12:25 PM, Marek Polacek pola...@redhat.com wrote:
On 08/19/2014 12:41 PM, Ilya Verbin wrote:
On 18 Aug 18:24, Bernd Schmidt wrote:
I think I'd be happier if the function was called init_offload_table
and also collected functions, rather than leaving that to
expand_omp_target. I think the patch would be ok with that change.
For the functions
On Tue, 2014-08-19 at 13:22 +0200, Richard Biener wrote:
On Tue, Aug 19, 2014 at 1:05 PM, Mark Wielaard m...@redhat.com wrote:
On Mon, 2014-07-07 at 11:57 +0200, Richard Biener wrote:
Btw, why doesn't it succeed with LTO? I suspect it's because
we drop the unused variables - try adding
On Tue, Aug 19, 2014 at 2:10 PM, Mark Wielaard m...@redhat.com wrote:
On Tue, 2014-08-19 at 13:22 +0200, Richard Biener wrote:
On Tue, Aug 19, 2014 at 1:05 PM, Mark Wielaard m...@redhat.com wrote:
On Mon, 2014-07-07 at 11:57 +0200, Richard Biener wrote:
Btw, why doesn't it succeed with LTO?
Hi,
two more functions. Tested x86_64-linux.
Thanks,
Paolo.
///
2014-08-19 Paolo Carlini paolo.carl...@oracle.com
* parser.c (cp_parser_assignment_expression,
cp_parser_constant_expression): Add default arguments.
(cp_parser_primary_expression,
On 19 Aug 13:55, Bernd Schmidt wrote:
On 08/19/2014 12:41 PM, Ilya Verbin wrote:
For the functions it's not so easy to identify which of them to add into the
table, e.g.:
#pragma omp target
#pragma omp parallel
x++;
Here 2 functions with omp declare target attribute are
On 08/19/2014 03:37 PM, Dominique Dhumieres wrote:
Makes sense, I've attached new patch (retested as usual).
Well, you probably did not test with -m32:
Right, I only did plain make check until now. Thanks for reporting,
I'll check this tomorrow.
-Y
This makes us accept
(plus @0 INTEGER_CST)
and parse it as (plus @0 (INTEGER_CST)), that is, handle operator
names in predicate position as expression with zero operands.
This makes those leafs better integrate with the decision tree
and also removes the need to define all those INTEGER_CST_P
Applied.
Richard.
2014-08-19 Richard Biener rguent...@suse.de
* genmatch.c (c_expr::output_code): New method.
(c_expr::gen_transform): Call it.
(outline_c_exprs): Likewise.
Index: gcc/genmatch.c
===
---
On 11/08/14 18:34, Mike Stump wrote:
On Aug 11, 2014, at 2:06 AM, Richard Earnshaw rearn...@arm.com wrote:
Not quite, read the subject line again.
Doh. I did miss that entirely. The solutions I gave were for other cases than
the case at hand.
I'm not sure what the correct change to the
VandeVondele Joost wrote:
So the negative version is -Wno-use-no-only? That sounds weird.
What about -Wuse-without-only?
Would be fine with me. Approved with this change ?
That would be the patch: https://gcc.gnu.org/ml/fortran/2014-06/msg00114.html
+Warn if a use statement has no only
On 19 Aug 13:55, Bernd Schmidt wrote:
In any case, I think let's get this checked in for now and iterate later.
Committed revision 214148.
-- Ilya
On 18/08/14 19:52, Richard Henderson wrote:
On 08/18/2014 05:24 AM, Kyrill Tkachov wrote:
-(define_insn *csinc2mode_insn
+(define_insn *csinc2mode_CC_ZERO:mode_insn
[(set (match_operand:GPI 0 register_operand =r)
(plus:GPI (match_operator:GPI 2 aarch64_comparison_operator
-
On Sat, Aug 16, 2014 at 11:21 PM, Marc Glisse marc.gli...@inria.fr wrote:
Hello,
here is a patch extending DSE to handle calls to memset, memcpy and memmove.
A number of alias functions didn't have a version taking an ao_ref*, so I
added those. Instead of naming things _1, _2, etc I took
This fixes Honzas previous commit to not build integer_zero_node (what?).
Oops, thanks!
Honza
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied.
Richard.
2014-08-19 Richard Biener rguent...@suse.de
* gimple-fold.c (fold_gimple_assign): Properly build a
The vector compare intrinsics (vc[gl][et]z, vceqz, vtst) were written using
__builtin functions as (IIUC) at the time gcc vector extensions did not support
comparison ops across both C and C++ frontends. These have since been updated.
Following the first patch, we now get equal/better code
on the outermost level.
Committed.
Richard.
2014-08-19 Richard Biener rguent...@suse.de
* genmatch.c (decision_tree::gen_gimple): Use a switch statement
for the outermost level.
* gimple-match.h (code_helper::get_rep): New method.
Index: gcc/genmatch.c
Ping.
-Maxim
Original Message
Subject:[PATCH] Add patch for debugging compiler ICEs.
Date: Mon, 04 Aug 2014 21:03:22 +0400
From: Maxim Ostapenko m.ostape...@partner.samsung.com
To: GCC Patches gcc-patches@gcc.gnu.org
CC: Jeff Law l...@redhat.com, Jakub Jelinek
On 08/18/2014 10:51 AM, Kito Cheng wrote:
Hi Vladimir:
Here is a tiny typo in comment, allono - allocno.
ChangLog
2014-08-18 Kito Cheng k...@0xlab.org
* ira.c: Fix typo in comment.
Thanks, Kito. Of course, the patch is ok. You can commit it if it is
not committed yet.
On Tue, Aug 19, 2014 at 11:08:39AM +0200, Manuel López-Ibáñez wrote:
Not sure about that: it matters whether the CST is a LHS or a RHS
- because we want to say if the comparison is always true or false.
I tried to introduce some bool flag, but that didn't really help
readability IMHO.
On some archs, C[TL]Z_DEFINED_VALUE_AT_ZERO macros return only
true/false, so -Wbool-compare would warn. But on e.g. mips or
aarch64 they might yield 2. This patch casts the value to int
to quash that warning. Dropping the == 2 would be prettier,
but I don't want to break other archs.
The point
On Tue, 19 Aug 2014, Richard Biener wrote:
/* Return true whether REF may refer to global memory. */
bool
-ref_may_alias_global_p (tree ref)
+ref_may_alias_global_p (ao_ref *ref)
{
- tree base = get_base_address (ref);
+ tree base = ao_ref_base (ref);
if (DECL_P (base))
return
On Mon, Aug 18, 2014 at 2:07 PM, Joseph S. Myers
jos...@codesourcery.com wrote:
On Sat, 16 Aug 2014, Patrick Palka wrote:
2014-08-17 Patrick Palka ppa...@gcc.gnu.org
* c-typeck.c (build_c_cast): Do a conversion even when the
TYPE_MAIN_VARIANTs are the same.
2014-08-17
Hi Richard:
Hmm, I'm not sure about this. It might not be used at present, but on:
AArch64, with more call-clobbered registers than call-saved registers, I
would expect this ought to be a win. The fact that it isn't on today
may say more about the way it works than the concept that it's the
Applied for you at rev. 214153.
Kai
2014-08-12 20:37 GMT+02:00 Kai Tietz ktiet...@googlemail.com:
2014-08-12 20:34 GMT+02:00 Yaakov Selkowitz yselk...@redhat.com:
There is a syntax error in r213009 causing a FTBFS in trunk with
--target=i686-pc-cygwin. Patch attached, with which trunk now
On 08/19/2014 07:12 AM, Marek Polacek wrote:
On some archs, C[TL]Z_DEFINED_VALUE_AT_ZERO macros return only
true/false, so -Wbool-compare would warn.
Then we should fix them to return 0/1 instead.
r~
Committed.
Richard.
2014-08-19 Richard Biener rguent...@suse.de
* genmatch.c (dt_operand::gen_gimple): Remove excessive
braces.
(dt_operand::gen_generic): Likewise.
Index: gcc/genmatch.c
===
---
Applied for you at revision 214154.
Kai
On Tue, 2014-08-12 at 15:20 -0600, Jeff Law wrote:
On 08/06/14 11:19, David Malcolm wrote:
Yet more scaffolding: convert the NEXT_INSN/PREV_INSN macros
and their SET_* variants into functions.
Convert the rvalue-style functions into returning
rtx_insn * rather than plain rtx.
For
Applied for you at revision 214156.
Kai
Hi all,
This patch series converts the arm backend to output unified assembly
syntax for the VFP instructions.
This makes it more readable since most UAL mnemonics also include
various type suffixes such as .f32 and .f64 that quickly allow to
identify the data types being operated on.
Each
Applied for you at revision 214158.
Kai
Hi all,
This patch switches over the FP mul+add, mul+neg+add etc patterns.
Mixing some of the mnemonics up is a danger but the ARM ARM has a handy
section that maps each mnemonic to its UAL equivalent.
Ok for trunk?
Thanks,
Kyrill
2014-08-19 Kyrylo Tkachov kyrylo.tkac...@arm.com
*
Hi all,
These are the fp-fp and int-fp convert patterns. IMHO these are much
more readable than the pre-UAL ones because they use two suffixes to
indicate which data types are being converted to and from.
Ok for trunk?
Thanks,
Kyrill
2014-08-19 Kyrylo Tkachov kyrylo.tkac...@arm.com
Hi all,
The sqrt and floating point compare patterns.
For the case when we compare with floating-point 0 the ARM ARM uses the
syntax 'vcmp {s,d}n, #0.0'
but current gas has a bug and doesn't accept that form, only 'vcmp
{s,d}n, #0'
There is a gas patch in review on the binutils mailing list
Hi all,
The ARM ARM explicitly says that fmstat is translated into 'vmrs
APSR_nzcvm, FPSCR' in UAL syntax so this patch does that.
Ok for trunk?
Thanks,
Kyrill
2014-08-19 Kyrylo Tkachov kyrylo.tkac...@arm.com
* config/arm/vfp.md (*movcc_vfp): Use UAL syntax.commit
Hi all,
This patch updates some FP load/store multiple patterns and in the
vfp_output_vstmd case when the instruction is used as a push to sp it
now emits a vpush mnemonic instead of vstmdb.
Ok for trunk?
Thanks,
Kyrill
2014-08-19 Kyrylo Tkachov kyrylo.tkac...@arm.com
*
Manuel López-Ibáñez lopeziba...@gmail.com writes:
This patch is in preparation for further patches moving the Fortran FE
to use the common diagnostics machinery.
Fortran has its own way of printing the caret information, so we need
a way to override the default in the diagnostics machinery.
Committed revision 214159.
2014-08-19 Joost VandeVondele vond...@gcc.gnu.org
* MAINTAINERS (Write After Approval): Add myself.
Index: MAINTAINERS
===
--- MAINTAINERS (revision 214158)
+++ MAINTAINERS (working copy)
@@
Hi all,
Nothing too controversial here, convert the concerned patterns to UAL.
The size of the data types in the operation is expressed in the .f32 or
.f64 suffix.
Ok for trunk?
Thanks,
Kyrill
2014-08-19 Kyrylo Tkachov kyrylo.tkac...@arm.com
* config/arm/vfp.md (*abssf2_vfp): Use UAL
On Tue, 2014-08-12 at 15:20 -0600, Jeff Law wrote:
On 08/06/14 11:19, David Malcolm wrote:
DF_REF_INSN looks up the insn field of the referenced df_insn_info.
This will eventually be an rtx_insn *, but for now is just an rtx.
As further scaffolding: for now, convert DF_REF_INSN to a
Hi all,
In this patch the move patterns are updated.
For the fconst case where the constant is encoded in a decimal
representation before going into the immediate field of the assembly
instruction UAL syntax allows for the real operand to be output directly
and leaves the assembler to do the
On 08/19/2014 06:29 AM, Kyrill Tkachov wrote:
+(define_special_predicate cc_register_zero
+ (and (match_code reg)
+ (and (match_test REGNO (op) == CC_REGNUM)
+ (ior (match_test mode == GET_MODE (op))
+ (ior (match_test mode == VOIDmode
+
Patch is ok. Applied this patch at revision 214162 together with your
followup-patch at revision 214161.
Thanks,
Kai
Patch is ok. Applied this patch at revision 214161 together with your
followup-patch at revision 214162.
Thanks,
Kai
Applied at revision 214163.
Thanks,
Kai
On Tue, 2014-08-12 at 15:21 -0600, Jeff Law wrote:
On 08/06/14 11:19, David Malcolm wrote:
For now, convert DEP_PRO and DEP_CON into functions. We will eventually
change them back to macros once the relevant fields are of type
rtx_insn *.
gcc/
* sched-int.h (DEP_PRO): struct
Excuse me, I did not finish in time. At present, I shall be mainly focus
on testsuite. Before I finish it, I can do nothing (I should do nothing)
for this patch.
Originally:
- I get a PC (not buy), it is low quality which cause reboot after run
several hours with full cpu used, I wasted
On Tue, 2014-08-12 at 15:21 -0600, Jeff Law wrote:
On 08/06/14 11:19, David Malcolm wrote:
For now, convert into VINSN_INSN_RTX a pair of functions. We will
eventually change them back to a macro once the relevant field is of type
rtx_insn *.
gcc/
* sel-sched-ir.h
I noticed a typo in a diagnostic message. No test needs
adjusting.
Bootstrapped/regtested on x86_64-linux, applying to trunk.
2014-08-19 Marek Polacek pola...@redhat.com
* lex.c (_cpp_lex_direct): Fix a typo.
diff --git gcc/lex.c gcc/lex.c
index 0713f65..827cfb0 100644
--- gcc/lex.c
On 19/08/14 16:25, Richard Henderson wrote:
On 08/19/2014 06:29 AM, Kyrill Tkachov wrote:
+(define_special_predicate cc_register_zero
+ (and (match_code reg)
+ (and (match_test REGNO (op) == CC_REGNUM)
+ (ior (match_test mode == GET_MODE (op))
+(ior (match_test
Committed as revision 214157.
2014-08-19 Christopher Faylor me@cgf.cx
* MAINTAINERS: Remove myself
Index: MAINTAINERS
===
--- MAINTAINERS (revision 214153)
+++ MAINTAINERS (working copy)
@@ -136,7 +136,6 @@
VMS
On 08/19/2014 08:54 AM, Marek Polacek wrote:
Works as well. So is the following ok once the regtest finishes?
Bootstrapped on x86_64-linux.
2014-08-19 Marek Polacek pola...@redhat.com
* config/alpha/alpha.h (CLZ_DEFINED_VALUE_AT_ZERO,
CTZ_DEFINED_VALUE_AT_ZERO): Return
On Tue, 2014-08-12 at 15:22 -0600, Jeff Law wrote:
On 08/06/14 11:19, David Malcolm wrote:
gcc/
* sel-sched-ir.h (BB_NOTE_LIST): struct sel_region_bb_info_def's
note_list field will eventually be an rtx_insn *. To help with
transition, for now, convert from an access macro
(define_special_predicate cc_register_zero
(match_code reg)
{
return (REGNO (op) == CC_REGNUM
(GET_MODE (op) == CCmode
|| GET_MODE (op) == CC_Zmode
|| GET_MODE (op) == CC_NZmode));
})
... and now that I read the backend more closely, I see _zero
On Tue, Aug 19, 2014 at 07:52:33AM -0700, Richard Henderson wrote:
On 08/19/2014 07:12 AM, Marek Polacek wrote:
On some archs, C[TL]Z_DEFINED_VALUE_AT_ZERO macros return only
true/false, so -Wbool-compare would warn.
Then we should fix them to return 0/1 instead.
Works as well. So is the
On Mon, Aug 18, 2014 at 08:51:35AM -0700, H.J. Lu wrote:
When --enable-gold=no is used to configure gcc, we will see
configure: line 21572: test: =: unary operator expected
I checked in this patch to set install_gold_as_default to no for
--enable-gold=no. Tested on Linux/x86-64.
The
On Mon, Aug 18, 2014 at 12:29 PM, Uros Bizjak ubiz...@gmail.com wrote:
On Mon, Aug 18, 2014 at 9:16 PM, H.J. Lu hjl.to...@gmail.com wrote:
Attached patch fixes the problem with false data dependency on output
register for popcnt, lzcnt and tzcnt insns on sandybridge and haswell
targets.
The
On Aug 19, 2014, at 6:12 AM, Kyrill Tkachov kyrylo.tkac...@arm.com wrote:
So how about this?
Ok. Thanks.
On Tue, 2014-08-12 at 15:22 -0600, Jeff Law wrote:
On 08/06/14 11:19, David Malcolm wrote:
gcc/
* sel-sched-ir.h (BND_TO): insn_t will eventually be an
rtx_insn *. To help with transition, for now, convert from an
access macro into a pair of functions: BND_TO, returning an
On 19/08/14 17:09, Richard Henderson wrote:
(define_special_predicate cc_register_zero
(match_code reg)
{
return (REGNO (op) == CC_REGNUM
(GET_MODE (op) == CCmode
|| GET_MODE (op) == CC_Zmode
|| GET_MODE (op) == CC_NZmode));
})
... and now that I
On Mon, Aug 18, 2014 at 10:23:49PM +0200, Marek Polacek wrote:
On Mon, Aug 18, 2014 at 04:10:49PM -0400, Jason Merrill wrote:
On 08/18/2014 04:04 PM, Marek Polacek wrote:
Unfortunately, this warning cannot be enabled by -Wall yet, because
of a few blunders we have in the codebase. But we
Ping.
On 06-08-2014 17:21, Adhemerval Zanella wrote:
On 01-08-2014 12:31, Joseph S. Myers wrote:
On Thu, 31 Jul 2014, David Edelsohn wrote:
Thanks for implementing the FENV support. The patch generally looks
good to me.
My one concern is a detail in the implementation of update. I do not
On Tue, 2014-08-12 at 21:07 -0600, Jeff Law wrote:
On 08/06/14 11:19, David Malcolm wrote:
gcc/
* coretypes.h (class rtx_real_insn): Add forward declaration.
(class rtx_debug_insn): Likewise.
(class rtx_nonjump_insn): Likewise.
(class rtx_jump_insn): Likewise.
This patch adds the missing vrbit_p8 and vrbitq_p8 intrinsics to arm_neon.h, and
implements all the vrbit(q?)_[psu]8 intrinsics using a new builtin, rather than
the previous temporary asm. Also adds a testcase checking (a) execution results
and (b) that we output rbit vXX.8b,vYY.8b or
My recent patch broke bootstrap on ppc64, because, by default,
char on ppc defaults to be an unsigned char. But the code relied
on char being signed by default.
Furthermore, the compat warning about // comments shouldn't be issued
in C++ mode at all.
Sorry about that.
Bootstrapped on
Another needed tweak, applying to trunk.
commit aac24a75af315fb6aa6099a159c6bce339706bc1
Author: Jason Merrill ja...@redhat.com
Date: Mon Aug 18 21:34:12 2014 -0400
PR lto/53808
PR c++/61659
* decl.c (maybe_commonize_var): Don't use DECL_COMDAT to trigger
comdat_linkage.
This patch adds checks for user-defined logical operators during
constraint normalization and ensures that all atomics can be converted
to bool.
2014-08-14 Andrew Sutton andrew.n.sut...@gmail.com
Implement normalization checks.
* gcc/cp/constraint.cc (normalize_expr): Delegate
On Tue, 2014-08-12 at 15:59 -0600, Jeff Law wrote:
On 08/06/14 11:19, David Malcolm wrote:
These should all eventually require an rtx_insn * as an argument,
but we'll save that for a later patch.
gcc/
* rtl.h (previous_insn): Strengthen return type from rtx to
rtx_insn *.
On 08/19/2014 12:52 PM, Marek Polacek wrote:
+ tree cst = TREE_CODE (op0) == INTEGER_CST
+? op0 : TREE_CODE (op1) == INTEGER_CST ? op1 : NULL_TREE;
This indentation won't survive Emacs auto-indent; please add parentheses.
+ int sign = TREE_CODE (op0) == INTEGER_CST
+
On Tue, 2014-08-19 at 14:16 +0200, Richard Biener wrote:
On Tue, Aug 19, 2014 at 2:10 PM, Mark Wielaard m...@redhat.com wrote:
gcc/testsuite/ChangeLog
2014-08-19 Mark Wielaard m...@redhat.com
* gcc.dg/guality/restrict.c: Add `used' attribute to all variables.
(cpy): Change
The idea is that when we see a change of classification, and the
option was originally unspecified, we record the command-line status.
This way, when doing pop later, we already check if the option was
reclassified so we get the command-line status. This also works with
-Wall, since all dependent
On 08/06/2014 10:19 AM, David Malcolm wrote:
@@ -2772,11 +2772,11 @@ mn10300_adjust_sched_cost (rtx insn, rtx link, rtx
dep, int cost)
if (!TARGET_AM33)
return 1;
- if (GET_CODE (insn) == PARALLEL)
-insn = XVECEXP (insn, 0, 0);
+ if (GET_CODE (PATTERN (insn)) == PARALLEL)
On Tue, 2014-08-12 at 16:00 -0600, Jeff Law wrote:
On 08/06/14 11:19, David Malcolm wrote:
gcc/
* rtl.h (gen_label_rtx): Strengthen return type from rtx to
rtx_code_label *.
* emit-rtl.c (gen_label_rtx): Likewise.
Presumably at some point we'll look at the gen_XXX and
On Tue, Aug 19, 2014 at 01:41:44PM -0400, Jason Merrill wrote:
On 08/19/2014 12:52 PM, Marek Polacek wrote:
+ tree cst = TREE_CODE (op0) == INTEGER_CST
+ ? op0 : TREE_CODE (op1) == INTEGER_CST ? op1 : NULL_TREE;
This indentation won't survive Emacs auto-indent; please add
1 - 100 of 154 matches
Mail list logo
