Thanks Jeff for reviewing,
I forget to mention that I do not have write access
to gcc.
Can you install for me ?
Cheers
Zhouyi
> -Original Messages-
> From: "Jeff Law"
> Sent Time: Tuesday, December 23, 2014
> To: "Zhouyi Zhou" , gcc-patches@gcc.gnu.org
> Cc: "Zhouyi Zhou"
> Subject: Re
Ping.
Teresa
On Fri, Dec 19, 2014 at 5:40 PM, Teresa Johnson wrote:
> Passes regression tests, ok for google 4_9?
>
> Thanks,
> Teresa
>
> 2014-12-19 Teresa Johnson
>
> Google ref b/18800499
> * cgraph.c (cgraph_can_remove_if_no_direct_calls_and_refs_p): Make
> consiste
Hi!
On Tue, 11 Nov 2014 13:53:23 +, Julian Brown
wrote:
> On Tue, 23 Sep 2014 19:19:31 +0100
> Julian Brown wrote:
>
> > This patch contains the bulk of the OpenACC 2.0 runtime support,
> > building around, or on top of, the OpenMP 4.0 support (as previously
> > posted or already extant up
On 12/22/2014 04:51 PM, Paolo Carlini wrote:
Well, if they are both right, could we maybe do something closer to
clang, instead, thus not consider the for loop a range-based for loop
only because after the initializer there is a colon when the decl is
error_mark_node?
I think we want to tell th
Hi,
On 12/22/2014 10:12 PM, Jason Merrill wrote:
We could also peek for a colon in cp_parser_init_declarator after
parsing the initializer, and give an error then.
I see.
For
the record, clang vs edg appear to handle this case differently: clang
considers it a wrong for loop, edg a wrong range
So, I have a rough cut of a new feature to add pragma GCC unroll support to
gcc. It is safe wrt the test suite and code-gen, but there are some corners
that I need help or suggestions on how to round off nicely.
Things to think about, can we put this in even if we have TODO support for C++
tem
On 12/22/2014 03:11 PM, Paolo Carlini wrote:
That would be the place. But, at that point, it could still be a normal
for loop, thus we can't just give an error.
Ah, yes.
Assigning error_mark_node on
the other hand is correct, because later, if we find the colon, we
realize that the declaratio
> I think I've fixed them now though, and am committing the attached
> patch to trunk.
This commit (or a friend of it) breaks bootstrap on darwin
duplicate symbol std::__cxx11::time_get > >::do_get(std::istreambuf_iterator >, std::istreambuf_iterator
>, std::ios_base&, std::_Ios_Iostate&, tm*, c
2014-12-18 0:14 GMT+01:00 Tobias Burnus :
> As testing by Alessandro revealed, vector subscripts weren't properly
> handled.
>
> This patch fixes the compiler side (or at least those issues I found). In
> particular, for expressions ("get") it wrongly passed a NULL pointer,
> additionally, I used t
Hi,
On 12/22/2014 07:55 PM, Jason Merrill wrote:
On 12/04/2014 07:51 AM, Paolo Carlini wrote:
Ideally, it would be nice to say something
more detailed about the invalid declaration, but that doesn't seem
trivial...
How about giving that error in cp_parser_init_declarator?
/* An `=' or an
PING
On 18 December 2014, 00:14, Tobias Burnus wrote:
As testing by Alessandro revealed, vector subscripts weren't properly
handled.
This patch fixes the compiler side (or at least those issues I found).
In particular, for expressions ("get") it wrongly passed a NULL
pointer, additionally, I
Hi,
I've noticed that the test case mentioned in the PR hasn't been
committed. Moreover, setting -Os in sh/torture/pr58314.c doesn't
accomplish anything. Tested with
make -k check-gcc RUNTESTFLAGS="sh-torture.exp=pr58314*
--target_board=sh-sim
\{-m2/-ml,-m2/-mb,-m2a/-mb,-m4/-ml,-m4/-mb,-m4a/-ml
On 12/04/2014 07:51 AM, Paolo Carlini wrote:
Ideally, it would be nice to say something
more detailed about the invalid declaration, but that doesn't seem
trivial...
How about giving that error in cp_parser_init_declarator?
/* An `=' or an `(', or an '{' in C++0x, indicates an initializer.
Hi!
On Tue, 11 Nov 2014 13:53:23 +, Julian Brown
wrote:
> On Tue, 23 Sep 2014 19:19:31 +0100
> Julian Brown wrote:
>
> > This patch contains the bulk of the OpenACC 2.0 runtime support,
> > building around, or on top of, the OpenMP 4.0 support (as previously
> > posted or already extant up
Hi!
On Fri, 13 Sep 2013 15:29:30 +0400, "Michael V. Zolotukhin"
wrote:
> Here is the first patch for adding plugins support in libgomp - could you
> please
> take a look at it?
> --- a/libgomp/target.c
> +++ b/libgomp/target.c
> +/* This structure describes accelerator device.
> + It contai
>> here is a small patch which enhances the diagnostics for the intrinsic
>> KIND function. Regtested on x86_64-unknown-linux-gnu. Ok for trunk?
>
> OK.
Thanks, committed as r219027.
Cheers,
Janus
Hi!
On Tue, 11 Nov 2014 13:53:23 +, Julian Brown
wrote:
> On Tue, 23 Sep 2014 19:19:31 +0100
> Julian Brown wrote:
>
> > This patch contains the bulk of the OpenACC 2.0 runtime support,
> > building around, or on top of, the OpenMP 4.0 support (as previously
> > posted or already extant up
On 12/11/14 10:06, Jan Hubicka wrote:
Hi,
in firefox .optimized dumps one can see few places where __builtin_unreachable
is called (as a result of devirtualization code proving the code path to be
undefined). There is usually some argument setup for the parameters of
__builtin_unreachable that a
Hi!
On Tue, 11 Nov 2014 13:53:23 +, Julian Brown
wrote:
> On Tue, 23 Sep 2014 19:19:31 +0100
> Julian Brown wrote:
> > This patch contains the bulk of the OpenACC 2.0 runtime support,
> > building around, or on top of, the OpenMP 4.0 support (as previously
> > posted or already extant upstr
Hi!
On Tue, 11 Nov 2014 13:53:23 +, Julian Brown
wrote:
> On Tue, 23 Sep 2014 19:19:31 +0100
> Julian Brown wrote:
> > This patch contains the bulk of the OpenACC 2.0 runtime support,
> > building around, or on top of, the OpenMP 4.0 support (as previously
> > posted or already extant upstr
Hi,
minimally, fix _S_local_capacity for sizeof(_CharT) > 16. Probably we'll
come back to it in order to tune it more finely for char, wchar_t,
char16_t, and char32_t.
Tested x86_64-linux.
Thanks,
Paolo.
///
2014-12-22 Paolo Carlini
* include/bits/basic_strin
Hi!
On Tue, 11 Nov 2014 13:53:23 +, Julian Brown
wrote:
> On Tue, 23 Sep 2014 19:19:31 +0100
> Julian Brown wrote:
>
> > This patch contains the bulk of the OpenACC 2.0 runtime support,
> > building around, or on top of, the OpenMP 4.0 support (as previously
> > posted or already extant up
On Mon, Dec 22, 2014 at 04:48:02PM +0100, Janus Weil wrote:
>
> here is a small patch which enhances the diagnostics for the intrinsic
> KIND function. Regtested on x86_64-unknown-linux-gnu. Ok for trunk?
>
> Cheers,
> Janus
>
>
> 2014-12-22 Janus Weil
>
> PR fortran/63363
> * check
Hi!
We had committed to gomp-4_0-branch code to handle TO_PSET memory mapping
(used with Fortran allocatable arrays). It turns out that is not
actually useful; committed to gomp-4_0-branch in r219022:
commit b2c3a33803b074052c5178fb1b6cabbd834cfa72
Author: tschwinge
Date: Mon Dec 22 17:12:40
On Dec 20, 2014, at 8:58 AM, FX wrote:
> This patch below allows ubsan to run when libstdc++ is built but not
> installed (something which happens on darwin, in particular). This fixes all
> 658 ubsan failures when run in this particular configuration.
>
> OK to commit?
Ok.
On 12/19/14 16:45, John David Anglin wrote:
I believe that this version addresses the above issues. While there
may be some opportunity to
optimize the handling of sibling call arguments, I think it is more
important to get the overall logic
correct. Also, it's obviously a rare situation for th
On 12/09/14 14:09, John David Anglin wrote:
On 12/9/2014 3:00 PM, Jeff Law wrote:
Thoughts on using both patches to ensure the we don't have extraneous
entries in the GEN set and that we're adding more stuff to the KILL
set for sibcalls?
Maybe we should add all stores to the KILL set
for a sibl
Hi!
On Fri, 12 Dec 2014 16:59:21 +0100, I wrote:
> In r218677, I have committed a merge from trunk r217500 (2014-11-13) into
> gomp-4_0-branch.
>
> This spans the "offloading" commits in trunk (plus two interleaved
> non-"offloading" commits).
This failed to update the (new) intelmic plugin in l
On 12/21/14 23:34, Zhouyi Zhou wrote:
In function ira_flattening, conflict relationship will be rebuilt in case
of any new pseudo register has been created during ira_emit.
By adding the current object to OBJECTS_LIVE after traversing OBJECTS_LIVE, a
conflict
obj compare can be saved.
Bootstr
On Dec 22, 2014, at 4:50 AM, Iain Sandoe wrote:
> When we find "[" and we're in objcp mode, and the "[" introduces a lambda, we
> try to parse a message expression and if that fails we punt (thus we are
> unable to recognise lambdas).
Even though Jason reviewed it, I also took a look. Looks go
On Dec 22, 2014, at 4:08 AM, Iain Sandoe wrote:
> After the usual fun with tracking down a GTY(()) issue, it turned out to be
> triggered by 216263 (changes in the gtfiles for cp that were not reflected in
> objcp). However, it appears that the list was a bit out of sync in general.
>
> Probab
Hi!
On Wed, 17 Dec 2014 23:31:56 +0100, I wrote:
> On Wed, 17 Dec 2014 23:24:17 +0100, I wrote:
> > Committed to gomp-4_0-branch in r218839:
> >
> > commit 1c4f05a68c6d0d5b6137bb6d85a293d16727b389
> > Author: tschwinge
> > Date: Wed Dec 17 22:23:02 2014 +
> >
> > libgomp updates.
>
>
On 22/12/14 16:02 +0100, Kai Tietz wrote:
2014-12-20 3:28 GMT+01:00 Jonathan Wakely :
People keep hitting the problem that std::stoi and std::to_string are
not defined when using MinGW, because of the fix for
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=37522
If the problem is only with vswprin
Hi!
On Tue, 23 Sep 2014 19:19:31 +0100, Julian Brown
wrote:
> This patch contains the bulk of the OpenACC 2.0 runtime support,
> --- /dev/null
> +++ b/libgomp/plugin-nvptx.c
> @@ -0,0 +1,1854 @@
> +/* Plugin for NVPTX execution.
> +const char *
> +get_name (void)
> +{
> + return "nvidia";
> +
Hi,
On 12/04/2014 01:54 PM, Paolo Carlini wrote:
... oops, sent the wrong patch. See the below instead.
Paolo.
It occurs to me that a while ago I sent this patch: what do you think?
Is the diagnostic good enough?
https://gcc.gnu.org/ml/gcc-patches/2014-12/msg00376.html
https://gcc.gn
Hi!
On Thu, 18 Dec 2014 11:21:20 +0100, Jakub Jelinek wrote:
> On Wed, Dec 17, 2014 at 11:48:01PM +0100, Thomas Schwinge wrote:
> > I have another suggestion: on gomp-4_0-branch, we had already started
> > using a libgomp-test-support.exp file for a similar purpose. I now
> > changed your code o
Hi,
This adds another test case for the SH div0s insn utilization.
Tested with
make -k check-gcc RUNTESTFLAGS="sh.exp=pr52933* --target_board=sh-sim
\{-m2/-ml,-m2/-mb,-m2a/-mb,-m4/-ml,-m4/-mb,-m4a/-ml,-m4a/-mb}"
Committed as r219017.
Cheers,
Oleg
gcc/testsuite/ChangeLog:
PR target/52933
On 18/12/14 02:40 +0200, Ville Voutilainen wrote:
Currently finishing testing on Linux-x64. The only implementation change
in this patch is the initialization of the iterator variables in
__minmax_element,
otherwise it's just adding constexpr.
Thanks, committed to trunk.
commit b31a5c2c4b
Hi all,
here is a small patch which enhances the diagnostics for the intrinsic
KIND function. Regtested on x86_64-unknown-linux-gnu. Ok for trunk?
Cheers,
Janus
2014-12-22 Janus Weil
PR fortran/63363
* check.c (gfc_check_kind): Reject polymorphic and non-data arguments.
2014-12-22
On Mon, Dec 22, 2014 at 6:51 AM, Gerald Pfeifer wrote:
> On Friday 2014-12-12 14:47, Kirill Yukhin wrote:
>> This change mantions AVX-512* new instructions support in GCC: news
>> section of index.html and gcc-5/changes.html.
>
> Thanks Kirill, this looks fine with just one change: We have
> guid
> Hello.
>
> Following IPA ICF patch restricts thunk creation for static-chain thunks.
> Patch can bootstrap on x86_64-linux-pc and no new regression has been seen.
>
> Ready for thunk?
> Thanks,
> Martin
> >From b247009735cd294b3fd8b727905cb47b5cdaeb51 Mon Sep 17 00:00:00 2001
> From: mliska
>
On 12/18/2014 12:55 PM, Paolo Carlini wrote:
Thanks. I'm attaching what I just committed. This is a regression, I
suppose the patch is ok for 4_9-branch too, right?
Yes.
Jason
Hi!
I'm sending this again with some more people copied -- because I see
you're working on tree.h/tree-core.h flattening, or know you're familiar
with GCC plugins. ;-) Here is a question concerning both of that, where
I'd appreciate your input.
(That said, I don't find a "GCC plugins" person lis
OK.
On Friday 2014-12-12 14:47, Kirill Yukhin wrote:
> This change mantions AVX-512* new instructions support in GCC: news
> section of index.html and gcc-5/changes.html.
Thanks Kirill, this looks fine with just one change: We have
guidance not to link to vendors of proprietary software, so at
least
On 12/18/2014 08:40 PM, Jan Hubicka wrote:
2014-12-08 Martin Liska
* cgraph.h (symbol_table::allocate_cgraph_symbol): Summary UID
is filled up.
* symbol-summary.h: New file.
* gengtype.c (open_base_files): Add symbol-summary.h.
* toplev.c (general_init)
Committed to trunk.
commit f072f78db945365de5ac5f5c66a6ce936f02682a
Author: Jonathan Wakely
Date: Mon Dec 22 14:48:29 2014 +
* doc/xml/manual/status_cxx2011.xml: Update.
* doc/html/manual/status.html: Regenerate.
diff --git a/libstdc++-v3/doc/html/manual/status.html b/libstdc+
Richard,
I changed algorithm for bool pattern repair.
It turned out that ifcvt_local_dce phaase is required since for
test-case I sent you in previous mail vectorization is not performed
without dead code elimination:
For the loop
#pragma omp simd safelen(8)
for (i=0; i<512; i++)
{
float
> hmm.. I wasn't aware of a [> 4.2.1] arm-mach-o backend .. if that's a general
> issue, then I'll consider it for future patches.
Yes please.
> > Tristan (cc:ed) has prepared a patch for this issue which works on both
> > MAC OS and darwin), which I'll merge soon. This will address this PR.
>
>
hi Arnaud,
On 22 Dec 2014, at 13:53, Arnaud Charlet wrote:
> The change isn't quite right for iOS (aka arm-darwin), but the general idea
> is the right one.
hmm.. I wasn't aware of a [> 4.2.1] arm-mach-o backend .. if that's a general
issue, then I'll consider it for future patches.
> Tristan
> PR ada/64349
> * env.c (__gnat_environ): Use the _NSGetEnviron() interface to access
> the environment for Darwin.
OK if Tristan has no objections to the patch:
https://gcc.gnu.org/ml/gcc-patches/2014-12/msg01753.html
--
Eric Botcazou
Iain,
The change isn't quite right for iOS (aka arm-darwin), but the general idea
is the right one.
Tristan (cc:ed) has prepared a patch for this issue which works on both
MAC OS and darwin), which I'll merge soon. This will address this PR.
Arno
On 15/04/14 23:20 +0200, Rüdiger Sonderfeld wrote:
* libstdc++-v3/include/std/iomanip (_Get_time): New struct.
(get_time): New manipulator [ext.manip].
(operator>>): New overloaded function.
* libstdc++-v3/testsuite/27_io/manipulators/extended/get_time/char/1.cc:
* libstdc++-v3/testsuite/27_io/
On 15/04/14 23:19 +0200, Rüdiger Sonderfeld wrote:
+ TESTHEAD("check broken format");
+ iss.str("2014-04-14 01:09:35");
+ format = "%";
+ ret = tget.get(istreambuf_iterator(iss), end, iss, err, &time,
+ format.data(), format.data()+format.size());
+ VERIFY(err == ios_base::fa
Hi,
The principle of Objective-c/c++ is that they should be supersets of the parent
language. Thus, it is intended that valid c++ (or c) should be parsed by the
relevant objective-c/c++ FE.
While trying to build some external code, I found that lambdas confuse things
at present for objcp.
Wh
On Mon, Dec 22, 2014 at 12:48:08PM +0100, Thomas Schwinge wrote:
> In my understanding, we'd like to support the modes that either all
> compilers are installed (which is what a user will be using), or all are
> tested from their build trees. Or, do we also have to support the mode
> that only the
On 19/12/2014 19:01, Xingxing Pan wrote:
On 19/12/2014 18:38, Kyrill Tkachov wrote:
Hi Xingxin,
It seems that your mail client mangled this patch, at least the
following hunk doesn't apply, even when I try to get it from the web
archives.
Could you please resend it as an attachment perhaps?
Th
Hi,
Whilst working on another patch for ObjC++, I found that trunk ICEs on #include
(on Darwin always, on Linux if you tickle ggc).
After the usual fun with tracking down a GTY(()) issue, it turned out to be
triggered by 216263 (changes in the gtfiles for cp that were not reflected in
objcp).
Hi!
On Tue, 21 Oct 2014 21:13:23 +0400, Ilya Verbin wrote:
> This patchset would contain target-specific things to support offloading to
> the
> devices with Intel MIC architecture.
> Particularly: mkoffload tool, liboffloadmic library and a plugin for libgomp.
By chance (when tracking down a d
Hi Eric,
For the main executable, it is permitted for Darwin to access _environ
directly. For shared libraries, it is not (one is supposed use
_NSGetEnviron()).
At present, the more recent darwin ports are "getting away" with the omission
because they have a blanket setting of -Wl,-undefined,
Hi!
On Thu, 18 Dec 2014 18:41:18 +0100, Ilya Verbin wrote:
> 2014-12-18 16:27 GMT+01:00 Thomas Schwinge :
> > On Thu, 30 Oct 2014 14:40:01 +0300, Ilya Verbin wrote:
> >> This patch allows to run non-fallback 'make check-target-libgomp'. It
> >> passes to
> >> the host compiler additional -B op
On Mon, Dec 22, 2014 at 12:25:32PM +0100, Thomas Schwinge wrote:
> Hi!
>
> On Wed, 22 Oct 2014 22:57:01 +0400, Ilya Verbin wrote:
> > --- /dev/null
> > +++ b/gcc/config/i386/intelmic-mkoffload.c
> > @@ -0,0 +1,541 @@
> > +/* Offload image generation tool for Intel MIC devices.
>
> > +/* Shows if
On Mon, Dec 22, 2014 at 12:20:58PM +0100, Thomas Schwinge wrote:
> On Wed, 22 Oct 2014 22:57:01 +0400, Ilya Verbin wrote:
> > --- /dev/null
> > +++ b/gcc/config/i386/intelmic-mkoffload.c
> > @@ -0,0 +1,541 @@
> > +/* Offload image generation tool for Intel MIC devices.
>
> > +static const char *
Hi!
On Wed, 22 Oct 2014 22:57:01 +0400, Ilya Verbin wrote:
> --- /dev/null
> +++ b/gcc/config/i386/intelmic-mkoffload.c
> @@ -0,0 +1,541 @@
> +/* Offload image generation tool for Intel MIC devices.
> +/* Shows if we should compile binaries for i386 instead of x86-64. */
> +bool target_ilp32 =
Hi!
On Wed, 22 Oct 2014 22:57:01 +0400, Ilya Verbin wrote:
> --- /dev/null
> +++ b/gcc/config/i386/intelmic-mkoffload.c
> @@ -0,0 +1,541 @@
> +/* Offload image generation tool for Intel MIC devices.
> +static const char *
> +prepare_target_image (const char *target_compiler, int argc, char **arg
Revised version after Joseph's comments and latest libgcc changes.
gcc/ChangeLog
2014-12-22 Eric Botcazou
* config.gcc: Add Visium support.
* configure.ac: Likewise.
* configure: Regenerate.
* doc/extend.texi (interrupt attribute): Add Visium.
* doc/in
On 12/22/2014 10:53 AM, Martin Liška wrote:
Hello.
Following IPA ICF patch restricts thunk creation for static-chain thunks.
Patch can bootstrap on x86_64-linux-pc and no new regression has been seen.
Ready for thunk?
Thanks,
Martin
In previous mail, I forgot to add one obvious hunk that is q
NaNs are invalid values in Ada so the language doesn't specify how they should
be handled but GNAT nevertheless attempts to implement sensible semantics for
them. This patch extends this to the Min and Max attributes.
Tested on x86_64-suse-linux, applied on the mainline.
2014-12-22 Eric Botc
Hi!
On Mon, 15 Dec 2014 23:36:11 +0100, Jan Hubicka wrote:
> this is final version I comitted.
> PR lto/64043
> * tree-streamer.c (preload_common_nodes): Skip preloading
> of main_identifier_node, pid_type and optimization/option nodes.
> --- tree-streamer.c (revision 21872
pragma Volatile forces fields to be of nominal size in record types but we
nevertheless accept volatile oversized variables. This changes the former.
Tested on x86_64-suse-linux, applied on the mainline.
2014-12-22 Eric Botcazou
* gcc-interface/decl.c (gnat_to_gnu_field): Post the
pragma Atomic forces fields to be aligned in record types but we nevertheless
accept atomic misaligned standalone variables. This changes the latter.
Tested on x86_64-suse-linux, applied on the mainline.
2014-12-22 Eric Botcazou
* gcc-interface/decl.c (check_ok_for_atomic): Rename
Hello.
Following IPA ICF patch restricts thunk creation for static-chain thunks.
Patch can bootstrap on x86_64-linux-pc and no new regression has been seen.
Ready for thunk?
Thanks,
Martin
>From b247009735cd294b3fd8b727905cb47b5cdaeb51 Mon Sep 17 00:00:00 2001
From: mliska
Date: Mon, 22 Dec 201
> Hmm, I further revised comment in the patch since in try_combine, i2
> is always after i0/i1. The original comment is inaccurate about that.
OK.
--
Eric Botcazou
> 2014-12-22 Bin Cheng
>
> PR rtl-optimization/62151
> * combine.c (try_combine): New local variables local_elim_i1
> and local_elim_i0. Set elim_i1 and elim_i0 using the local
> version variables. Distribute notes from i0notes or i1notes
> using the local variavbles.
>
>
On Mon, Dec 22, 2014 at 3:54 PM, Bin.Cheng wrote:
> On Sat, Dec 20, 2014 at 8:18 PM, Eric Botcazou wrote:
>>> As described both in the PR and patch comments, this patch fixes PR62151 by
>>> setting right value to ELIM_I0/ELIM_I1 when distributing REG_DEAD notes from
>>> i0/i1. It is said that di
75 matches
Mail list logo
