On 12/02/2015 06:23 AM, Michael Matz wrote:
Hi,
On Tue, 1 Dec 2015, Jeff Law wrote:
So, okay for trunk?
-ENOPATCH
Sigh :)
Here it is.
Ciao,
Michael.
* common.opt (-fsplit-loops): New flag.
* passes.def (pass_loop_split): Add.
* opts.c (default_options_table): Add O
On 12/04/15 13:28, Dmitry Vyukov wrote:
On Fri, Dec 4, 2015 at 6:39 PM, Jakub Jelinek wrote:
On Fri, Dec 04, 2015 at 06:32:38PM +0100, Dmitry Vyukov wrote:
+2015-12-04 Dmitry Vyukov
+
+ * sancov.c: New file.
+ * Makefile.in (OBJS): Add sancov.o.
+ * invoke.texi (-fsanitize-cover
On Fri, Dec 4, 2015 at 11:26 AM, Vladimir Makarov wrote:
> The following patch fixes
>
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68349
>
> The patch was tested and bootstrapped on x86/x86-64.
>
> Committed as rev. 231300.
unsigned long strlen();
^^^
I got
.
On Fri, Dec 4, 2015 at 10:04 AM, Jan Hubicka wrote:
> Hi,
> this is the patch implementing renaming and fixing logic in ipa-icf/lto-symtab
> WRT compuattion of this flag. I don't seem to be able to construct testcase
> for
> this: the merged flag is currently used in inliner only to decide wheth
On 12/04/2015 01:48 PM, Paolo Bonzini wrote:
gcc:
PR sanitizer/68418
* c-family/c-ubsan.c (ubsan_instrument_shift): Disable
sanitization of left shifts for wrapping signed types as well.
gcc/testsuite:
PR sanitizer/68418
* gcc.dg/ubsan/c99-wrapv-shift-1.
On 12/04/2015 01:48 PM, Paolo Bonzini wrote:
gcc:
PR sanitizer/68418
* c-family/c-ubsan.c (ubsan_instrument_shift): Disable
sanitization of left shifts for wrapping signed types as well.
gcc/testsuite:
PR sanitizer/68418
* gcc.dg/ubsan/c99-wrapv-shift-1.
This patch adds some more runtime test coverage for all of the clauses
supported by the kernels, parallel and loop directives. Specifically,
acc loops inside parallel regions were missing coverage for the tile
clause. Kernels loop were missing coverage for gang, worker, vector,
seq, auto, collapse,
On Wed, 25 Nov 2015, Marc Glisse wrote:
So, I'd really prefer doing x-y>x to y>x only for single use.
Ok
Let me post this patch (needs testing on x86, I only tested on ppc which
does not implement the new optabs) so I can more easily find it again at
next stage 1.
It produces imag()==0 o
LGTM. Thanks!
Tobias Burnus wrote:
> If there are no objections, I intent to commit the following patch
> tomorrow as obvious.
>
> Tobias
>
> On Tue, Oct 27, 2015 at 11:27:41AM +0100, Tobias Burnus wrote:
> > recently, support for ISL 0.15 was added to GCC and also
> > ftp://gcc.gnu.org/pub/gcc
On Fri, 2015-12-04 at 15:45 -0500, David Malcolm wrote:
> On Fri, 2015-12-04 at 12:09 +0100, Bernd Schmidt wrote:
> > On 12/03/2015 09:33 PM, David Malcolm wrote:
> > > The attached patch updates the handling of %q+D, simplifying
> > > the implementation, and ensuring that it retains the range
> >
On 01.12.15 22:47, Andreas Tobler wrote:
On 29.11.15 22:15, Andreas Tobler wrote:
Hi all,
I'd like to commit this patch to trunk.
It is FreeBSD only.
If nobody objects I'll commit it within two days.
Thanks,
Andreas
2015-11-29 Andreas Tobler
* config/rs6000/freebsd64.h (ELFv2_A
On 01.12.15 22:55, Jeff Law wrote:
On 12/01/2015 02:50 PM, Andreas Tobler wrote:
On 30.11.15 23:30, Jeff Law wrote:
On 11/26/2015 11:49 AM, Andreas Tobler wrote:
Hi all,
the attached patch fixes the build issue from this ticket if bootstrap
is disabled.
Tested on x86_64-*-linux* and on x86_6
> >> gcc:
> >>PR sanitizer/68418
> >>* c-family/c-ubsan.c (ubsan_instrument_shift): Disable
> >>sanitization of left shifts for wrapping signed types as well.
> >>
> >> gcc/testsuite:
> >>PR sanitizer/68418
> >>* gcc.dg/ubsan/c99-wrapv-shift-1.c,
> >>gcc.dg/ubsan/c99-wrapv-
On Fri, 2015-12-04 at 12:09 +0100, Bernd Schmidt wrote:
> On 12/03/2015 09:33 PM, David Malcolm wrote:
> > The attached patch updates the handling of %q+D, simplifying
> > the implementation, and ensuring that it retains the range
> > information of the decl, giving:
> >
> > diagnostic-ranges-1.c:6
gcc.dg/vla-24.c uses alloca. Marked accordingly and committed as obvious.
nathan
2015-12-04 Nathan Sidwell
* gcc.dg/vla-24.c: Requires alloca.
Index: testsuite/gcc.dg/vla-24.c
===
--- testsuite/gcc.dg/vla-24.c (revision 23130
I would highly recommend updating the required version of ISL to isl-0.15:
that would simplify the existing code, removing a lot of code under "#ifdef
old ISL",
and allow us to fully transition to schedule_trees instead of dealing with
the
old antiquated union_maps in the scheudler. The result is
Since r226675, we have been seeing these failures:
FAIL: gcc.dg/vect/vect-widen-mult-const-s16.c -flto -ffat-lto-objects
scan-tree-dump-times vect "pattern recognized" 2
FAIL: gcc.dg/vect/vect-widen-mult-const-s16.c scan-tree-dump-times vect
"pattern recognized" 2
FAIL: gcc.dg/vect/vect-widen-mult
tested on aarch64-none-elf
> with no issues.
>
> OK?
>
> Thanks,
> James
>
> ---
> gcc/
>
> 2015-12-04 James Greenhalgh
>
> * config/aarch64/aarch64.md (add3_pluslong): Add register
> constraints.
>
> gcc/testsuite/
>
> 2015-12-04 James Greenhalgh
>
> * gcc.c-torture/compile/20151204.c: New.
>
+main(int argc, char** argv)
Space before (.
OK
/M
constraints.
gcc/testsuite/
2015-12-04 James Greenhalgh
* gcc.c-torture/compile/20151204.c: New.
diff --git a/gcc/config/aarch64/aarch64.md b/gcc/config/aarch64/aarch64.md
index 765df6a..79d1414 100644
--- a/gcc/config/aarch64/aarch64.md
+++ b/gcc/config/aarch64/aarch64.md
@@ -1613,9
On Fri, Dec 04, 2015 at 06:19:19PM +, Manuel López-Ibáñez wrote:
> On 4 December 2015 at 17:53, Jakub Jelinek wrote:
> > +
> > + if (e->unknown_error)
> > + error_at (loc, e->unknown_error, option->opt_text);
> > + else
> > +
Ping.
Thanks, Teresa
On Wed, Dec 2, 2015 at 12:46 PM, Teresa Johnson wrote:
> Remove an assert that was overly-strict and already partially redundant
> with an immediately prior assert. In this case we had a hidden visibility
> function clone that was created after the LIPO link due to indirect c
The following patch fixes
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68349
The patch was tested and bootstrapped on x86/x86-64.
Committed as rev. 231300.
Index: ChangeLog
===
--- ChangeLog (revision 231299)
+++ ChangeLog (wo
On 12/04/2015 10:51 AM, Paolo Bonzini wrote:
On 25/11/2015 14:55, Paolo Bonzini wrote:
Left shifts into the sign bit is a kind of overflow, and the
standard chooses to treat left shifts of negative values the
same way.
However, the -fwrapv option modifies the language to one where
integers ar
On 12/04/2015 03:00 AM, Tobias Burnus wrote:
And now with attached patch ...
On Fri, Dec 04, 2015 at 10:59:29AM +0100, Tobias Burnus wrote:
A few warning options lack the 'Warning' flag, which since r228094 (2015-09-24)
has the effect that -W(no-)error= doesn't work for them. Additionally,
--he
On 12/04/15 13:44, Nathan Sidwell wrote:
I've committed this patch to unify the variable emission machinery. All four
hooks were doing small variations of a common theme. This introduces
nvptx_assemble_decl_begin to do the common work. While there, I expanded the
init initialization code to wo
On December 4, 2015 12:44:57 PM CST, Jeff Law wrote:
>On 12/02/2015 03:23 PM, Jan Sommer wrote:
>> Am Wednesday 02 December 2015, 08:13:20 schrieb Joel Sherrill:
>>>
>>> On December 2, 2015 2:14:22 AM EST, Jeff Law
>>> wrote:
On 12/01/2015 12:56 PM, Jan Sommer wrote:
> Am Monday 30 Nov
On 12/02/2015 03:23 PM, Jan Sommer wrote:
Am Wednesday 02 December 2015, 08:13:20 schrieb Joel Sherrill:
On December 2, 2015 2:14:22 AM EST, Jeff Law
wrote:
On 12/01/2015 12:56 PM, Jan Sommer wrote:
Am Monday 30 November 2015, 16:19:30 schrieb Jeff Law:
On 11/30/2015 03:06 PM, Jan Sommer wr
I've committed this patch to unify the variable emission machinery. All four
hooks were doing small variations of a common theme. This introduces
nvptx_assemble_decl_begin to do the common work. While there, I expanded the
init initialization code to work with multi-dimensional arrays.
nath
2015-12-02 Aditya Kumar
Sebastian Pop
* gcc.dg/graphite/id-29.c: New test.
gcc/ChangeLog:
2015-12-02 Aditya Kumar
Sebastian Pop
* graphite-isl-ast-to-gimple.c (translate_isl_ast_node_user): Improve
debug.
(get_rename_from_scev): Check
The check for dominance while extending the scop assumed that
multiple successors meant a loop which is not true in case of
conditionals around the loop.
Improved pretty printers for better debugging.
gcc/ChangeLog:
2015-12-02 Aditya Kumar
Sebastian Pop
* graphite-scop-
On Fri, Dec 4, 2015 at 6:39 PM, Jakub Jelinek wrote:
> On Fri, Dec 04, 2015 at 06:32:38PM +0100, Dmitry Vyukov wrote:
>> +2015-12-04 Dmitry Vyukov
>> +
>> + * sancov.c: New file.
>> + * Makefile.in (OBJS): Add sancov.o.
>> + * invoke.texi (-fsanitize-coverage=trace-pc): Describe.
>>
> Minor nit. Since this is used in an unspec_volatile, the name should be
> UNSPECV_ and defined in the unspecv enum.
Ouch. It turns out that the ARM implementation has it too so I have installed
the attached patchlet on top of the others after minimal retesting.
PR middle-end/65958
On 04/12/15 17:46, Ramana Radhakrishnan wrote:
On 04/12/15 16:04, Richard Biener wrote:
On December 4, 2015 4:32:33 PM GMT+01:00, Alan Lawrence
wrote:
On 27/11/15 08:30, Richard Biener wrote:
This is part 1 of a fix for PR68533 which shows that some targets
cannot can_vec_perm_p on an ide
On 4 December 2015 at 17:53, Jakub Jelinek wrote:
> +
> + if (e->unknown_error)
> + error_at (loc, e->unknown_error, option->opt_text);
> + else
> + error_at (loc, "unrecognized argument in option %qs",
> +
On Fri, 2015-12-04 at 12:10 -0500, Jason Merrill wrote:
> On 12/03/2015 09:55 AM, David Malcolm wrote:
> > @@ -362,10 +362,11 @@ convert_to_real_1 (tree type, tree expr, bool fold_p)
> > case REAL_TYPE:
> > /* Ignore the conversion if we don't need to store intermediate
> > resul
Hi,
this is the patch for fold-const.c. Can you think of some testcase for the
MR_DEPENDENCE_CLIQUE comparsion? I am not that familiar with the code to
be able to construct it :(
Bootstrapped/regtested x86_64-linux, OK?
Honza
* fold-const.c (operand_equal_p): Do not use flag_strict_alias
Hi,
this is the patch implementing renaming and fixing logic in ipa-icf/lto-symtab
WRT compuattion of this flag. I don't seem to be able to construct testcase for
this: the merged flag is currently used in inliner only to decide whether to
ingore optimize_size/optimize levels which should not lead
On Dec 4, 2015, at 5:13 AM, Alan Lawrence wrote:
> On 05/11/15 21:43, Sebastian Pop wrote:
>>* graphite-optimize-isl.c (optimize_isl): Call
>>isl_options_set_schedule_maximize_band_depth.
>>
>>* gcc.dg/graphite/fuse-1.c: New.
>>* gcc.dg/graphite/fuse-2.c: New.
>>
The Cilk+ code bizarrely parses function-trailing attributes in a
different place from normal operation, and tells the compiler that they
were C++11 attributes when they were actually specified with the GNU
syntax, which changes what they apply to. This patch fixes the second
issue, which is e
On Fri, Dec 04, 2015 at 06:01:58PM +0100, Bernd Schmidt wrote:
> I think marking stuff with Warning as appropriate qualifies as obvious.
>
> On 12/04/2015 05:37 PM, Jakub Jelinek wrote:
> >+ /* If the switch takes an integer, convert it. */
> >+ if (arg && cl_options[opt_index].cl_uinte
On 25/11/2015 14:55, Paolo Bonzini wrote:
> Left shifts into the sign bit is a kind of overflow, and the
> standard chooses to treat left shifts of negative values the
> same way.
>
> However, the -fwrapv option modifies the language to one where
> integers are defined as two's complement---whic
On 12/03/2015 11:42 PM, Martin Sebor wrote:
+ if (next && TREE_CODE (next) == FIELD_DECL)
This will break if there's a non-field between the array and the next field.
@@ -4114,7 +4115,10 @@ walk_subobject_offsets (tree type,
/* Avoid recursing into objects that are not interes
On 04/12/15 16:04, Richard Biener wrote:
> On December 4, 2015 4:32:33 PM GMT+01:00, Alan Lawrence
> wrote:
>> On 27/11/15 08:30, Richard Biener wrote:
>>>
>>> This is part 1 of a fix for PR68533 which shows that some targets
>>> cannot can_vec_perm_p on an identity permutation. I chose to fix
On Fri, Dec 04, 2015 at 06:32:38PM +0100, Dmitry Vyukov wrote:
> +2015-12-04 Dmitry Vyukov
> +
> + * sancov.c: New file.
> + * Makefile.in (OBJS): Add sancov.o.
> + * invoke.texi (-fsanitize-coverage=trace-pc): Describe.
> + * passes.def (sancov_pass): Add.
> + * tree-pass.h
This patch ensures that this_thread::get_id() returns a value that is
distinct from the "not a thread" value, and avoids undefined behaviour
in pthread_equal.
Previously programs using glibc but not linking to libpthread would
get the "not a thread" value for std::this_thread::get_id() in main.
W
On 12/04/2015 04:18 PM, Andre Vieira wrote:
Reworked following Joern's suggestion.
Is this OK?
Yes.
Bernd
On Fri, Dec 4, 2015 at 2:45 PM, Yury Gribov wrote:
> On 12/04/2015 04:41 PM, Jakub Jelinek wrote:
>>
>> Hi!
>>
>> While this has been posted after stage1 closed and I'm not really happy
>> that it missed the deadline, I'm willing to grant an exception, the patch
>> is small enough that it is ok at
On Fri, Dec 4, 2015 at 2:41 PM, Jakub Jelinek wrote:
> Hi!
>
> While this has been posted after stage1 closed and I'm not really happy
> that it missed the deadline, I'm willing to grant an exception, the patch
> is small enough that it is ok at this point of stage3. That said, next time
> please
Hi Guys,
I am applying the attached patch to reduce the number of multilibs for
the MSP430 target. This is at the request of TI, on behalf of their
customers, who complained that the toolchain was too large. The patch
only affects MSP430 specific files, and parts of files. It does not
On December 4, 2015 5:20:52 PM GMT+01:00, Jeff Law wrote:
>On 12/04/2015 03:19 AM, Richard Biener wrote:
>> On Thu, Dec 3, 2015 at 6:54 PM, Jeff Law wrote:
>>> This is something I noticed while working on fixing 67816.
>>>
>>> Essentially I was seeing trivially true or trivially false
>conditiona
On 12/04/2015 03:12 AM, Richard Biener wrote:
On Thu, Dec 3, 2015 at 9:29 PM, Jeff Law wrote:
On 12/02/2015 08:35 AM, Richard Biener wrote:
The most interesting side effect, and one I haven't fully analyzed yet is
an
unexpected jump thread -- which I've traced back to differences in what
the
Version 5 with the latest requested changes. Seems to work now.
I've dropped the extra patch and rather marked the failing tests
as "xfail".
Ciao
Dominik ^_^ ^_^
--
Dominik Vogt
IBM Germany
gcc/ChangeLog
* config/s390/s390.c (s390_expand_setmem): Use new expanders.
* config/
Version 6 with another fix. This should work now.
Ciao
Dominik ^_^ ^_^
--
Dominik Vogt
IBM Germany
gcc/ChangeLog
* config/s390/s390.md ("movstr", "*movstr"): Fix warning.
("movstr"): New indirect expanders used by "movstr".
gcc/testsuite/ChangeLog
* gcc.target/s390/
On 12/04/2015 05:45 PM, Jakub Jelinek wrote:
This patch fixes it to use explicit UNKNOWN_LOCATION, instead of
explicit or implicit input_location, which for most of process_options
is somewhere on line 1 of the main source file.
Ok.
Bernd
On 12/04/2015 11:45 AM, David Malcolm wrote:
On Fri, 2015-12-04 at 11:01 -0500, Jason Merrill wrote:
On 12/03/2015 05:08 PM, David Malcolm wrote:
On Thu, 2015-12-03 at 15:38 -0500, Jason Merrill wrote:
On 12/03/2015 09:55 AM, David Malcolm wrote:
Testcase g++.dg/template/ref3.C:
1
On 12/03/2015 09:55 AM, David Malcolm wrote:
@@ -362,10 +362,11 @@ convert_to_real_1 (tree type, tree expr, bool fold_p)
case REAL_TYPE:
/* Ignore the conversion if we don't need to store intermediate
results and neither type is a decimal float. */
- return build1 ((f
I think marking stuff with Warning as appropriate qualifies as obvious.
On 12/04/2015 05:37 PM, Jakub Jelinek wrote:
+ /* If the switch takes an integer, convert it. */
+ if (arg && cl_options[opt_index].cl_uinteger)
+ {
+ value = integral_argument (arg);
+
> On Fri, 4 Dec 2015, Jan Hubicka wrote:
>
> > > Indeed we don't do code hoisting yet. Maybe one could trick PPRE
> > > into doing it.
> > >
> > > Note that for OBJ_TYPE_REFs in calls you probably should better use
> > > gimple_call_fntype instead of the type of the OBJ_TYPE_REF anyway
> > > (we
>
> I wonder if you can split out the re-naming at this stage. Further
> comments below.
OK, I will commit the renaming and ipa-icf fix separately.
>
> > Bootstrapped/regtested x86_64-linux, OK?
> >
> > I will work on some testcases for the ICF and fold-const that would lead
> > to wrong code
Hi!
As mentioned in the PR, the process_options diagnostics is about errors
on the command line (incompatible options, unsupported options etc.),
which aren't really related to any source code location.
This patch fixes it to use explicit UNKNOWN_LOCATION, instead of
explicit or implicit input_loc
On Fri, 2015-12-04 at 11:01 -0500, Jason Merrill wrote:
> On 12/03/2015 05:08 PM, David Malcolm wrote:
> > On Thu, 2015-12-03 at 15:38 -0500, Jason Merrill wrote:
> >> On 12/03/2015 09:55 AM, David Malcolm wrote:
> >>> Testcase g++.dg/template/ref3.C:
> >>>
> >>>1 // PR c++/28341
> >>>
I've committed the obvious patch after confirmation from
the original author that it is correct.
2015-12-04 Steven G. Kargl
PR fortran/68684
* resolve.c (resolve_lock_unlock_event): Fix logic faux pas.
Index: resolve.c
==
Hi!
GCC changed recently to disallow -Werror= on W options that don't have
Warning keyword set on them, the following patch fixes 3 spots where
Warning has been unintentionally omitted. The PR also mentions -Wpsabi,
but I believe -Werror=psabi is not appropriate, because -Wpsabi is not
really a w
The PowerPC port provides reciprocal sqrt but doesn't implement the
extra incantation to utilize it for sqrtf.
The current implementation re-associates terms in the N-R iteration to
utilize one constant instead of two, but does not provide a
pre-computed estimate multiplied by the source, which re
On 12/04/2015 03:19 AM, Richard Biener wrote:
On Thu, Dec 3, 2015 at 6:54 PM, Jeff Law wrote:
This is something I noticed while working on fixing 67816.
Essentially I was seeing trivially true or trivially false conditionals left
in the IL for DOM to clean up.
While DOM can and will clean tha
On Thu, 3 Dec 2015, Martin Sebor wrote:
> The only C change in this patch is to include the size of excessively
> large types in diagnostics (I found knowing the size helpful when
> adding tests and I think it might be helpful to others as well).
I don't see what that C change has to do with flex
On December 4, 2015 4:32:33 PM GMT+01:00, Alan Lawrence
wrote:
>On 27/11/15 08:30, Richard Biener wrote:
>>
>> This is part 1 of a fix for PR68533 which shows that some targets
>> cannot can_vec_perm_p on an identity permutation. I chose to fix
>> this in the vectorizer by detecting the identity
On 12/03/2015 05:08 PM, David Malcolm wrote:
On Thu, 2015-12-03 at 15:38 -0500, Jason Merrill wrote:
On 12/03/2015 09:55 AM, David Malcolm wrote:
Testcase g++.dg/template/ref3.C:
1// PR c++/28341
2
3template struct A {};
4
5template st
On 27/11/15 08:30, Richard Biener wrote:
This is part 1 of a fix for PR68533 which shows that some targets
cannot can_vec_perm_p on an identity permutation. I chose to fix
this in the vectorizer by detecting the identity itself but with
the current structure of vect_transform_slp_perm_load this
On 17/11/15 16:30, Andre Vieira wrote:
On 17/11/15 12:29, Bernd Schmidt wrote:
On 11/16/2015 04:48 PM, Andre Vieira wrote:
On 16/11/15 15:34, Joern Wolfgang Rennecke wrote:
I just happened to stumble on this problem with another port.
The volatile & test solution doesn't work, though.
What do
Hi Richard.
Thanks a lot for your review.
Below are my answers.
You asked why I inserted additional check to
++ b/gcc/tree-ssa-forwprop.c
@@ -373,6 +373,11 @@ combine_cond_expr_cond (gimple *stmt, enum
tree_code code, tree type,
gcc_assert (TREE_CODE_CLASS (code) == tcc_comparison);
+ /* Do
On 02 Dec 16:27, Richard Biener wrote:
> On Wed, Dec 2, 2015 at 4:24 PM, Ilya Enkovich wrote:
> >
> > The problem is that conversion is supposed to be handled by
> > vectorizable_conversion,
> > but it fails to because it is not actually a conversion. I suppose it
> > may be handled
> > in vectori
On 04/12/15 14:40, Kyrill Tkachov wrote:
Ping.
This almost fell through the cracks.
https://gcc.gnu.org/ml/gcc-patches/2015-10/msg00055.html
Thanks,
Kyrill
On 01/10/15 14:00, Kyrill Tkachov wrote:
Hi all,
As part of the SWITCHABLE_TARGET work I inadvertently changed the
behaviour of -mgeneral
On Fri, Dec 4, 2015 at 9:34 AM, Segher Boessenkool
wrote:
> This implements cstore for the last case we do not yet handle, using
> the superopt algo from the venerable CWG. The only integer cases we
> do still not handle after this are for -m32 -mpowerpc64. Those case
> now generate a branch seq
On 12/04/2015 02:23 AM, Dominik Vogt wrote:
gcc/testsuite/ChangeLog
* gcc.dg/Wframe-address.c: S/390 requires the -mbackchain option to
access arbitrary stack frames.
* gcc.dg/Wno-frame-address.c: Likewise.
Applied. Thanks!
-Andreas-
This implements cstore for the last case we do not yet handle, using
the superopt algo from the venerable CWG. The only integer cases we
do still not handle after this are for -m32 -mpowerpc64. Those case
now generate a branch sequence, which is better than what we had
before.
Tested on powerpc6
On Fri, Dec 4, 2015 at 9:34 AM, Segher Boessenkool
wrote:
> "register_operand" was a bit confusing. Also some other minor cleanups.
>
> Tested on powerpc64-linux; okay for mainline?
>
>
> Segher
>
>
> 2015-12-04 Segher Boessenkool
>
> * (cstore4_unsigned): Use gpc_reg_operand instead o
On 09/25/2015 04:02 PM, Dominik Vogt wrote:
> On Fri, Sep 25, 2015 at 02:59:41PM +0100, Dominik Vogt wrote:
>> The following set of two patches implements the function
>> __attribute__ ((target("..."))) and the corresponding #pragma GCC
>> target("...") on S/390. It comes with certain limitations:
Ping.
This almost fell through the cracks.
https://gcc.gnu.org/ml/gcc-patches/2015-10/msg00055.html
Thanks,
Kyrill
On 01/10/15 14:00, Kyrill Tkachov wrote:
Hi all,
As part of the SWITCHABLE_TARGET work I inadvertently changed the behaviour of
-mgeneral-regs-only with respect to the .arch dire
On 12/04/2015 03:14 PM, Dominik Vogt wrote:
> Next version of the patch with the changes discussed internally.
> (Sorry for the version number; I now use my internam branch
> numbers for sake of clarity.)
Applied. Thanks!
-Andreas-
"register_operand" was a bit confusing. Also some other minor cleanups.
Tested on powerpc64-linux; okay for mainline?
Segher
2015-12-04 Segher Boessenkool
* (cstore4_unsigned): Use gpc_reg_operand instead of
register_operand. Remove empty constraints. Use std::swap.
On Thu, 2015-12-03 at 17:17 -0500, Jason Merrill wrote:
> On 12/03/2015 04:43 PM, David Malcolm wrote:
> > On Thu, 2015-12-03 at 15:33 -0500, Jason Merrill wrote:
> >> On 12/03/2015 09:55 AM, David Malcolm wrote:
> >>> This patch adds bulletproofing to detect purged tokens, and avoid using
> >>> th
Next version of the patch with the changes discussed internally.
(Sorry for the version number; I now use my internam branch
numbers for sake of clarity.)
Ciao
Dominik ^_^ ^_^
--
Dominik Vogt
IBM Germany
gcc/ChangeLog
* config/s390/s390.opt (s390_arch_string): Remove.
(s390_t
Hello,
Patch in the bottom fixes miscompare issue on Spec2k6/434.zeus.
Bootstrapped & reg-tested.
If no objections - I'll commit it into GCC main trunk on Monday.
gcc/
PR target/68633
* config/i386/sse.md (define_insn "kunpckhi"): Fix operands order.
(define_insn "kunpcksi
The PTX backend has to emit PTX function prototypes and prologues in addition to
the regular argument marshalling machinery. And of course these all have to
agree with each other. The first two are done by two different pieces of code,
This patch changes things so that prototype and prologue
> + (unspec_volatile:PTR [(match_operand:PTR 1 "register_operand" "0")
> + (match_operand:PTR 2 "register_operand" "r")]
> +UNSPEC_PROBE_STACK_RANGE))]
Minor nit. Since this is used in an unspec_volatile, the name should be
UNSPECV_ and de
The recently added test graphite/id-28.c requires pthreads due to its use of
-fcilkplus. Committed as obvious.
nathan
2015-12-04 Nathan Sidwell
* gcc.dg/graphite/id-28.c: Requires pthreads.
Index: testsuite/gcc.dg/graphite/id-28.c
==
Hi Ilya,
On 02 Dec 16:51, Ilya Enkovich wrote:
> Hi,
>
> This patch fixes wrong alignment check in _store_mask
> pattern. Currently we check a register operand instead of a memory
> one. This fixes segfault on 481.wrf compiled at -O3 for KNL target.
> I bootstrapped and tested this patch on x86_
On 12/04/2015 04:41 PM, Jakub Jelinek wrote:
Hi!
While this has been posted after stage1 closed and I'm not really happy
that it missed the deadline, I'm willing to grant an exception, the patch
is small enough that it is ok at this point of stage3. That said, next time
please try to submit new
Hello.
The patch handles builtins mentioned in the email subject and was installed
to the HSA branch.
Thanks,
Martin
>From b56ba10d46c03cadfda16c6658dd3134f5da09f8 Mon Sep 17 00:00:00 2001
From: marxin
Date: Thu, 3 Dec 2015 13:31:28 +0100
Subject: [PATCH 2/3] HSA: implement __builtin_bzero
gcc/
Hello.
Following patch handles just floating pointer internal functions and
for the rest if displays warning message.
Patch has been installed to the HSA branch.
Thanks,
Martin
>From 5c6581d913c754e4e0a197e073bfa3c17c20b31f Mon Sep 17 00:00:00 2001
From: marxin
Date: Fri, 4 Dec 2015 09:56:11 +0
Hi!
While this has been posted after stage1 closed and I'm not really happy
that it missed the deadline, I'm willing to grant an exception, the patch
is small enough that it is ok at this point of stage3. That said, next time
please try to submit new features in time.
Are there any plans for GCC
Status
==
The GCC 5 branch is open again for regression and documentation fixes.
If nothing unusual happens you can expect GCC 5.4 somewhen closely
before GCC 6 is released.
Quality Data
Priority # Change from last report
--- --
I pressed "Send" too early - as the testsuite fails unless the following
patch is applied. I think I will just use this test case (with patch)
instead of adding a new test-suite file. Required patch:
--- a/gcc/testsuite/gfortran.dg/coarray_args_2.f90
+++ b/gcc/testsuite/gfortran.dg/coarray_args_2.
On 05/11/15 21:43, Sebastian Pop wrote:
* graphite-optimize-isl.c (optimize_isl): Call
isl_options_set_schedule_maximize_band_depth.
* gcc.dg/graphite/fuse-1.c: New.
* gcc.dg/graphite/fuse-2.c: New.
* gcc.dg/graphite/interchange-13.c: Remove bogus check.
PING.
-- Forwarded message --
From: H.J. Lu
Date: Mon, Oct 19, 2015 at 1:04 PM
Subject: PING: [PATCH] X86: Optimize access to globals in PIE with copy reloc
To: GCC Patches , Richard Biener
, Jakub Jelinek ,
Richard Henderson
PING.
-- Forwarded message --
Fro
On Fri, Mar 27, 2015 at 03:04:05PM -0300, Alexandre Oliva wrote:
> This patch reworks the out-of-ssa expander to enable coalescing of SSA
> partitions that don't share the same base name. This is done only when
> optimizing.
>
> The test we use to tell whether two partitions can be merged no long
Ping?
On 27 November 2015 at 14:00, Christophe Lyon
wrote:
> Hi,
>
> After the recent commits from Christian adding target attributes
> support for ARM FPU settings, I've noticed that some of the tests
> were failing because of incorrect assumptions wrt to the default
> cpu/fpu/float-abi of the
This patch permits
interface
subroutine sub (x)
real x(10)[*]
end subroutine
end interface
real :: x(100)[*]
call sub (x(10))
end
where one passes an array element ("x(10)") of a contiguous array to a
coarray dummy argument. That's permitted per interpretation
As described in pr other/67627, the all-multi target can be
built in parallel with the %_.lo targets which generate make
dependencies that are parsed during the build of all-multi.
gcc -MD does not generate the makefile dependencies in an
atomic way so make can fail if it concurrently parses thos
1 - 100 of 135 matches
Mail list logo