Hi all,
This ICE in this PR occurs when we're trying to split unaligned_loaddi into two
SImode unaligned loads.
The problem is in the addressing mode. When reload was picking the addressing
mode we accepted an offset of
-256 because the mode in the pattern is advertised as DImode and that was
Hi,
On 11/10/2015 05:36 PM, Marek Polacek wrote:
+
+ /* Did array size calculations overflow or does the array
+ cover more than half of the address-space? */
+ if (COMPLETE_TYPE_P (type)
+ && TREE_CODE (TYPE_SIZE_UNIT (type)) == I
On Tue, Nov 10, 2015 at 05:22:40PM +0100, Christophe Lyon wrote:
> On 10 November 2015 at 12:41, Robert Suchanek
> wrote:
> > Hi Christophe,
> >
> >> Hi,
> >>
> >> Since you committed this (r230087 if I'm correct), I can see that GCC
> >> fails to build
> >> ligfortran for target arm-none-linuxgnu
On Tue, Nov 10, 2015 at 12:16:09PM +0100, Bernd Schmidt wrote:
> On 11/09/2015 08:33 AM, Segher Boessenkool wrote:
> >If we have
> >
> > (truncate:M1 (and:M2 (lshiftrt:M2 (x:M2) C) C2))
> >
> >we can write it instead as
> >
> > (and:M1 (lshiftrt:M1 (truncate:M1 (x:M2)) C) C2)
> >
> >
> >+
2015-11-10 Evandro Menezes
gcc/
* config/aarch64/aarch64.md (predicated): Copy attribute from
"arm.md".
This patch duplicates an attribute from arm.md so that the same pipeline
model can be used for both AArch32 and AArch64.
Bootstrapped on arm-unknown-linux-gnueabihf,
2015-11-10 Evandro Menezes
gcc/
* config/aarch64/aarch64-cores.def: Use the Exynos M1 sched model.
* config/aarch64/aarch64.md: Include "exynos-m1.md".
* config/arm/arm-cores.def: Use the Exynos M1 sched model.
* config/arm/arm.md: Include "exynos-m1.md".
Hello!
Recent AS patches introduced the above compilation failure. We have to
treat movabs operands in a special way - without %rip and inside
square bracket for -masm=intel.
Also, the patch removes dead code while at it.
2015-11-10 Uros Bizjak
* config/i386/i386.c (ix86_print_operand):
On Tue, 10 Nov 2015, David Malcolm wrote:
> This is the most trivial example of a real fix-it example I could think
> of: if the user writes
> ptr.field
> rather than ptr->field.
>
> gcc/c/ChangeLog:
> * c-typeck.c (build_component_ref): Special-case POINTER_TYPE when
> generati
On 11/10/2015 06:54 PM, Uros Bizjak wrote:
- return "movabs{}\t{%1, %0|%0, %1}";
+ return "movabs{}\t{%1, %P0|[%P0], %1}";
The thing that's missing from this, that's present in the patch that I sent you
off-list, is the thing for Intel syntax.
Would you prefer to just add that bac
On Tue, 10 Nov 2015, Marek Polacek wrote:
> While both C and C++ FEs are able to reject e.g.
> int a[__SIZE_MAX__ / sizeof(int)];
> they are accepting code such as
> int (*a)[__SIZE_MAX__ / sizeof(int)];
>
> As Joseph pointed out, any construction of a non-VLA type whose size is half
> or
> more
On Tue, Nov 10, 2015 at 5:50 PM, Evandro Menezes wrote:
>2015-11-10 Evandro Menezes
>
>gcc/
>
>* config/aarch64/aarch64.md (predicated): Copy attribute from
>"arm.md".
>
> This patch duplicates an attribute from arm.md so that the same pipeline
> model can be used for bot
On Tue, Nov 10, 2015 at 6:01 PM, Ramana Radhakrishnan
wrote:
> On Tue, Nov 10, 2015 at 5:50 PM, Evandro Menezes
> wrote:
>>2015-11-10 Evandro Menezes
>>
>>gcc/
>>
>>* config/aarch64/aarch64.md (predicated): Copy attribute from
>>"arm.md".
>>
>> This patch duplicates an
On Tue, Nov 10, 2015 at 7:00 PM, Richard Henderson wrote:
> On 11/10/2015 06:54 PM, Uros Bizjak wrote:
>>
>> - return "movabs{}\t{%1, %0|%0, %1}";
>> + return "movabs{}\t{%1, %P0|[%P0], %1}";
>
>
> The thing that's missing from this, that's present in the patch that I sent
> you off-list
> > Index: tree.c
> > ===
> > --- tree.c (revision 229968)
> > +++ tree.c (working copy)
> > @@ -13198,6 +13198,7 @@ gimple_canonical_types_compatible_p (con
> >/* If the types have been previously registered and found equal
> >
One may need to execute extra steps after linking program. This is required
for example for DJGPP to run stubify.exe on file generated by linker.
The only way how to achieve was to use LINK_COMMAND_SPEC. It would be much
easier
and less error prone to use new macro POST_LINK_SPEC introduced in t
On Tue, Nov 10, 2015 at 7:14 PM, Uros Bizjak wrote:
> On Tue, Nov 10, 2015 at 7:00 PM, Richard Henderson wrote:
>> On 11/10/2015 06:54 PM, Uros Bizjak wrote:
>>>
>>> - return "movabs{}\t{%1, %0|%0, %1}";
>>> + return "movabs{}\t{%1, %P0|[%P0], %1}";
>>
>>
>> The thing that's missing fro
On Tue, 2015-11-10 at 17:26 +, Szabolcs Nagy wrote:
> On 09/11/15 00:19, Torvald Riegel wrote:
> > Hi,
> >
> > I'd like to summarize the current state of support for the TM TS, and
> > outline the current plan for the work that remains to complete the
> > support.
> >
> > I'm aware we're at the
This patch adds support for the MADDLD instruciton, which is a fused
multiply/add instruction for integers. At this time, it is for 64-bit
multiplies only. Eventually, we will restructure 128-bit multiply so that we
can use the 64x64 + 64 high bit varients.
I have bootstrapped a compiler with th
On Tue, Nov 10, 2015 at 12:41:07AM +, Joseph Myers wrote:
> I don't see any conversions between KFmode and TImode (in either
> direction, signed or unsigned) here - I suppose there are no instructions
> for that?
No in power9 there is no instruction that converts 128-bit integer to IEEE
128-
On November 10, 2015 6:29:36 PM GMT+01:00, Joseph Myers
wrote:
>On Tue, 10 Nov 2015, Richard Biener wrote:
>
>> Looks ok but I wonder if this is dead code with
>>
>> (for pows (POW)
>> sqrts (SQRT)
>> cbrts (CBRT)
>> (simplify
>> (pows @0 REAL_CST@1)
>> (with {
>> const REAL_V
On 11/10/2015 09:36 AM, Marek Polacek wrote:
While both C and C++ FEs are able to reject e.g.
int a[__SIZE_MAX__ / sizeof(int)];
they are accepting code such as
int (*a)[__SIZE_MAX__ / sizeof(int)];
As Joseph pointed out, any construction of a non-VLA type whose size is half or
more of the addre
On Mon, Nov 09, 2015 at 03:51:32AM -0600, Segher Boessenkool wrote:
> > >From the original patch submission, it looks that this patch would
> > also benefit x86_32.
>
> Yes, that is what I thought too.
>
> > Regarding the above code size increase - do you perhaps have a
> > testcase, to see what
Tested on Linux-X64.
2015-11-10 Ville Voutilainen
LWG 2510, make the default constructors of library tag types
explicit.
* include/bits/mutex.h (defer_lock_t, try_lock_t,
adopt_lock_t): Add an explicit default constructor.
* include/bits/stl_pair.h (piecewise_construct_t):
On Tue, Oct 27, 2015 at 11:01:39PM +0900, Oleg Endo wrote:
> On Mon, 2015-10-26 at 22:47 -0400, Rich Felker wrote:
> > On Sun, Oct 25, 2015 at 11:28:51PM +0900, Oleg Endo wrote:
> > > On Fri, 2015-10-23 at 02:32 -0400, Rich Felker wrote:
> > > > Here's my updated version of the FDPIC patch with all
Richard Biener writes:
> On Sat, Nov 7, 2015 at 2:23 PM, Richard Sandiford
> wrote:
>> diff --git a/gcc/genmatch.c b/gcc/genmatch.c
>> index cff32b0..7139476 100644
>> --- a/gcc/genmatch.c
>> +++ b/gcc/genmatch.c
>> @@ -4638,6 +4638,11 @@ main (int argc, char **argv)
>>cpp_callbacks *cb = cpp
On November 10, 2015 9:13:25 PM GMT+01:00, Richard Sandiford
wrote:
>Richard Biener writes:
>> On Sat, Nov 7, 2015 at 2:23 PM, Richard Sandiford
>> wrote:
>>> diff --git a/gcc/genmatch.c b/gcc/genmatch.c
>>> index cff32b0..7139476 100644
>>> --- a/gcc/genmatch.c
>>> +++ b/gcc/genmatch.c
>>> @@
This patch changes the Go tool to always use --whole-archive when
linking gccgo packages. This fixes cases where a Go package uses cgo
to call C code in which the only referenced symbol is a C global
variable. This is a backport of https://golang.org/cl/16775 in the
master Go sources. This fixes
I've committed this to gomp4. In preparing the reworked firstprivate patch
changes for gomp4's gimplify.c I discovered these testcases were passing by
accident, and lacked a data clause.
nathan
2015-11-10 Nathan Sidwell
* testsuite/libgomp.oacc-fortran/parallel-reduction.f90: Fix data
m
On 11/10/15 11:28, Thomas Schwinge wrote:
Hi Nathan!
On Tue, 10 Nov 2015 09:19:50 -0500, Nathan Sidwell wrote:
I've committed this to backport a bunch of random bits from trunk to gomp4, and
thereby reduce divergence.
Yeah, I had some of these on my list, too.
--- omp-low.c (revision 230
I applied this patch as obvious. I missed submitting it in my original patch
for the power9 support (it was in the sandbox I was testing power9 support on).
2015-11-10 Michael Meissner
* config.gcc (powerpc*-*-*, rs6000*-*-*): Add power9 to hosts that
default to 64-bit.
--
M
Hi,
While investigating another issue, I observed some repeated logic in
rs6000_secondary_reload_direct_move (). This patch takes it out. No
functional change intended, and quite straightforward, so I'll plan to
commit shortly if no concerns are raised.
Bootstrapped and tested on powerpc64le-li
On 11/10/2015 06:44 PM, Segher Boessenkool wrote:
Yes I know. All the rest of the code around is it like this though.
Do you want this written in a saner way?
I won't object to leaving it as-is for now, but in the future it would
be good to keep this in mind.
I'm not entirely sure what th
The ft32 and moxie ports failed to build after the recent output_address
changes. Fixed thusly and committed to the trunk after verifying the
moxie and ft32 ports in config-list.mk build again.
Jeff
commit b408dd85568c5d0c0a9673810280a8438753b60f
Author: law
Date: Tue Nov 10 21:11:07 2015 +
Andrew, can you go ahead and commit those changes into contrib? I think
in a subdirectory would be best so that you can include the README.
Make sure the permissions are set correctly. Applying them as a patch
kept mucking them up.
header-tools or somesuch should be a fine directory name
On 11/10/2015 09:44 AM, David Malcolm wrote:
This is a followup to:
[PATCH 10/10] Compress short ranges into source_location
which adds some worked examples of what a source_location/location_t
can encode.
Successfully bootstrapped®rtested on x86_64-pc-linux-gnu
(although it only touches a co
Richard Biener writes:
> On Sat, Nov 7, 2015 at 2:31 PM, Richard Sandiford
> wrote:
>> This patch short-circuits the builtins.c expansion code for a particular
>> gimple call if:
>>
>> - the function has an associated internal function
>> - the target implements that internal function
>> - the ca
On 11/10/2015 11:16 AM, Andris Pavenis wrote:
One may need to execute extra steps after linking program. This is required
for example for DJGPP to run stubify.exe on file generated by linker.
The only way how to achieve was to use LINK_COMMAND_SPEC. It would be
much easier
and less error prone t
On November 10, 2015 1:02:57 PM GMT+01:00, Richard Biener
wrote:
>On Sat, Nov 7, 2015 at 12:41 PM, Kumar, Venkataramanan
> wrote:
>> Hi Richard,
>>
>> I have now implemented storing of DR and references using hash maps.
>> Please find attached patch.
>>
>> As discussed, I am now storing the ref,
On Sun, Nov 8, 2015 at 9:21 AM, Rainer Orth
wrote:
>
> There were two remaining problems:
>
> * Before Solaris 12, sendfile only lives in libsendfile. This lead to
> link failures in gotools.
>
> * Solaris 12 introduced a couple more types that use _in6_addr_t, which
> are filtered out by mk
This patch d-form addressing to float/double scalars for the PowerPC that was
added in ISA 3.0 (power9). This patch does not yet turn on D-form addressing
as default. It is likely that patch #11, which will add limited d-form
addressing to vector registers will enable it by default.
I have boots
Hi all,
the attached patch removes the hard-coded requirement for the link
operation with -ldl. On FreeBSD we do not need that, it breaks compilation.
The patch is tested against CentOS7.1
Ok to apply for trunk?
TIA,
Andreas
2015-11-10 Andreas Tobler
* configure.ac: Replace the
On 11/09/2015 11:53 PM, Trevor Saunders wrote:
I don't really know him, but I don't really disagree with where he wants
to get to. However I think we work fairly different ways, and review
things differently. When I review patches (mostly for stuff more
directly related to Mozilla my standards
On Mon, 2015-11-09 at 16:37 -0700, Sandra Loosemore wrote:
> On 11/08/2015 06:55 AM, David Malcolm wrote:
> > I've been experimenting with using Sphinx [1] for GCC's documentation.
> >
> > [snip]
> >
> > The primary advantages of .rst/sphinx over .texi/texinfo I see are in
> > the generated HTML:
>
Hi all,
> > Now that 'make check' has had enough time to run, I can see several
> > regressions in the configurations where GCC still builds.
> > For more details:
> > http://people.linaro.org/~christophe.lyon/cross-validation/gcc/trunk/230087/report-build-info.html
> >
>
> This also causes failu
I've committed this patch to trunk. It implements a partitioning optimization
for a loop partitioned over both vector and worker axes. We can elide the inner
vector partitioning state propagation, if there are no intervening instructions
in the worker-partitioned outer loop other than the fork
> I've been unable to introduce a testcase for this. The difficulty is we want
> to check an rtl dump from the acceleration compiler, and there doesn't
> appear to be existing machinery for that in the testsuite. Perhaps
> something to be added later?
I haven't tried it, but doesn't
/* { dg-optio
On 11/10/2015 11:33 PM, Robert Suchanek wrote:
The reason behind the failures is that the terminated_this_insn had
a different number of consecutive registers (and mode) to the input
operand in a move currently being considered for tying. In the fix,
I allow tying only if there is matching numbe
On Nov 10, 2015, Alan Lawrence wrote:
> FAIL: gcc.target/aarch64/aapcs64/func-ret-4.c execution, -O2
Ugh, sorry. I even checked that testcase by hand before submitting the
patch, because I knew it took the paths I was changing, but I didn't
realize the stack store and load would amount to shif
On Mon, 9 Nov 2015, Sandra Loosemore wrote:
> If we're going to switch documentation formats, I'd rather we used DocBook.
> I've had to use "restructured text" before and found it really awkward.
I should perhaps note that the Sphinx extensions to reST are a lot better
documented than the ZWiki
On 11/10/2015 12:35 PM, Nathan Sidwell wrote:
> I've committed this to gomp4. In preparing the reworked firstprivate
> patch changes for gomp4's gimplify.c I discovered these testcases were
> passing by accident, and lacked a data clause.
It used to be if a reduction was on a parallel construct,
On 11/10/2015 03:58 PM, Alexandre Oliva wrote:
On Nov 10, 2015, Alan Lawrence wrote:
FAIL: gcc.target/aarch64/aapcs64/func-ret-4.c execution, -O2
Ugh, sorry. I even checked that testcase by hand before submitting the
patch, because I knew it took the paths I was changing, but I didn't
real
On 11/10/2015 09:36 AM, Marek Polacek wrote:
While both C and C++ FEs are able to reject e.g.
int a[__SIZE_MAX__ / sizeof(int)];
they are accepting code such as
int (*a)[__SIZE_MAX__ / sizeof(int)];
As Joseph pointed out, any construction of a non-VLA type whose size is half or
more of the addre
This patch removes the definition of TARGET_PROMOTE_PROTOTYPES from MIPS,
where it was defined as true, so that it now defaults to false.
Currently MIPS does prototype promotion in the caller and the callee and this
patch removes the TARGET_PROMOTE_PROTOTYPES macro definition so that
the promotion
Hi,
> > Bernd, do you think that this check would be sufficient and safe?
> > I'm not sure what would be better: check the mode, nregs plus perhaps
> > consider tying only if nregs == 1.
>
> Hmm, but shouldn't the regno still be the same? Or is this a case where
> we have a multi-word chain like a
On Tue, Nov 10, 2015 at 04:56:15PM -0500, Michael Meissner wrote:
> This patch d-form addressing to float/double scalars for the PowerPC that was
> added in ISA 3.0 (power9). This patch does not yet turn on D-form addressing
> as default. It is likely that patch #11, which will add limited d-form
Arghh, forgot the the patch once again.
[gcc]
2015-11-10 Michael Meissner
* config/rs6000/constraints.md (wb constraint): New constraint for
ISA 3.0 d-form scalar addressing.
* config/rs6000/rs6000.c (mode_supports_vmx_dform): Add support
for ISA 3.0 D
On Tue, Nov 10, 2015 at 11:02:04PM +0100, Bernd Schmidt wrote:
> On 11/09/2015 11:53 PM, Trevor Saunders wrote:
> >I don't really know him, but I don't really disagree with where he wants
> >to get to. However I think we work fairly different ways, and review
> >things differently. When I review
[resending, gcc-patches won't allow a screenshot attachment]
On 11/10/2015 03:27 PM, David Malcolm wrote:
Correct, readable and well-organized documentation are laudable goals...
but I think that, to a first approximation, we're already there: I just
feel that the content is hidden behind a poo
This patch adds some asserts of cpu and machine to fix an assert
testcase and makes the defines consistent with PPC64 Linux.
boottrapped on powerpc-ibm-aix7.1.0.0
* config/rs6000/aix.h (TARGET_OS_AIX_CPP_BUILTINS): Add cpu and
machine asserts. Update defines for 64 bit.
Thanks, David
Index: ai
On 11/10/2015 11:20 PM, Jeff Law wrote:
On 11/10/2015 11:16 AM, Andris Pavenis wrote:
One may need to execute extra steps after linking program. This is required
for example for DJGPP to run stubify.exe on file generated by linker.
The only way how to achieve was to use LINK_COMMAND_SPEC. It wo
On 09/12/2015 04:38 AM, Ajit Kumar Agarwal wrote:
-Original Message- From: Jeff Law [mailto:l...@redhat.com]
Sent: Thursday, September 10, 2015 3:10 AM To: Ajit Kumar Agarwal;
Richard Biener Cc: GCC Patches; Vinod Kathail; Shail Aditya Gupta;
Vidhumouli Hunsigida; Nagaraju Mekala Subjec
101 - 161 of 161 matches
Mail list logo
