On Mar 8, 2018, Jakub Jelinek wrote:
> An upper bound would be fine, count any potential spot where you introduce a
> view and if the upper bound fits into 8-bit value, use data1, if into 16-bit
> value, use data2, otherwise use data4.
[IEPM] [PR debug/84620] use constant form for DW_AT_GNU_ent
/i386/winnt.c (i386_pe_seh_cold_init): Use small pre-allocation
when the function accesses prior frames.
2018-03-09 Eric Botcazou
* gcc.c-torture/compile/20180309-1.c: New test.
--
Eric BotcazouIndex: config/i386/winnt.c
.. after a few hours of sleep, I realize that in principle we could
also call any_erroneous_template_args_p directly. In that case however,
I believe get_template_info does nothing 100% of the case, not sure we
want to do that.
Paolo.
On Fri, Mar 09, 2018 at 05:49:34AM -0300, Alexandre Oliva wrote:
> @@ -27699,6 +27780,9 @@ dwarf2out_source_line (unsigned int line, unsigned
> int column,
> {
> if (!RESETTING_VIEW_P (table->view))
> {
> + table->symviews_since_reset++;
> + if (table->s
On Wed, Feb 28, 2018 at 6:17 AM, Alexandre Oliva wrote:
> On Feb 21, 2018, Alexandre Oliva wrote:
>
>> On Feb 15, 2018, Szabolcs Nagy wrote:
>>> i see assembler slow downs with these location view patches
>>> i opened https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84408
>
>
>> [LVU] reset view at
On Fri, Mar 9, 2018 at 9:48 AM, Bin.Cheng wrote:
> On Wed, Feb 28, 2018 at 6:17 AM, Alexandre Oliva wrote:
>> On Feb 21, 2018, Alexandre Oliva wrote:
>>
>>> On Feb 15, 2018, Szabolcs Nagy wrote:
i see assembler slow downs with these location view patches
i opened https://gcc.gnu.org/b
It's looks fixed
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82965#c12 on mips64el.
Thanks.
On Mon, Feb 26, 2018 at 8:02 PM, Bin.Cheng wrote:
> Ping^2
>
> Thanks,
> bin
>
> On Mon, Feb 19, 2018 at 5:14 PM, Jakub Jelinek wrote:
>> Hi!
>>
>> Honza, do you think you could have a look at this P1
On Fri, Mar 09, 2018 at 09:48:45AM +, Bin.Cheng wrote:
> On Wed, Feb 28, 2018 at 6:17 AM, Alexandre Oliva wrote:
> > On Feb 21, 2018, Alexandre Oliva wrote:
> >
> >> On Feb 15, 2018, Szabolcs Nagy wrote:
> >>> i see assembler slow downs with these location view patches
> >>> i opened https:/
This patch fixes the ICE in check-local-shadow, but reverts behaviour to
the earlier ICE in pop_local_binding.
Check local shadow had a check when finding an outer local scope for
function_parm_scope. But that code didn't make sense and was
unreachable with the current testsuite. So I remove
On Thu, Mar 08, 2018 at 04:06:24PM -0500, Jason Merrill wrote:
> On Thu, Mar 8, 2018 at 1:01 PM, Jakub Jelinek wrote:
> > The C FE just warns and doesn't override builtins, but C++ FE
> > on say int __builtin_trap (); will override the builtin, so later
> > builtin_decl_explicit will return the bo
With -Os we inhibit most loop header copying which in turn will disable
any vectorization attempt either via -fopenmp or via -ftree-vectorize.
The following makes sure the aggressive gate in
should_duplicate_loop_header_p is not applied for force-vectorize loops.
There's still PARAM_MAX_LOOP_HEA
The following reverts the fix for PR84178 again and instead prunes those
bogus immediate uses from stmts we didn't yet insert. Micha has sth
in the works to clean up the mess somewhat in GCC 9.
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied.
Richard.
2018-03-09 Richard Biener
On Wed, 2018-02-14 at 23:36 +0100, FX wrote:
> I can confirm that, with the attached revised patch, a bootstrap with
> --enable-languages=c,c++,jit --enable-host-shared is successful on
> macOS.
>
> FX
Looks good to me; thanks for fixing.
Release managers:
I'd like to apply FX's patch here:
On Fri, Mar 09, 2018 at 09:14:13AM -0500, David Malcolm wrote:
> On Wed, 2018-02-14 at 23:36 +0100, FX wrote:
> > I can confirm that, with the attached revised patch, a bootstrap with
> > --enable-languages=c,c++,jit --enable-host-shared is successful on
> > macOS.
> >
> > FX
>
> Looks good to me
On 03/09/2018 01:45 AM, Alexandre Oliva wrote:
LRA gets very confused when non-addresses are passed as operands to
asms with address contraints. Even if other constraints are
available, and the operand is a perfect fit for them, we'd still
attempt to process the operand as an address, and fail
On Fri, Mar 9, 2018 at 10:25 AM, Paul Hua wrote:
> It's looks fixed
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82965#c12 on mips64el.
Hmm, is it fixed? or is it exposed now on mips64el? I read the latter
from the comment.
I think the issue is like explained, but haven't dug into when/why it
On Fri, Mar 9, 2018 at 4:02 AM, Jakub Jelinek wrote:
> On Fri, Mar 09, 2018 at 05:49:34AM -0300, Alexandre Oliva wrote:
>> @@ -27699,6 +27780,9 @@ dwarf2out_source_line (unsigned int line, unsigned
>> int column,
>> {
>> if (!RESETTING_VIEW_P (table->view))
>> {
>> +
You should be able to say
if (any_erroneous_template_args (type))
and I don't think you need the goto; the normal logic should handle
the truncated vecs fine.
On Fri, Mar 9, 2018 at 4:14 AM, Paolo Carlini wrote:
> .. after a few hours of sleep, I realize that in principle we could also
> call
Hi!
On Thu, 1 Mar 2018 13:17:01 -0800, Cesar Philippidis
wrote:
> To reduce the size of the final patch,
> I've separated all of the misc. function and variable renaming into this
> patch.
Yes, please always do such refactoring changes independently of other
functionality changes.
> This patc
On 03/09/2018 07:29 AM, Thomas Schwinge wrote:
> On Thu, 1 Mar 2018 13:17:01 -0800, Cesar Philippidis
> wrote:
>> To reduce the size of the final patch,
>> I've separated all of the misc. function and variable renaming into this
>> patch.
>
> Yes, please always do such refactoring changes indep
Hi all,
Currently when handling an invalid -march or -mcpu option on a toolchain
without an explicit --with-mode configuration
and compiling without an explicit -mthumb or -marm the arm specs end up calling
arm_target_thumb_only to determine
the "thumbness" of the target, which involves parsing
On 22/02/18 12:15, Martin Liška wrote:
On 02/22/2018 11:59 AM, Kyrill Tkachov wrote:
Hi Martin,
On 22/02/18 08:51, Martin Liška wrote:
On 02/21/2018 10:23 AM, Kyrill Tkachov wrote:
On arm we also support "native" as a value for -mcpu and -mtune. These are both
handled by
the arm_print_hint_
Hi,
On 09/03/2018 16:17, Jason Merrill wrote:
You should be able to say
if (any_erroneous_template_args (type))
Indeed, that's what I tried to express earlier today, after a cappuccino ;)
and I don't think you need the goto; the normal logic should handle
the truncated vecs fine.
It should
On 03/09/2018 04:43 PM, Kyrill Tkachov wrote:
On 22/02/18 12:15, Martin Liška wrote:
On 02/22/2018 11:59 AM, Kyrill Tkachov wrote:
Hi Martin,
On 22/02/18 08:51, Martin Liška wrote:
On 02/21/2018 10:23 AM, Kyrill Tkachov wrote:
On arm we also support "native" as a value for -mcpu and -mtune.
Hmm, actually I think any_erroneous_template_args will return false
for error_mark_node. Just moving the error_mark_node check up should
correct that. OK with that change.
On Fri, Mar 9, 2018 at 10:40 AM, Paolo Carlini wrote:
> Hi,
>
> On 09/03/2018 16:17, Jason Merrill wrote:
>>
>> You should
OK.
On Fri, Mar 9, 2018 at 6:05 AM, Jakub Jelinek wrote:
> On Thu, Mar 08, 2018 at 04:06:24PM -0500, Jason Merrill wrote:
>> On Thu, Mar 8, 2018 at 1:01 PM, Jakub Jelinek wrote:
>> > The C FE just warns and doesn't override builtins, but C++ FE
>> > on say int __builtin_trap (); will override th
The following patch fixes
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83712
It is another "cannot find a spill reg for reload" problem. LRA has
already a code splitting hard reg live ranges to avoid such problem.
This code is in LRA inheritance pass. Unfortunately, the code does
splitting
On 03/09/2018 12:31 AM, Cesar Philippidis wrote:
Nvidia Volta GPUs now support warp-level synchronization.
Well, let's try to make that statement a bit more precise.
All Nvidia architectures have supported synchronization of threads in a
warp on a very basic level: by means of convergence (an
On Thu, Mar 8, 2018 at 4:26 PM, Jakub Jelinek wrote:
> On Thu, Mar 08, 2018 at 04:20:40PM -0500, Jason Merrill wrote:
>> On Thu, Mar 8, 2018 at 1:08 PM, Jakub Jelinek wrote:
>> > The reporter complains that -Wformat incorrectly reports std::string*
>> > passed
>> > to "%s" format rather than std
On 03/09/2018 09:16 AM, Vladimir Makarov wrote:
> The following patch fixes
>
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=83712
>
> It is another "cannot find a spill reg for reload" problem. LRA has
> already a code splitting hard reg live ranges to avoid such problem.
> This code is in LRA
On 03/09/2018 08:21 AM, Tom de Vries wrote:
> On 03/09/2018 12:31 AM, Cesar Philippidis wrote:
>> Nvidia Volta GPUs now support warp-level synchronization.
>
> Well, let's try to make that statement a bit more precise.
>
> All Nvidia architectures have supported synchronization of threads in a
>
OK.
On Fri, Mar 9, 2018 at 12:21 PM, Jakub Jelinek wrote:
> On Fri, Mar 09, 2018 at 11:28:47AM -0500, Jason Merrill wrote:
>> On Thu, Mar 8, 2018 at 4:26 PM, Jakub Jelinek wrote:
>> > On Thu, Mar 08, 2018 at 04:20:40PM -0500, Jason Merrill wrote:
>> >> On Thu, Mar 8, 2018 at 1:08 PM, Jakub Jelin
On Fri, Mar 09, 2018 at 02:23:13PM +0100, Richard Biener wrote:
>
> With -Os we inhibit most loop header copying which in turn will disable
> any vectorization attempt either via -fopenmp or via -ftree-vectorize.
>
> The following makes sure the aggressive gate in
> should_duplicate_loop_header_
On Fri, Mar 09, 2018 at 11:28:47AM -0500, Jason Merrill wrote:
> On Thu, Mar 8, 2018 at 4:26 PM, Jakub Jelinek wrote:
> > On Thu, Mar 08, 2018 at 04:20:40PM -0500, Jason Merrill wrote:
> >> On Thu, Mar 8, 2018 at 1:08 PM, Jakub Jelinek wrote:
> >> > The reporter complains that -Wformat incorrectl
This libgo patch adds internal/trace to noinst_DATA. The
internal/trace package is only imported by tests (specifically the
tests in runtime/trace) so it must be in noinst_DATA to ensure that it
is built before running the tests. This was mostly working because
internal/trace has tests itself, an
Hi!
On the following testcase we ICE because when cloning the ctor using
tree-inline.c infrastructure we don't remap the type of INDIRECT_REF,
which needs to be remapped if it a variable length type, otherwise we
refer to the parameters of the original ctor rather than of the base
or complete ctor
On 03/08/2018 11:45 PM, Alexandre Oliva wrote:
> LRA gets very confused when non-addresses are passed as operands to
> asms with address contraints. Even if other constraints are
> available, and the operand is a perfect fit for them, we'd still
> attempt to process the operand as an address, and
On 03/07/2018 03:52 PM, Martin Sebor wrote:
> On 03/07/2018 12:37 PM, Jeff Law wrote:
>> On 02/26/2018 10:32 AM, Martin Sebor wrote:
>>>
>>> PR tree-optimization/84526 - ICE in generic_overlap at
>>> gcc/gimple-ssa-warn-restrict.c:927 since r257860
>>>
>>>
>>> gcc/ChangeLog:
>>>
>>> PR tree-op
On 03/09/2018 11:06 AM, Jakub Jelinek wrote:
> Hi!
>
> On the following testcase we ICE because when cloning the ctor using
> tree-inline.c infrastructure we don't remap the type of INDIRECT_REF,
> which needs to be remapped if it a variable length type, otherwise we
> refer to the parameters of t
On 03/08/2018 05:46 PM, Martin Sebor wrote:
> On 03/08/2018 01:59 AM, Richard Biener wrote:
>> On Thu, Mar 8, 2018 at 12:01 AM, Martin Sebor wrote:
>>> I have become convinced that issuing -Wrestrict in gimple-fold
>>> for calls to memcpy() where the source pointer is the same as
>>> the destinati
On Wed, Mar 07, 2018 at 06:50:41PM -0600, Peter Bergner wrote:
> PR83969 shows another bug in mem_operand_gpr() (which implements the "Y"
> constraint) accepting reg+reg addresses. This was fixed by adding a call
> to rs6000_offsettable_memref_p() to verify the address is a valid offsettable
> add
On 03/08/2018 06:22 AM, Richard Biener wrote:
> On Thu, Mar 8, 2018 at 1:54 PM, Michael Matz wrote:
>> Hi,
>>
>> On Wed, 7 Mar 2018, Peter Bergner wrote:
>>
>>> On 3/7/18 12:01 AM, Jeff Law wrote:
I believe so by nature that the setjmp dominates the longjmp sites and
thus also dominates
On 03/07/2018 05:04 PM, Peter Bergner wrote:
> On 3/7/18 12:01 AM, Jeff Law wrote:
>> I believe so by nature that the setjmp dominates the longjmp sites and
>> thus also dominates the dispatcher. But it's something I want to
>> explicitly check before resubmitting.
>
> Are we sure a setjmp has to
Hi,
On 09/03/2018 17:11, Jason Merrill wrote:
Hmm, actually I think any_erroneous_template_args will return false
for error_mark_node. Just moving the error_mark_node check up should
correct that. OK with that change.
Ah, thanks for spotting that: it wasn't clear to me that TI_ARGS cannot
be
On March 9, 2018 8:42:16 PM GMT+01:00, Jeff Law wrote:
>On 03/08/2018 06:22 AM, Richard Biener wrote:
>> On Thu, Mar 8, 2018 at 1:54 PM, Michael Matz wrote:
>>> Hi,
>>>
>>> On Wed, 7 Mar 2018, Peter Bergner wrote:
>>>
On 3/7/18 12:01 AM, Jeff Law wrote:
> I believe so by nature that the
wg21.link/p0588 changes i from not captured to captured by the lambda
in the testcase, but an implementation is allowed to optimize away
that capture if it doesn't affect the semantics of the program. Since
we didn't capture it in previous versions of GCC, the optimization is
very desirable to avo
On 3/9/18 1:31 PM, Segher Boessenkool wrote:
> On Wed, Mar 07, 2018 at 06:50:41PM -0600, Peter Bergner wrote:
>> This passed bootstrap and regtesting on powerpc64-linux, running the
>> testsuite in both 32-bit and 64-bit modes with no regressions.
>> Ok for trunk?
>
> Sorry this took a while to re
Hi!
As the following testcase shows, we emit bad debug info if a scoped
enum is used before the enumerators are defined.
gen_enumeration_type_die has support for enum forward declarations that
have NULL TYPE_SIZE (i.e. incomplete), but in this case it is complete,
just is ENUM_IS_OPAQUE and the en
Hi!
The following is a powerpcspe variant of the sparc PR39645, and rs6000
has the same code (not sure if ever used or dead after powerpcspe removal).
When gimplifying the builtin, because va_arg_tmp.N is not marked as
addressable from the beginning, we call prepare_gimple_addressable which
assign
Now that argument packs don't have a type, the check for a non-type
argument with a typedefy type wasn't working for them. So let's
recurse.
Tested x86_64-pc-linux-gnu, applying to trunk.
commit 7a163ab3c89382ad5c3bda2f345950760bc499c7
Author: Jason Merrill
Date: Fri Mar 9 21:56:18 2018 -0500
If we are missing some earlier template arguments, we need to be
prepared for template parameters to remain in the result of
substitution. In such a situation we need to make sure
processing_template_decl is set so things don't get confused by those
template parameters.
Tested x86_64-pc-linux-gnu
Another case where we need to set a flag on the ck_identity conversion
so that we don't mistakenly pull out the constant value of something
that we want the address of.
Tested x86_64-pc-linux-gnu, applying to trunk.
commit 493898f505c4b38145356ef3b51dd986312497a8
Author: Jason Merrill
Date: Fri
In fixing PR fortran/83633, it seems the patch I committed
introduced an ICE for nonsensical invalid Fortran. The
attached patch cures the ICE and now (re)issues an error
message.
The basic problem seems to boil down to the recursive
calling of gfc_simplify_expr reduces "huge(1_8)+1_8" to
"cons
On Fri, Mar 09, 2018 at 09:13:10PM -0800, Steve Kargl wrote:
> In fixing PR fortran/83633, it seems the patch I committed
> introduced an ICE for nonsensical invalid Fortran. The
> attached patch cures the ICE and now (re)issues an error
> message.
>
> The basic problem seems to boil down to the
On Mar 9, 2018, Jakub Jelinek wrote:
> Also, I'd say there should be
> symview_upper_bound = 0;
> in dwarf2out_c_finalize for better behavior of GCC JIT.
Thanks. I added code to clear zero_view_p too.
Here's what I'm checking in.
[IEPM] [PR debug/84620] use constant form for DW_AT_GNU_ent
55 matches
Mail list logo
