On 09/19/2014 02:16 AM, Joseph S. Myers wrote:
On Thu, 11 Sep 2014, Maxim Ostapenko wrote:
In general, when cc1 or cc1plus ICE-es, we try to reproduce the bug by running
compiler 3 times and comparing stderr and stdout on each attempt with
respective ones that were gotten as the result of prev
On Fri, Sep 19, 2014 at 11:23:11AM +0100, Richard Sandiford wrote:
> tsaund...@mozilla.com writes:
> > diff --git a/gcc/ChangeLog b/gcc/ChangeLog
> > index c048672..5b27aa8 100644
> > --- a/gcc/ChangeLog
> > +++ b/gcc/ChangeLog
> > @@ -1,3 +1,8 @@
> > +2014-09-12 Trevor Saunders
> > +
> > + *
> On Sep 22, 2014, at 11:43 PM, Zhenqiang Chen wrote:
>
> Ping?
>
> Patch is attached for easy to apply.
Note I have been using an earlier version of this patch set in house and not
have found any issues with it.
Thanks,
Andrew
>
> Thanks!
> -Zhenqiang
>
>> -Original Message-
On Tue, Sep 23, 2014 at 8:48 AM, Ilya Enkovich wrote:
> 2014-09-22 22:51 GMT+04:00 Uros Bizjak :
>> On Mon, Sep 22, 2014 at 5:30 PM, Ilya Enkovich
>> wrote:
>>> On 19 Sep 18:21, Uros Bizjak wrote:
On Fri, Sep 19, 2014 at 2:53 PM, Ilya Enkovich
wrote:
>> > This patch adds i3
On Mon, 22 Sep 2014, Jan Hubicka wrote:
> Hi,
> it now seems to work, finaly :)
> Bootstrapped/regtested x86_64-linux, OK?
>
> * tree.c (int_bit_position): Move to...
> * tree.h (int_bit_position): ... here; implement using
> direct wide_int calculation instead of folding.
> Ind
On Mon, 22 Sep 2014, Uros Bizjak wrote:
> On Mon, Sep 22, 2014 at 5:30 PM, Ilya Enkovich wrote:
> > On 19 Sep 18:21, Uros Bizjak wrote:
> >> On Fri, Sep 19, 2014 at 2:53 PM, Ilya Enkovich
> >> wrote:
> >>
> >> >> > This patch adds i386 target hooks for Pointer Bounds Checker.
> >>
> >> > New ve
On Mon, Sep 15, 2014 at 11:56:03AM +0100, Andrew Stubbs wrote:
> On 15/09/14 10:46, Richard Earnshaw wrote:
> > Hmm, I wonder if arm_override_options should reject neon + (arch < 7).
>
> Is this more to your taste?
Is this really such a good idea? It causes carnage throughout the
testsuite if you
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied.
Richard.
2014-09-22 Richard Biener
* gimple-match-head.c (gimple_resimplify2): Also swap
comparison operands.
(gimple_simplify): Likewise.
Index: gcc/gimple-match-head.c
===
Hi,
The comment on widest_int_mode_for_size claims that it returns the
widest integer mode no wider than size. The implementation looks more
like it finds the widest integer mode smaller than size. Everywhere it
is used, the mode it is looking for is ultimately checked against an
expected alignme
Hi,
code removing references to indirectly-inlined functions can remove a
few too many when speculative devirtualization gets into the picture.
Although we already handle the case when speculative devirtualization
clones an edge with a CONST jump function by also cloning the
appropriate reference,
cess errors)
Excess errors:
/daten/aranym/gcc/gcc-20140923/gcc/testsuite/g++.old-deja/g++.law/ctors10.C:16:46:
error: call of overloaded 'basic_ostream(NULL)' is ambiguous
Andreas.
--
Andreas Schwab, SUSE Labs, sch...@suse.de
GPG Key fingerprint = 0196 BAD8 1CE9 1970 F4BE 1748 E4D
/ctors10.C -std=c++11 (test for excess errors)
Excess errors:
/daten/aranym/gcc/gcc-20140923/gcc/testsuite/g++.old-deja/g++.law/ctors10.C:16:46:
error: call of overloaded 'basic_ostream(NULL)' is ambiguous
Thanks, I'll adjust the new constructor.
On 19/09/14 00:23 +0200, Patrick Wollgast wrote:
Index: libstdc++-v3/acinclude.m4
===
--- libstdc++-v3/acinclude.m4 (Revision 214408)
+++ libstdc++-v3/acinclude.m4 (Arbeitskopie)
@@ -2321,7 +2321,17 @@ AC_DEFUN([GLIBCXX_ENABLE_VT
Hi,
Attached please fined the updated patch.
Cheers,
Felix
On Tue, Sep 23, 2014 at 10:48 AM, Yangfei (Felix) wrote:
>> On 09/22/14 08:40, Felix Yang wrote:
>> > Hi,
>> >
>> > I find that update_equiv_regs in ira.c sets the wrong EQUIV
>> > reg-note for pseudo with more than one definit
Hi,
Ignore the previous message.
Attached please find the updated patch.
Bootstrapped on x86_64-suse-linux. Please apply this patch if OK for trunk.
Index: gcc/ChangeLog
===
--- gcc/ChangeLog(revision 215500)
+++ gcc/
On 15 September 2014 15:48, Wilco Dijkstra wrote:
> Hi,
>
> This patch improves the register move costs for 128-bit types.
>
> OK for commit?
>
> ChangeLog:
> 2014-09-15 Wilco Dijkstra
>
> * gcc/config/aarch64/aarch64.c (aarch64_register_move_cost):
> Add register move costs for
As promised this allows you to define custom predicate expressions
(or operators), like one modeled after
tree-ssa-forwprop.c:lookup_logical_inverted_value:
(match logical_inverted_value
(bit_not truth_valued_p@0)
(logical_inverted_value @0))
(match logical_inverted_value
(eq @0 integer_zerop
Hello,
Patch in the bottom extends pmov insns.
Bootstrapped.
AVX-512* tests on top of patch-set all pass
under simulator.
Is it ok for trunk?
gcc/
* config/i386/sse.md
(define_insn "avx2_v16qiv16hi2"): Add masking.
(define_insn "avx512bw_v32qiv32hi2"): New.
(defin
Hello,
Patch in the bottom extends insert insn
patterns.
It also removes some exapnds.
Bootstrapped.
AVX-512* tests on top of patch-set all pass
under simulator.
Is it ok for trunk?
gc
On 19 September 2014 18:28, Wilco Dijkstra wrote:
> This patch makes -fsched-pressure the default on AArch64, like on ARM. This
> improves performance and
> reduces codesize due to fewer unnecessary spills.
>
> OK for commit?
>
> ChangeLog:
> 2014-09-19 Wilco Dijkstra
>
> * gcc/common/
Hello,
Patch in the bottom extends more shuffle patterns.
Bootstrapped.
AVX-512* tests on top of patch-set all pass
under simulator.
Is it ok for trunk?
gcc/
* config/i386/i386.c
(ix86_expand_args_builtin): Handle CODE_FOR_sse2_shufpd,
CODE_FOR_sse2_sse2_shufpd_mask, CODE
On 22/09/14 23:51 +0200, François Dumont wrote:
New patch in a couple of day then.
OK, thanks.
On 21/09/14 23:29 +0200, François Dumont wrote:
With all those modifications I have been able to run all testsuite
in profile mode with success.
I've looked over the patch and it looks fine.
I don't know the details of the Profile Mode, so if you're happy that
these changes are an improveme
Hello,
Bootstrapped.
AVX-512* tests on top of patch-set all pass
under simulator.
Is it ok for trunk?
gcc/
* config/i386/sse.md
(define_c_enum "unspec"): Add UNSPEC_PSH
Hello,
Patch in the bottom extends truncation insn patterns.
Bootstrapped.
AVX-512* tests on top of patch-set all pass
under simulator.
Is it ok for trunk?
gcc/
* config/i386/s
On 22 September 2014 12:16, Alan Lawrence wrote:
> Ok thanks Jeff. In that case I think I should draw this to the attention of
> the AArch64 maintainers to check the testsuite updates are OK before I
> commit...?
OK with me.
Cheers
/Marcus
> Bootstrapped.
> AVX-512* tests on top of patch-set all pass
> under simulator.
>
> Is it ok for trunk?
Forgot to mention, that patch adds support for PSHUF[HL]W
insns.
--
Thanks, K
On Mon, Sep 22, 2014 at 10:59:38AM +0200, Andreas Arnez wrote:
> > + sub_qual = TYPE_QUAL_RESTRICT;
> > + if ((sub_quals & ~sub_qual) != TYPE_UNQUALIFIED
> > + && get_qualified_type (type, sub_quals & ~sub_qual) != NULL_TREE)
> > +return true;
> > +
> > + sub_qual = TYPE_QUAL_CONST & TY
This restructures outer if parsing to maintain a stack of active
ifs. This gets rid of all the reverse if stuff and fixes outer ifs
on (match ...)es.
What is left to be done is delaying 'for' lowering.
Richard.
2014-09-23 Richard Biener
* genmatch.c
(dt_simplify::gen): Adj
On 19 September 2014 08:51, Iain Buclaw wrote:
> On 4 August 2014 16:52, Ian Lance Taylor wrote:
>> On Sun, Aug 3, 2014 at 11:12 AM, Iain Buclaw wrote:
>>>
>>> - I haven't signed any copyright assignments to GCC. But I have
>>> papers from Donald ready to send across.
>>
>> Definitely necessary
On Mon, Sep 22 2014, Andreas Arnez wrote:
> On Sat, Sep 20 2014, Mark Wielaard wrote:
>
>> When adding DW_TAG_restrict_type I made a mistake when updating the
>> code that handled types with multiple modifiers. This patch fixes it
>> by putting the logic for finding the "sub-qualified" type in a s
On Tue, Sep 23, 2014 at 1:12 PM, Kirill Yukhin wrote:
> Patch in the bottom extends pmov insns.
>
> Bootstrapped.
> AVX-512* tests on top of patch-set all pass
> under simulator.
>
> Is it ok for trunk?
>
> gcc/
> * config/i386/sse.md
> (define_insn "avx2_v16qiv16hi2"): Add maskin
On Tue, Sep 23, 2014 at 1:17 PM, Kirill Yukhin wrote:
> Hello,
> Patch in the bottom extends insert insn patterns.
> It also removes some exapnds.
>
> Bootstrapped.
> AVX-512* tests on top of patch-set all pass
> under simulator.
>
> Is it ok for trunk?
>
> gcc/
> * config/i386/i386.c
>
On Tue, Sep 23, 2014 at 1:22 PM, Kirill Yukhin wrote:
> Patch in the bottom extends more shuffle patterns.
>
> Bootstrapped.
> AVX-512* tests on top of patch-set all pass
> under simulator.
>
> Is it ok for trunk?
>
> gcc/
> * config/i386/i386.c
> (ix86_expand_args_builtin): Handl
On Tue, Sep 23, 2014 at 1:32 PM, Kirill Yukhin wrote:
> Patch in the bottom extends truncation insn patterns.
>
> Bootstrapped.
> AVX-512* tests on top of patch-set all pass
> under simulator.
>
> Is it ok for trunk?
>
> gcc/
> * config/i386/sse.md
> (define_insn
> "_fix_n
On Tue, Sep 23, 2014 at 1:30 PM, Kirill Yukhin wrote:
> Bootstrapped.
> AVX-512* tests on top of patch-set all pass
> under simulator.
>
> Is it ok for trunk?
>
> gcc/
> * config/i386/sse.md
> (define_c_enum "unspec"): Add UNSPEC_PSHUFHW, UNSPEC_PSHUFLW.
> (define_insn "av
On 4 September 2014 15:14, Jiong Wang wrote:
> this patch enabled stack shrink-wrap support on AArch64.
>
> no regression on aarch64-none-elf bare-metal.
> aarch64 bootstrap OK.
>
> ok to install?
>
> 2014-09-04 Renlin Li
>
> gcc/
> * config/aarch64/aarch64.md (return): New expand.
> (simple_r
Hi,
in this C++14 rejects valid issue, we reject the testcase with "‘x’ was
not declared in this scope", because in cp_parser_initializer_list we
tentatively try to parse what follows the '[' as a
cp_parser_constant_expression. As far as I can see, we can easily solve
the issue with the appro
On Mon, Sep 22, 2014 at 5:35 PM, Michael Meissner
wrote:
> I was working to support enhancing addressing modes in the the compiler that
> is
> indended to better support load fusion in power8, and to finally allow
> DFmode/SFmode scalar values in the traditional altivec registers. I noticed
> th
On Thu, Sep 18, 2014 at 8:57 AM, Alan Lawrence wrote:
> Patch 12 of 14 (https://gcc.gnu.org/ml/gcc-patches/2014-09/msg01475.html)
> will break bigendian targets implementing vec_shr. This is a PowerPC
> parallel of patch 13 of 14
> (https://gcc.gnu.org/ml/gcc-patches/2014-09/msg01477.html) for AAr
Hi Cesar!
On Mon, 15 Sep 2014 16:35:02 -0700, Cesar Philippidis
wrote:
> This patch adds initial support for the OpenACC routine directive.
Thanks!
> It's
> not complete just yet because it doesn't implement any of the optional
> clauses, except for the optional function/subroutine name. As su
On 23/09/14 11:48 +0200, Andreas Schwab wrote:
FAIL: g++.old-deja/g++.law/ctors10.C -std=c++11 (test for excess errors)
Excess errors:
/daten/aranym/gcc/gcc-20140923/gcc/testsuite/g++.old-deja/g++.law/ctors10.C:16:46:
error: call of overloaded 'basic_ostream(NULL)' is ambiguous
Fix
This is a first cleanup step towards lowering for late, not during
parsing.
This intermediate step removes the e_operation indirection and
makes parse_for temporarily insert operators it defines so we
can avoid lazily accepting anything as user-defined and thus
report proper error locations.
Boo
Committed to branch dmalcolm/jit:
In the initial commit on the JIT branch I moved
gcc/ipa-pure-const.c: register_hooks to be a method of
class pass_ipa_pure_const and the static local "init_p" within it
to be member data of the class, to ensure that the pass can be rerun
by separate invocations wi
On 03 Sep 16:19, Vladimir Makarov wrote:
> On 2014-08-29 2:47 AM, Ilya Enkovich wrote:
> >Seems your patch doesn't cover all cases. Attached is a modified
> >patch (with your changes included) and a test where double constant is
> >wrongly rematerialized. I also see in ira dump that there is stil
Jonathan Wakely writes:
> This adds move and swap functions to the iostream classes.
>
> Although this is a pretty large patch, it's a pure addition that only
> affects C++11 mode, and should have no effect on existing code because
> it won't be moving or swapping streams.
>
> I wanted to use C++
On 23 Sep 09:51, Richard Biener wrote:
> On Mon, 22 Sep 2014, Uros Bizjak wrote:
>
> > On Mon, Sep 22, 2014 at 5:30 PM, Ilya Enkovich
> > wrote:
> > > On 19 Sep 18:21, Uros Bizjak wrote:
> > >> On Fri, Sep 19, 2014 at 2:53 PM, Ilya Enkovich
> > >> wrote:
> > > +static int
> > > +ix86_initializ
The following implements very simple caret diagnostics. A
slow implementation is possible as we're only emitting
fatal errors (and thus at most one).
Committed.
Richard.
2014-09-23 Richard Biener
* genmatch.c (error_cb): Implement simple caret diagnostics.
Index: gcc/genmatch.c
==
On Tue, 23 Sep 2014, Ilya Enkovich wrote:
> On 23 Sep 09:51, Richard Biener wrote:
> > On Mon, 22 Sep 2014, Uros Bizjak wrote:
> >
> > > On Mon, Sep 22, 2014 at 5:30 PM, Ilya Enkovich
> > > wrote:
> > > > On 19 Sep 18:21, Uros Bizjak wrote:
> > > >> On Fri, Sep 19, 2014 at 2:53 PM, Ilya Enkovic
Attached patch introduces an optional warning for an OMP parallel/task/teams
construct without explicit default(none). This would effectively catch the
number one reason of easy-to-avoid OMP bugs in our project.
Tested with check-fortran, full bootstrap & check in progress. OK for trunk if
this
On Mon, Sep 22, 2014 at 11:44:57AM -0700, Konstantin Serebryany wrote:
> re-sending with the patch compressed:
>
> === gcc/testsuite/ChangeLog
> 2014-09-22 Kostya Serebryany
>
> Update to match the changed asan API.
> * asan.c (asan_global_struct): Update the __asan
On 23/09/14 15:58 +0200, Rainer Orth wrote:
This patch broke Solaris bootstrap with Sun ld: when linking
libstdc++.so, ld complains
ld: fatal: libstdc++-symbols.ver-sun: 4520: symbol 'std::basic_ios
>::move(std::basic_ios >&&)': symbol version conflict
and many more. In that case, I find that
On Tue, Sep 23, 2014 at 3:54 PM, Ilya Enkovich wrote:
> Here is a patch which combines results of my and Vladimir's work on EBX
> enabling.
>
> It works OK for SPEC2000 and SPEC2006 on -Ofast + LTO. It passes bootstrap
> but there are few new failures in make check.
>
> gcc.target/i386/pic-1.c
Committed to branch dmalcolm/jit:
gcc/jit/ChangeLog.jit:
* docs/conf.py (__read_file): New helper function, for
extracting...
(gcc_BASEVER): New variable, read from "BASE-VER" in gcc src dir.
(gcc_DEVPHASE): Likewise, from file "DEV-PHASE".
(gcc_DATESTAMP):
Hi!
This patch changes get_array_descr_info langhook in Fortran FE not to create
VAR_DECLs, because the langhook is only called with -g and not with -g0
and thus DECL_UIDs go out of sync too early.
For VTA, we allow DECL_UID changes, but lhd_set_assembler_name sometimes
emits DECL_UID into symbol
On Tue, Sep 23, 2014 at 05:54:37PM +0400, Ilya Enkovich wrote:
> use fixed EBX at least until we make sure pseudo PIC doesn't harm debug
> info generation. If we have such option then gcc.target/i386/pic-1.c and
For debug info, it seems you are already handling this in
delegitimize_address target
On Tue, Sep 23, 2014 at 02:16:58PM +, VandeVondele Joost wrote:
> Attached patch introduces an optional warning for an OMP
> parallel/task/teams construct without explicit default(none). This would
> effectively catch the number one reason of easy-to-avoid OMP bugs in our
> project.
a) I don
My patch to fix a memory leak with target_globals accidentally
introduced a double destructor call for default_target_ira. Normally
this was benign because free_register_move_costs clears the freed
pointers afterwards. However, with LTO -O3 that clearing could get
inlined into the destructor and
On 09/23/2014 02:07 AM, Uros Bizjak wrote:
> On Tue, Sep 23, 2014 at 3:26 AM, Vladimir Makarov wrote:
>> The previous patch to solve PR61360 fixed the problem in IRA (it was
>> easier for me to do as I know the code well)
>>
>> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61360
>>
>> Although
On Tue, 2014-09-23 at 14:17 +0200, Andreas Arnez wrote:
> > Also note that the logic wouldn't scale too well for yet more
> > qualifiers...
>
> Considering this, I've tried a different approach below. What do you
> think?
Having support in tree.c instead of doing it by hand in dwarf2out.c is
cer
On Tue, Sep 23, 2014 at 4:52 PM, Vladimir Makarov wrote:
> On 09/23/2014 02:07 AM, Uros Bizjak wrote:
>> On Tue, Sep 23, 2014 at 3:26 AM, Vladimir Makarov
>> wrote:
>>> The previous patch to solve PR61360 fixed the problem in IRA (it was
>>> easier for me to do as I know the code well)
>>>
>>>
> Hi,
>
> code removing references to indirectly-inlined functions can remove a
> few too many when speculative devirtualization gets into the picture.
> Although we already handle the case when speculative devirtualization
> clones an edge with a CONST jump function by also cloning the
> appropri
> a) I don't like the option name, -Womp-no-default-clause would be IMHO better
a bit related to b), I first had -Womp-no-default-none-clause but it becomes
long and contains twice 'no' (also consider the no-omp-no-default... form).
If you like that more, I'll make that change (see below) other
OK.
Jason
Hi all,
Some intrinsics had the wrong name (inconsistent with the NEON
intrinsics spec). This patch fixes that and adds the vrndx_f32 and
vrndxq_f32 intrinsics that were missing.
These map down to vrintx.f32 NEON instructions (d and q forms). We
already had builtins defined for them, just the
On Tue, Sep 23 2014, Mark Wielaard wrote:
> On Mon, Sep 22, 2014 at 10:59:38AM +0200, Andreas Arnez wrote:
>> > + sub_qual = TYPE_QUAL_RESTRICT;
>> > + if ((sub_quals & ~sub_qual) != TYPE_UNQUALIFIED
>> > + && get_qualified_type (type, sub_quals & ~sub_qual) != NULL_TREE)
>> > +return t
On 23.09.2014 16:27, Jakub Jelinek wrote:
This patch changes get_array_descr_info langhook in Fortran FE not to create
VAR_DECLs, because the langhook is only called with -g and not with -g0
and thus DECL_UIDs go out of sync too early.
For VTA, we allow DECL_UID changes, but lhd_set_assembler_nam
Maybe the original patch is better? Or maybe it should reconfigure the FPU
instead of erroring out? But reconfigure it to what?
Andrew
From: James Greenhalgh [james.greenha...@arm.com]
Sent: 23 September 2014 09:27
To: Stubbs, Andrew
Cc: Richard Earnshaw;
On 09/23/2014 11:02 AM, Uros Bizjak wrote:
> On Tue, Sep 23, 2014 at 4:52 PM, Vladimir Makarov wrote:
>> On 09/23/2014 02:07 AM, Uros Bizjak wrote:
>>>
>> Uros, my patch does not result in PR60704 (I tested it before submitting
>> the patch).
> No, we didn't understand each other. The fix for PR60
On Tue, Sep 23, 2014 at 5:02 PM, Uros Bizjak wrote:
>>> On Tue, Sep 23, 2014 at 3:26 AM, Vladimir Makarov
>>> wrote:
The previous patch to solve PR61360 fixed the problem in IRA (it was
easier for me to do as I know the code well)
https://gcc.gnu.org/bugzilla/show_bug.cgi?
On Tue, Sep 23, 2014 at 05:20:34PM +0200, Tobias Burnus wrote:
> On 23.09.2014 16:27, Jakub Jelinek wrote:
> >This patch changes get_array_descr_info langhook in Fortran FE not to create
> >VAR_DECLs, because the langhook is only called with -g and not with -g0
> >and thus DECL_UIDs go out of sync
On 23/09/14 16:07, Kyrill Tkachov wrote:
Hi all,
Some intrinsics had the wrong name (inconsistent with the NEON
intrinsics spec). This patch fixes that and adds the vrndx_f32 and
vrndxq_f32 intrinsics that were missing.
For reference, the NEON intrinsics spec can be found at:
http://infocente
Segher Boessenkool writes:
> On Thu, Sep 18, 2014 at 01:44:55PM -0500, Segher Boessenkool wrote:
>> I am testing a patch that is just
>>
>>
>> diff --git a/contrib/dg-extract-results.py b/contrib/dg-extract-results.py
>> index cccbfd3..3781423 100644
>> --- a/contrib/dg-extract-results.py
>> +++
On Tue, Sep 23, 2014 at 5:22 PM, Vladimir Makarov wrote:
>>> You are right constrain_operands is not upto LRA possibilities and we
>>> should make the following change:
>>>
>>> Index: recog.c
>>> ===
>>> --- recog.c (revision 21
On Tue, Sep 23, 2014 at 04:33:19PM +0100, Richard Sandiford wrote:
> FWIW, the \S+ thing was deliberate. When one test is run multiple times
> with different options, those options aren't necessarily tried in
> alphabetical order. The old sh/awk script therefore used just the test
> name as the k
Jan Hubicka writes:
> +/* Like bit_position, but return as an integer. It must be representable in
> + that way (since it could be a signed value, we don't have the
> + option of returning -1 like int_size_in_byte can. */
> +
> +static inline HOST_WIDE_INT
> +int_bit_position (const_tree fie
Committed to branch dmalcolm/jit:
I wrote toyvm to be run standalone, but with
63ff70786e01bd6265e9e7ac1052e4cb8fe5dd38 it's also being run by
jit.exp in the testsuite.
Fix it up so that it meaningfully exercises all this code when run in
this way.
This brings the number of expected passes in ji
On 09/22/14 00:40, Ilya Enkovich wrote:
Bounds don't have to vary for different pointers. E.g. p and p + 1
always have equal bounds. In this particular case we have function
pointers and all of them have default bounds.
OK. It looked a bit odd and I wanted to make sure there wasn't
something
On 09/23/14 08:23, Uros Bizjak wrote:
On Tue, Sep 23, 2014 at 3:54 PM, Ilya Enkovich wrote:
Here is a patch which combines results of my and Vladimir's work on EBX
enabling.
It works OK for SPEC2000 and SPEC2006 on -Ofast + LTO. It passes bootstrap but
there are few new failures in make ch
Jakub Jelinek writes:
> look at the sizes of .debug_info/.debug_loc sections with/without the
> patch, or use the locstat utility from elfutils
Not actually part of elfutils, but available either here:
https://github.com/pmachata/dwlocstat
... or packaged in Fedora.
Thanks,
PM
On 09/23/14 08:34, Jakub Jelinek wrote:
On Tue, Sep 23, 2014 at 05:54:37PM +0400, Ilya Enkovich wrote:
use fixed EBX at least until we make sure pseudo PIC doesn't harm debug
info generation. If we have such option then gcc.target/i386/pic-1.c and
For debug info, it seems you are already hand
On Fri, Sep 19, 2014 at 10:03 PM, Jeff Law wrote:
> On 09/19/14 13:36, Ilya Enkovich wrote:
>>
>> Hi,
>>
>> During my work on enabling pseudo PIC register I've found that cfg cleaunp
>> may remove lables with LABEL_PRESERVE_P set to 1. In my case I generated
>> SET_RIP during expand pass and cfg
On Tue, Sep 23, 2014 at 10:00:00AM -0600, Jeff Law wrote:
> On 09/23/14 08:34, Jakub Jelinek wrote:
> >On Tue, Sep 23, 2014 at 05:54:37PM +0400, Ilya Enkovich wrote:
> >>use fixed EBX at least until we make sure pseudo PIC doesn't harm debug
> >>info generation. If we have such option then gcc.tar
On 09/23/14 10:01, Steven Bosscher wrote:
On Fri, Sep 19, 2014 at 10:03 PM, Jeff Law wrote:
On 09/19/14 13:36, Ilya Enkovich wrote:
Hi,
During my work on enabling pseudo PIC register I've found that cfg cleaunp
may remove lables with LABEL_PRESERVE_P set to 1. In my case I generated
SET_RIP
On 09/23/14 10:03, Jakub Jelinek wrote:
On Tue, Sep 23, 2014 at 10:00:00AM -0600, Jeff Law wrote:
On 09/23/14 08:34, Jakub Jelinek wrote:
On Tue, Sep 23, 2014 at 05:54:37PM +0400, Ilya Enkovich wrote:
use fixed EBX at least until we make sure pseudo PIC doesn't harm debug
info generation. If
On September 23, 2014 5:33:35 PM CEST, Uros Bizjak wrote:
>On Tue, Sep 23, 2014 at 5:22 PM, Vladimir Makarov
>wrote:
>
You are right constrain_operands is not upto LRA possibilities and
>we should make the following change:
Index: recog.c
==
On 09/23/14 08:10, Ilya Enkovich wrote:
Please use fold_convert (size_ptr, build_fold_addr_expr (var)).
Is 'var' always accessed via a size_t effective type? Watch out
for TBAA issues if not. (if it is, why is 'var' not of type size_t
or size_t[]?)
var has pointer bounds type. I have to ini
This patch fixes the support for patching for instrumentation so that it
works with function splitting, by using the correct helper method for locating
the current section.
Added a couple tests cloned from existing function splitting tests that
I confirmed expose the problem.
Passes regression te
ok.
David
On Tue, Sep 23, 2014 at 10:27 AM, Teresa Johnson wrote:
> This patch fixes the support for patching for instrumentation so that it
> works with function splitting, by using the correct helper method for locating
> the current section.
>
> Added a couple tests cloned from existing funct
On Fri, Sep 19, 2014 at 03:39:39PM -0400, Jason Merrill wrote:
> On 09/19/2014 09:29 AM, Marek Polacek wrote:
> >But we also started to warn on
> >CPP_KEYWORD and two others: "case value not in enumerated type".
> >Fixed by moving CPP_KEYWORD, CPP_TEMPLATE_ID, and CPP_NESTED_NAME_SPECIFIER
> >into
On 09/23/14 04:51, Felix Yang wrote:
Hi,
Ignore the previous message.
Attached please find the updated patch.
Bootstrapped on x86_64-suse-linux. Please apply this patch if OK for trunk.
Index: gcc/ChangeLog
===
--- g
Hello!
Attached patch avoids triggering denormal exceptions when FP insns are
used to check for non-zero denormal values.
2014-09-23 Uros Bizjak
PR target/58757
* gcc.dg/c11-true_min-1.c (checkz): Add.
(main): Call checkz to check value for zero.
Patch was tested on x86_64 {,-m32
On Tue, 23 Sep 2014, Uros Bizjak wrote:
> Hello!
>
> Attached patch avoids triggering denormal exceptions when FP insns are
> used to check for non-zero denormal values.
But I thought the point of the test was to verify that the compiler's
understanding of existence of subnormal values was cons
On 09/23/14 00:31, Ilya Enkovich wrote:
I did this change a couple of years ago and don't remember exactly
what problem was caused by PARALLEL. But from my comment it seems
parallel lead to values in BND0 and BND1 not to be actually defined by
call from DF point of view. I'll try to reproduce
On Tue, Sep 23, 2014 at 7:57 PM, Joseph S. Myers
wrote:
>> Attached patch avoids triggering denormal exceptions when FP insns are
>> used to check for non-zero denormal values.
>
> But I thought the point of the test was to verify that the compiler's
> understanding of existence of subnormal valu
On Tue, Sep 23, 2014 at 7:16 AM, Jakub Jelinek wrote:
> On Mon, Sep 22, 2014 at 11:44:57AM -0700, Konstantin Serebryany wrote:
>> re-sending with the patch compressed:
>>
>> === gcc/testsuite/ChangeLog
>> 2014-09-22 Kostya Serebryany
>>
>> Update to match the changed asan AP
On 09/17/14 03:18, Zhenqiang Chen wrote:
Hi,
When assign_hard_reg, we always check_hard_reg_p, which has code
if (! TEST_HARD_REG_BIT (profitable_regs, hard_regno))
return false;
i.e. If a hard_regno is not in profitable_regs, we can not allocate it to
the ira_allocno_t A.
So the conflict
This patch is by Jakub Jelinek, and was originally posted here:
https://gcc.gnu.org/ml/gcc-patches/2013-09/msg01098.html
Parts of the patch subsequently landed on mainline as part of the
following patch:
https://gcc.gnu.org/ml/gcc-patches/2013-10/msg00505.html
But not the OpenMP "target" pa
Hi,
The upcoming patch series constitutes our current (still in-progress)
implementation of run-time support for OpenACC 2.0 in libgomp. We've
tried to build on top of the (also currently WIP) support for OpenMP
4.0's "target" construct, sharing code where possible: because of this,
I've also prep
This patch is by Michael Zolotukhin and was originally posted here:
https://gcc.gnu.org/ml/gcc-patches/2013-09/msg01469.html
It contains an initial implementation of plugin support for libgomp,
for implementing different hardware devices for pieces of accelerated
code to be offloaded to.
I als
1 - 100 of 143 matches
Mail list logo
