On Wed, Aug 20, 2014 at 1:24 PM, Roman Gareev wrote:
> Which configure options do you use? Have you tried to bootstrap
> r214105 with same configure options?
> Have you tried to manually export the path to isl to LD_RUN_PATH?
> (You've probably tried. I just want to make sure.)
I can't access the
Hi there,
The gcc now uses immediate_operand for const_double_operand, update this
split pattern accordingly. Tested with gcc regression test on thumb1 target,
no regression. Is it ok to trunk?
BR,
Terry
2014-08-20 Terry Guo
* config/arm/thumb1.md (64bit splitter): Replace const_double_
Which configure options do you use? Have you tried to bootstrap
r214105 with same configure options?
Have you tried to manually export the path to isl to LD_RUN_PATH?
(You’ve probably tried. I just want to make sure.)
2014-08-20 9:28 GMT+06:00 Bin.Cheng :
> I suspect this causes arm/aarch64 nativ
Ping.
On Wed, Aug 13, 2014 at 01:33:08PM +0200, Marek Polacek wrote:
> I've put together a few lines describing what I (except
> -fsanitize=alignment) implemented for GCC 5.
> It's the file wwwdocs/htdocs/gcc-5/changes.html.
>
> Ok?
>
> --- changes.html.mp 2014-08-13 12:21:57.059610612 +0200
>
I suspect this causes arm/aarch64 native bootstrap failure with below messages.
aarch64-none-linux-gnu-g++ -c -g -O2 -DIN_GCC-fno-exceptions
-fno-rtti -fasynchronous-unwind-tables -W -Wall -Wno-narrowing
-Wwrite-strings -Wcast-qual -Wmissing-format-attribute
-Woverloaded-virtual -pedantic -W
On 08/19/2014 10:17 AM, Marek Polacek wrote:
> 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
On Wed, 2014-08-20 at 00:15 +0200, Tobias Burnus wrote:
> I am not sure about which patch causes the problem, but with Rev. 214197
> boostrapping fails here (x86-64-gnu-linux) while an incremental build
> with a few other of your changes it worked.
>
> Currently, it fails in Stage 1 with the Sta
This patch makes -fcall-saved-rX for X > 7 on Thumb target when optimizing for
size. It works by adding a new field x_user_set_call_save_regs in struct
target_hard_regs to track whether an entry in fields x_fixed_regs,
x_call_used_regs and x_call_really_used_regs was user set or is in its defaul
BTW, here is one particular kind of static analysis which I am very
much interested in:
https://code.google.com/p/address-sanitizer/wiki/CompileTimeOptimizations#Escape_analysis_for_use-after-return
Here we can prove that a does not escape foo() and thus we may avoid
putting it on fake stack.
int
On 08/19/2014 02:35 PM, David Malcolm wrote:
> This one is quite ugly: the pre-existing code has two locals named
> "note", both of type rtx, with one shadowing the other. This patch
> introduces a third, within the scope where the name "note" is used for
> insns. In the other scopes the two othe
On Mon, Aug 18, 2014 at 1:05 PM, Yuri Gribov wrote:
> On Fri, Aug 15, 2014 at 10:34 PM, Konstantin Serebryany
> wrote:
>> If this is -O1 or higher, then most (but not all) of your cases
>> *should* be optimized by the compiler before asan kicks in.
>
> You mean hoisting memory accesses from bran
Hi
Current AArch64 backend can generate rtl expressions like
(vec_duplicate:DI (const_int 0 [0])), which causes ICE in
simplify_const_unary_operation because vec_duplicate should generate
vector mode only.
As suggested by Andrew in the bug entry, I split the original insn
patterns to avoid scalar
On Wed, 20 Aug 2014, Manuel L?pez-Ib??ez wrote:
> The fix is almost trivial. Just use the appropriate flags when warning.
>
> Bootstrapped and regression tested on x86_64-linux-gnu.
>
> OK?
OK.
--
Joseph S. Myers
jos...@codesourcery.com
I am not sure about which patch causes the problem, but with Rev. 214197
boostrapping fails here (x86-64-gnu-linux) while an incremental build
with a few other of your changes it worked.
Currently, it fails in Stage 1 with the Stage 1 compiler with the
following message.
Tobias
../../../..
The fix is almost trivial. Just use the appropriate flags when warning.
Bootstrapped and regression tested on x86_64-linux-gnu.
OK?
gcc/ChangeLog:
2014-08-20 Manuel López-Ibáñez
* incpath.c (remove_duplicates): Use cpp_warning.
gcc/c-family/ChangeLog:
2014-08-20 Manuel López-Ibáñez
On Tue, 19 Aug 2014, Manuel L?pez-Ib??ez wrote:
> The following patch adds a new CPP() to *.opt files that tells the
> options machinery to keep in sync libcpp options and GCC options.
>
> This will allow us to fix a lot of latent bugs where CPP warnings do
> not follow the rules of other warning
On Fri, 15 Aug 2014, Sylvestre Ledru wrote:
> It is indeed useless. I removed it. Thanks
> http://sylvestre.ledru.info/0001-Enable-warning-Wreturn-type-by-default.patch
I don't think most of the testsuite changes in this patch should be
needed, and we should be conservative about changing existi
On Tue, 2014-08-19 at 13:57 -0700, Richard Henderson wrote:
> On 08/06/2014 10:23 AM, David Malcolm wrote:
> > diff --git a/gcc/cfgrtl.c b/gcc/cfgrtl.c
> > index 59d633d..5e42a97 100644
> > --- a/gcc/cfgrtl.c
> > +++ b/gcc/cfgrtl.c
> > @@ -1604,6 +1604,7 @@ force_nonfallthru_and_redirect (edge e, b
I was pinged about the lack of documentation for the
check_effective_target_non_strict_align procedure I added
to target-supports.exp. This patch fixes that. If I
don't get any objections in the next day or two I will
check this in as an obvious patch.
Steve Ellcey
sell...@mips.com
2014-08-19
On Tue, 2014-08-19 at 13:56 -0700, Richard Henderson wrote:
> On 08/06/2014 10:23 AM, David Malcolm wrote:
> >else if (computed_jump_p (insn))
> > {
> > for (rtx_expr_list *lab = forced_labels; lab; lab = lab->next ())
> > - maybe_record_trace_start (lab->element (), insn);
On Wed, 2014-08-13 at 11:56 -0600, Jeff Law wrote:
> On 08/06/14 11:20, David Malcolm wrote:
> > gcc/
> > * rtl.h (unlink_insn_chain): Strengthen return type from rtx to
> > rtx_insn *.
> > (duplicate_insn_chain): Likewise.
> > * cfgrtl.c (unlink_insn_chain): Strengthen return type
On Wed, 2014-08-13 at 11:55 -0600, Jeff Law wrote:
> On 08/06/14 11:20, David Malcolm wrote:
> > gcc/
> > * rtl.h (next_cc0_user): Strengthen return type from rtx to
> > rtx_insn *.
> > (prev_cc0_setter): Likewise.
> >
> > * emit-rtl.c (next_cc0_user): Strengthen return type from rt
On 08/06/2014 10:23 AM, David Malcolm wrote:
> This patch updates NEXT_INSN and PREV_INSN to work on rtx_insn *, rather
> than plain rtx - plus miscellaneous fixes needed to get there.
Ug. Bigger than I'd have liked, but still ok.
r~
On 08/06/2014 10:23 AM, David Malcolm wrote:
> /
> rtx-classes-status.txt: Delete
> ---
> rtx-classes-status.txt | 9 -
> 1 file changed, 9 deletions(-)
> delete mode 100644 rtx-classes-status.txt
#230 - #236 are ok.
r~
On 08/06/2014 10:23 AM, David Malcolm wrote:
> gcc/
> * rtl.h (tablejump_p): Strengthen first param from const_rtx to
> const rtx_insn *.
> (label_is_jump_target_p): Likewise for second param.
>
> * rtlanal.c (tablejump_p): Likewise for param "insn".
> (label_is_jump_
On Aug 18, 2014, at 4:48 PM, Janis Johnson wrote:
> Tested with gcc.target/neon tests for mainline arm-none-eabi with many
> sets of multilib options; OK for mainline and 4.9?
Ok.
On 08/06/2014 10:23 AM, David Malcolm wrote:
> gcc/
> * output.h (insn_current_reference_address): Strengthen param
> from rtx to rtx_insn *.
> * final.c (insn_current_reference_address): Likewise.
#223 and #224 are ok.
r~
On 08/06/2014 10:23 AM, David Malcolm wrote:
> diff --git a/gcc/cfgrtl.c b/gcc/cfgrtl.c
> index 59d633d..5e42a97 100644
> --- a/gcc/cfgrtl.c
> +++ b/gcc/cfgrtl.c
> @@ -1604,6 +1604,7 @@ force_nonfallthru_and_redirect (edge e, basic_block
> target, rtx jump_label)
>
>if (EDGE_COUNT (e->src->s
On Wed, 2014-08-13 at 11:54 -0600, Jeff Law wrote:
> On 08/06/14 11:20, David Malcolm wrote:
> > gcc/
> > * expr.h (emit_move_insn): Strengthen return type from rtx to
> > rtx_insn *.
> > (emit_move_insn_1): Likewise.
> > (emit_move_complex_push): Likewise.
> > (emit_move_comple
On 08/06/2014 10:23 AM, David Malcolm wrote:
>else if (computed_jump_p (insn))
> {
> for (rtx_expr_list *lab = forced_labels; lab; lab = lab->next ())
> - maybe_record_trace_start (lab->element (), insn);
> + maybe_record_trace_start (lab->insn (), insn);
>
On 08/06/2014 10:23 AM, David Malcolm wrote:
> gcc/
> * function.h (struct rtl_data): Strengthen fields "x_return_label"
> and "x_naked_return_label" from rtx to rtx_code_label *.
> ---
> gcc/function.h | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
#207 - #220 are all OK
On 08/06/2014 10:23 AM, David Malcolm wrote:
> gcc/
> * rtl.h (rtx_expr_list::insn): New method.
> ---
> gcc/rtl.h | 9 +
> 1 file changed, 9 insertions(+)
>
> diff --git a/gcc/rtl.h b/gcc/rtl.h
> index d028be1..d5811c2 100644
> --- a/gcc/rtl.h
> +++ b/gcc/rtl.h
> @@ -414,6 +414,10
Here is an official submission for the switch optimization described in
PR 54742. I have addressed the formatting/comment issues that were raised
and also added a test case based on comment #27 from PR 54742 and I fixed a
bug I found while doing benchmarking with SPEC2006 (the perl benchmark was
g
On Wed, 2014-08-13 at 11:52 -0600, Jeff Law wrote:
> On 08/06/14 11:20, David Malcolm wrote:
> > More scaffolding: strengthen the return types from the various emit_
> > functions from rtx to rtx_insn * (or to the rtx_barrier * subclass in a
> > few cases).
> >
> > These will ultimately have their
On Wed, 2014-08-13 at 07:45 -0600, Jeff Law wrote:
> On 08/06/14 11:20, David Malcolm wrote:
> > gcc/
> > * emit-rtl.c (emit_jump_table_data): Strengthen return type from
> > rtx to rtx_jump_table_data *. Also for local.
> > * rtl.h (emit_jump_table_data): Likwise.
> OK.
Thanks. Comm
On Wed, 2014-08-06 at 13:20 -0400, David Malcolm wrote:
> gcc/
> * basic-block.h (create_basic_block_structure): Strengthen third
> param "bb_note" from rtx to rtx_note *.
> * rtl.h (emit_note_before): Strengthen return type from rtx to
> rtx_note *.
> (emit_note_after
Any news regarding this proposal ?
Thanks
François
On 30/07/2014 23:39, François Dumont wrote:
Hi
Now that patch on testsuite allocator is in I would like to
reactivate this one. Here it is again.
See previous answer below regarding modification of
_M_begin/_M_cbegin.
201
On Wed, 2014-08-13 at 07:44 -0600, Jeff Law wrote:
> On 08/06/14 11:20, David Malcolm wrote:
> > gcc/
> > * function.h (struct rtl_data): Strengthen field
> > "x_parm_birth_insn" from rtx to rtx_insn *.
> > * function.c (struct assign_parm_data_all): Strengthen fields
> > "first_con
On Wed, 2014-08-13 at 07:42 -0600, Jeff Law wrote:
> On 08/06/14 11:20, David Malcolm wrote:
> > gcc/
> > * cfgexpand.c (expand_used_vars): Strengthen return type from rtx
> > to rtx_insn *; also for local "var_end_seq".
> > (maybe_dump_rtl_for_gimple_stmt): Likewise for param "since".
On Tue, 2014-08-12 at 21:56 -0600, Jeff Law wrote:
> On 08/06/14 11:20, David Malcolm wrote:
> > gcc/
> > * asan.h (asan_emit_stack_protection): Strengthen return type from
> > rtx to rtx_insn *.
> > * asan.c (asan_emit_stack_protection): Likewise. Add local
> > "insns" to hold the
On Tue, 2014-08-12 at 21:55 -0600, Jeff Law wrote:
> On 08/06/14 11:20, David Malcolm wrote:
> > gcc/
> > * basic-block.h (bb_note): Strengthen return type from rtx to rtx_note
> > *.
> > * sched-int.h (bb_note): Likewise.
> > * cfgrtl.c (bb_note): Likewise. Add a checked cast to rtx_
Ping, pretty please.
--
Evandro Menezes Austin, USA
e.mene...@samsung.com +1-512-425-3365
-Original Message-
From: Evandro Menezes [mailto:e.mene...@samsung.com]
Sent: Monday, August 18, 2014 10:02
To: 'James Greenhalgh'
Cc: 'gcc-patches@gcc
On Tue, 2014-08-12 at 21:54 -0600, Jeff Law wrote:
> On 08/06/14 11:20, David Malcolm wrote:
> > Doing so means strengthening the types of the make_raw callbacks within
> > emit-rtl.c from rtx to rtx_insn * as used by the emit_pattern_*
> > internal functions. There's more that could be done in te
On Tue, 2014-08-12 at 21:52 -0600, Jeff Law wrote:
> On 08/06/14 11:20, David Malcolm wrote:
> > gcc/
> > * emit-rtl.c (last_call_insn): Strengthen return type from rtx to
> > rtx_call_insn *.
> > * rtl.h (is_a_helper ::test): New overload, accepting
> > an rtx_insn *.
> > (last
On Tue, 2014-08-19 at 07:02 -0400, Hans-Peter Nilsson wrote:
> 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
On 19 August 2014 21:21, Tobias Burnus wrote:
> [Side remark: By itself, using "^" would be fine also for gfortran; however,
> it uses a digit like "1" because it also has a few error messages of the
> kind "Duplicate statement label %d at %L and %L", which uses two locations
> with the labels "1"
On Tue, 2014-08-12 at 21:51 -0600, Jeff Law wrote:
> On 08/06/14 11:20, David Malcolm wrote:
> > gcc/
> > * rtl.h (delete_trivially_dead_insns): Strengthen initial param
> > "insns" from rtx to rtx_insn *.
> > * cse.c (delete_trivially_dead_insns): Likewise, also do it for
> > local
On Tue, 2014-08-12 at 21:51 -0600, Jeff Law wrote:
> On 08/06/14 11:20, David Malcolm wrote:
> > gcc/
> > * rtl.h (tablejump_p): Strengthen third param from rtx * to
> > rtx_jump_table_data **.
> >
> > * cfgbuild.c (make_edges): Introduce local "table", using it in
> > place of "tmp
Hi,
Manuel López-Ibáñez wrote:
This patch is relative to this one here:
https://gcc.gnu.org/ml/gcc-patches/2014-08/msg01652.html
It implements the Fortran style of prefix and caret line in the
gfc_diagnostic_starter by using the common pretty-printer.
Looks good to me. Thanks for the patch!
Sorry for the late reply. I took some time to make myself more
familiar with NORETURN and related code, and finally I understood what
you mean and saw why only GF_CALL_CTRL_ALTERING was enough and
GF_CALL_NORETURN was unneeded. With your suggestion, the change looks
much briefer! Please check if th
On Tue, 2014-08-12 at 21:04 -0600, Jeff Law wrote:
> On 08/06/14 11:20, David Malcolm wrote:
> > gcc/
> > * rtl.h (entry_of_function): Strengthen return type from rtx to
> > rtx_insn *.
> > * cfgrtl.c (entry_of_function): Likewise.
> OK.
Thanks; committed to trunk as r214182.
Hi,
This patch is relative to this one here:
https://gcc.gnu.org/ml/gcc-patches/2014-08/msg01652.html
It implements the Fortran style of prefix and caret line in the
gfc_diagnostic_starter by using the common pretty-printer.
This requires making configurable the caret character in the
diagnostic
The C++ front end already means to clear DECL_EXTERNAL on callable
functions, we just need to update that for devirtualization.
Tested x86_64-pc-linux-gnu, applying to trunk.
commit a87e57625b130bb9e1fec2130643a4e5d7a3b807
Author: Jason Merrill
Date: Tue Aug 19 13:52:25 2014 -0400
PR tr
On Tue, 2014-08-12 at 16:01 -0600, Jeff Law wrote:
> On 08/06/14 11:19, David Malcolm wrote:
> > Ultimately, the underlying fields should become rtx_insn *, but for now we
> > can do this with a checked cast.
> >
> > Note to self:
> >config/m32c/m32c.c: m32c_leaf_function_p directly manipulates
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 pare
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_XX
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)) == PAR
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 o
On Tue, 2014-08-19 at 14:16 +0200, Richard Biener wrote:
> On Tue, Aug 19, 2014 at 2:10 PM, Mark Wielaard wrote:
> > gcc/testsuite/ChangeLog
> > 2014-08-19 Mark Wielaard
> >
> > * gcc.dg/guality/restrict.c: Add `used' attribute to all variables.
> > (cpy): Change type of last argument t
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-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_
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
Implement normalization checks.
* gcc/cp/constraint.cc (normalize_expr): Delegate cast and
atomic no
Another needed tweak, applying to trunk.
commit aac24a75af315fb6aa6099a159c6bce339706bc1
Author: Jason Merrill
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.
diff --git a/gcc/c
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 ppc64-linux,
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 correspond
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.
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
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.
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 t
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, returni
On Aug 19, 2014, at 6:12 AM, Kyrill Tkachov wrote:
> So how about this?
Ok. Thanks.
On Mon, Aug 18, 2014 at 12:29 PM, Uros Bizjak wrote:
> On Mon, Aug 18, 2014 at 9:16 PM, H.J. Lu 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 new insn pattern sh
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.
>
>
> (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,
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 acces
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
>
> * config/alpha/alpha.h (CLZ_DEFINED_VALUE_AT_ZERO,
> CTZ_DEFINED_VALUE_AT_ZERO): Return 0/1 rather th
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
Committed as revision 214157.
2014-08-19 Christopher Faylor
* MAINTAINERS: Remove myself
Index: MAINTAINERS
===
--- MAINTAINERS (revision 214153)
+++ MAINTAINERS (working copy)
@@ -136,7 +136,6 @@
VMS
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 (mat
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
* lex.c (_cpp_lex_direct): Fix a typo.
diff --git gcc/lex.c gcc/lex.c
index 0713f65..827cfb0 100644
--- gcc/lex.c
+++ gcc/lex.c
@@ -
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 (VI
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 much
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):
Applied at revision 214163.
Thanks,
Kai
Patch is ok. Applied this patch at revision 214161 together with your
followup-patch at revision 214162.
Thanks,
Kai
Patch is ok. Applied this patch at revision 214162 together with your
followup-patch at revision 214161.
Thanks,
Kai
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
> +
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 enco
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
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
* config/arm/vfp.md (*abssf2_vfp): Use UAL assembly syntax.
(
Committed revision 214159.
2014-08-19 Joost VandeVondele
* MAINTAINERS (Write After Approval): Add myself.
Index: MAINTAINERS
===
--- MAINTAINERS (revision 214158)
+++ MAINTAINERS (working copy)
@@ -561,6 +561,7 @@ Markus
Manuel López-Ibáñez 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. A simple
> way is
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
* config/arm/arm.md (vfp_pop_multiple
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
* config/arm/vfp.md (*movcc_vfp): Use UAL syntax.commit f33bf477f1e30e1125250838843343facb2a7716
A
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}, #0.0'
but current gas has a bug and doesn't accept that form, only 'vcmp
{s,d}, #0'
There is a gas patch in review on the binutils mailing list but
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
* config/arm/vfp.md
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
* config/arm/vfp.md (*muls
Applied for you at revision 214158.
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 p
Applied for you at revision 214156.
Kai
1 - 100 of 156 matches
Mail list logo
