Hi,
can_refer_decl_in_current_unit_p is a black magic that carefuly papers around
a mine field created by combination of C++ ABI and ELF visibility attributes.
As such it is not pretty and grown up into quite a maze of rules.
Well, I think it is time to clean it up: for example it does twice very
Hi,
devirt-11.C is somewhat fragile testcase that originally was testing a specific
series of events
in inliner that are no longer happening anyway. It tests for a specific number
of devirutalization that
depends on several aspects. It seems more practical to relax this check,
because it is kin
Hi,
on AIX GCC 4.9 as well as mainline dies on callgraph corruption problem. THis
is caused
by ipa-inline walking node that is removed by ipa-inline-transform (because it
is a local
alias that become unreachable).
This patch solves it by adding a hook to inform user of inline_call when this
hap
> The new test fails on x86_64 Ubuntu 12.04 native build:
Yes, this should be fixed once kcc's patches get into trunk
(https://gcc.gnu.org/ml/gcc-patches/2014-05/msg00756.html). Current
asan_interface.h is indeed C++-only.
-Y
> On Tue, 20 May 2014, Jan Hubicka wrote:
>
> > Hi,
> > as disucssed some time ago, our assumption that every symbol of shared
> > library can
> > be interposed at runtime is expensive and prevents a lot of useful
> > optimizations,
> > including inlining or IPA propagation.
> >
> > While this
>
> Unfortunately, this commit has caused the following ICE for me when
> LTO building 471.omnetpp from SPEC 2006 or Firefox (but not libxul,
> something that gets built earlier):
>
> lto1: internal compiler error: in gimple_get_virt_method_for_vtable,
> at gimple-fold.c:3276
> 0x7437a3 gimple_ge
On Tue, 20 May 2014, Richard Earnshaw wrote:
> >>> If it's not broken in 4.8.2 but broken on the branch head then it's OK
> >>> for the branch.
> >>
> >> I thought I'd double-check with you that it is fine to push this change
> >> to trunk first. OK to apply?
> >
> > Of course it should go to
On 21 May 2014 00:54, Jeff Law wrote:
> On 05/20/14 01:11, Zhenqiang Chen wrote:
>>
>> Hi,
>>
>> The patch fix ICE issue triggered by shrink-wrapping enhancement.
>>
>> Bootstrap and no make check regression on X86-64.
>>
>> OK for trunk?
>>
>> Thanks!
>> -Zhenqiang
>>
>>
>> 2014-05-20 Zhenqiang
On May 14, 2014, at 4:13 AM, Yury Gribov wrote:
> Hi,
>
> Asan and Tsan allow sanitized applications to tweak runtime behavior via API
> defined in headers in libsanitizer/include/sanitizer. This patch adds
> installation code for these headers and a small test.
>
> Bootstrapped and regtested
Bug # 61144 is a regression in 4.9.0 that breaks building of musl libc
due to aggressive and semantically-incorrect constant folding of weak
aliases. The attached patch seems to fix the issue. A weak alias
should never be a candidate for constant folding because it may always
be replaced by a stron
One of the consequences of the (now-fixed) bug in PR60179 is that Nios
II code using target pragmas to specify custom instructions failed to
generate those instructions with -flto. We came up with this test case
for it, but there didn't seem to be an existing hook to scan LTO output.
Is this
> From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-
>
> I'll send the new patch as soon as all the tests are done.
Here you are. I also simplified the tests a bit having the reference as a
function
parameter instead of a local one.
Updated ChangeLogs:
*** gcc/ChangeLog ***
2014-05-20
On Tue, May 20, 2014 at 5:46 PM, Pat Haugen wrote:
> On 05/19/2014 11:27 PM, Maxim Kuvyrkov wrote:
>>
>> Changes to ia64 and rs6000 are mostly mechanical to update hook return
>> values, but port maintainers may wish to review those.
>
> I'm not a maintainer, but the rs6000 changes look good to me
Svante Signell, le Fri 16 May 2014 10:03:05 +0200, a écrit :
> is used in gcc-4.9-4.9.0/src/libgo/go/net/fd_unix.go:
> func dupCloseOnExec(fd int) (newfd int, err error) {
> if atomic.LoadInt32(&tryDupCloexec) == 1 && syscall.F_DUPFD_CLOEXEC!=0 {
> r0, _, e1 := syscall.Syscall(syscall.SYS_FCNTL, ui
Hi Richard,
Apologies for appearing to be pushy on this. I appreciate all of this work
is done in your spare time, I doubt I could manage that. I missed the
important part which was to ask if you had thoughts on the user visible
parts of the patch but you have been through the code now anyway...
> Yes, but why is that better than providing them directly? The latter
> would seem to work better with non-C-family languages like Ada.
That's correct, enumeration types don't have base types in Ada, i.e. they
are their own base types. But if the attributes cannot be expressed in DWARF,
then
On Tue, 20 May 2014, Marek Polacek wrote:
> * is missing tests for long doubles/-mlong-double-128,
Also missing tests for float - as far as I can see, only double is tested.
Ideally all of float, double, long double, __float128 (where supported),
__float80 (where supported) would be tested (th
On 05/19/2014 11:27 PM, Maxim Kuvyrkov wrote:
Changes to ia64 and rs6000 are mostly mechanical to update hook return values,
but port maintainers may wish to review those.
I'm not a maintainer, but the rs6000 changes look good to me.
-Pat
On Tue, 20 May 2014, Jan Hubicka wrote:
> Hi,
> as disucssed some time ago, our assumption that every symbol of shared
> library can
> be interposed at runtime is expensive and prevents a lot of useful
> optimizations,
> including inlining or IPA propagation.
>
> While this is useful feature, i
Jeff Law writes:
> On 05/20/14 02:16, Richard Sandiford wrote:
>> get_attr_enabled was showing up high in a -O0 compile of fold-const.ii.
>> At the moment, extract_insn calls this function for every alternative
>> on each extraction, which can be expensive for instructions like
>> moves that have
My CROSSING_JUMP_P patch didn't handle insns that were copied by
emit_copy_of_insn_after, which previously would copy REG_CROSSING_JUMP
notes by virtue of them being != REG_LABEL_OPERAND.
This patch adds the missing copy. Tested with profiledbootstrap on
x86_64-linux-gnu. Committed as obvious.
On 05/19/14 15:20, John David Anglin wrote:
The problem compiling c-common.c is very hard to debug. These routines
are called intensively and the ICE occurs after several million calls.
It takes a couple of hours of running under gdb to reach the failing
call to reset_insn_used_flags just counti
Hi,
On Fri, May 16, 2014 at 07:25:59PM +0200, Jan Hubicka wrote:
> Hi,
> this patch adds code to remove write only static variables. While analyzing
> effectivity of LTO on firefox, I noticed that surprisingly large part of
> binary's data segment is occupied by these. Fixed thus.
> (this is qui
Minor bugfixes. Committed to head and 4.9 branch.
* config/msp430/msp430.md (split): Don't allow subregs when
splitting SImode adds.
(andneghi): Fix subtraction logic.
* config/msp430/predicates.md (msp430_nonsubreg_or_imm_operand): New.
Index: config/msp430/pred
On 05/13/2014 08:59 PM, Ed Smith-Rowland wrote:
+ escape_it = escape_it || cpp_userdef_string_p (token->type)
+ || cpp_userdef_char_p (token->type);
Let's add the new cases to the previous statement instead of a new one.
OK with that change.
Jason
On May 20, 2014, at 10:58 AM, Segher Boessenkool
wrote:
> The current code converts every tree to signed hwi; this ICEs with
> values not representable as shwi, like 999ULL
Looks nice to me… Looks like the type of change I would have done for
wide-int, if I had tripped on this.
On 20/05/2014 21:36, Jonathan Wakely wrote:
OK. My sketch above avoided calling _M_moved_from() more than once per
object, but the compiler should be able to optimise your version to
avoid multiple calls anyway.
Here is the new patch limited to what I really want to commit this time.
Great.
On Tue, May 20, 2014 at 3:28 PM, Peter Bergner wrote:
> The following patch fixes a semi-latent bug for the HTM pattern used with
> the __builtin_ttest() builtin. This is supposed to expand to a tabortwci.
> instruction which sets cr0 and then some code that copies the cr0 value
> into a gpr and
Thanks for all your help. This is updated patch which:
* uses mix/min handling that Jakub kindly wrote,
* limited a test to ilp32 || lp64 + used sse2_runtime (but it's likely
I screwed it up as usually),
* adds a testcase for (unsigned) __int128,
* adds testcases for C and C++ bit-fields,
* mak
On Tue, May 20, 2014 at 4:32 AM, Richard Biener
wrote:
> On Mon, May 19, 2014 at 5:24 PM, Xinliang David Li wrote:
>> Sorry about it. Here is the patch. There is one remaining case where
>> cgraph_dump_file and dump_enable_p are checked separately --
>> cgraph_dump_file is set up differently fro
Hi,
as disucssed some time ago, our assumption that every symbol of shared library
can
be interposed at runtime is expensive and prevents a lot of useful
optimizations,
including inlining or IPA propagation.
While this is useful feature, it is rather incommon to use it for bigger C++
projects, l
On 05/20/2014 01:51 PM, Mark Wielaard wrote:
The DWARF part isn't what this patch is blocked on. That has already
been discussed on the DWARF standard list, coordinated with the gdb
hackers and approved some months ago.
Fair enough.
The part that hasn't been reviewed
and approved yet is the f
On 05/20/2014 02:11 AM, Jonathan Wakely wrote:
> On 19/05/14 14:57 -0600, Sandra Loosemore wrote:
>> On 05/17/2014 04:07 AM, Jonathan Wakely wrote:
>>> On 17 May 2014 10:50, Jonathan Wakely wrote:
On 17 May 2014 01:16, Sandra Loosemore wrote:
> It appears that this patch from last fall nev
On 19/05/14 22:27 +0200, François Dumont wrote:
On 15/05/2014 22:52, Jonathan Wakely wrote:
Does this get initialized in the constructors?
Would it make sense to give it an initializer?
__bucket_type_M_single_bucket = nullptr;
This bucket is replacing those normally allocated a
The following patch fixes a semi-latent bug for the HTM pattern used with
the __builtin_ttest() builtin. This is supposed to expand to a tabortwci.
instruction which sets cr0 and then some code that copies the cr0 value
into a gpr and then shifts and masks it into the lowest 2 bits in the gpr.
Wi
On 05/20/14 12:13, Richard Biener wrote:
I think it's still important for things like keeping SSA names
densely allocated and for reuse on SSA rewrite of virtual operands
for example. So I'm thinking of an explicit stop-/restart-reuse
interface.
Fair enough. WRT dense allocation, we could proba
On May 20, 2014 6:47:44 PM CEST, Jeff Law wrote:
>On 05/20/14 02:06, Richard Biener wrote:
>>
>> We're still going to have problems if SSA names are re-used
>> (as even released SSA names have to remain valid lattice
>> entries after this). But currently nothing will create new
>> SSA names (fing
On 05/20/14 02:19, Richard Sandiford wrote:
Following on from (and depending on) the last patch, process_constraints
also shows up high in the profile. This patch caches the recog_op_alt
information by insn code too. It also shrinks the size of the structure
from 1 pointer + 5 ints to 1 pointer
The current code converts every tree to signed hwi; this ICEs with
values not representable as shwi, like 999ULL in
pr34154.c (and if it didn't ICE, it would print the wrong value).
This fixes it. Bootstrapped and tested on powerpc64-linux. Okay
to apply?
Segher
2014-05-20 S
On Tue, May 20, 2014 at 8:39 AM, Xinliang David Li wrote:
> On Tue, May 20, 2014 at 6:32 AM, Teresa Johnson wrote:
>> On Mon, May 19, 2014 at 11:51 PM, Xinliang David Li
>> wrote:
>>> Why duplicating the merger functions in dyn-ipa.c? Should those in
>>> libgcov-merge.c be reused?
>>
>> The mer
On Tue, May 20, 2014 at 10:43:22AM -0400, Jason Merrill wrote:
> On 05/20/2014 02:55 AM, Mark Wielaard wrote:
> >On Mon, May 19, 2014 at 04:50:35PM -0400, Jason Merrill wrote:
> >>On 05/13/2014 03:21 AM, Mark Wielaard wrote:
> >>>So the debugger doesn't have to guess the properties of the enum's
>
On 05/20/14 02:16, Richard Sandiford wrote:
get_attr_enabled was showing up high in a -O0 compile of fold-const.ii.
At the moment, extract_insn calls this function for every alternative
on each extraction, which can be expensive for instructions like
moves that have many alternatives.
The attrib
OK.
Jason
Hi James
Thank you for pointing this out. In the new patch I removed the
modification of vqdmulh_n_s32 and vqdmulhq_n_s32.
Passed dejagnu testing on aarch64 qemu again. OK for trunk, 4.9 and 4.8?
2014-05-20 Guozhi Wei
* config/aarch64/arm_neon.h (vqdmulh_n_s16): Change
the la
OK. I've committed my patch.
Jason
On 05/19/14 22:27, Maxim Kuvyrkov wrote:
Hi,
This patch cleans up haifa-sched.c:choose_ready() function while
allow more powerful customization by backends at the same time.
The primary change is that
targetm.sched.first_cycle_multipass_dfa_lookahead_guard hook is
converted from returning a boo
> For now, please revert your original patch
Alex asked me to revert patch for him. Done in r210650.
-Y
On 05/20/14 11:14, Wei Mi wrote:
On Tue, May 20, 2014 at 12:13 AM, Bin.Cheng wrote:
On Tue, May 20, 2014 at 1:30 AM, Jeff Law wrote:
On 05/19/14 00:38, Bin.Cheng wrote:
On Sat, May 17, 2014 at 12:32 AM, Jeff Law wrote:
On 05/16/14 04:07, Bin.Cheng wrote:
But can't you go through movXX
On Tue, May 20, 2014 at 12:13 AM, Bin.Cheng wrote:
> On Tue, May 20, 2014 at 1:30 AM, Jeff Law wrote:
>> On 05/19/14 00:38, Bin.Cheng wrote:
>>>
>>> On Sat, May 17, 2014 at 12:32 AM, Jeff Law wrote:
On 05/16/14 04:07, Bin.Cheng wrote:
But can't you go through movXX
On 05/20/14 01:11, Zhenqiang Chen wrote:
Hi,
The patch fix ICE issue triggered by shrink-wrapping enhancement.
Bootstrap and no make check regression on X86-64.
OK for trunk?
Thanks!
-Zhenqiang
2014-05-20 Zhenqiang Chen
PR rtl-optimization/61220
Part of PR rtl-optimiza
Hi Guys,
I am checking in the patch below (to both the mainline and the 4.9
branch) in order to fix some unexpected failures in the gcc testsuite
with the msp430-elf toolchain. With this patch in place the following
tests now pass:
gcc.c-torture/execute/builtins/memcpy-chk.c
gcc.
On 05/20/14 01:13, Bin.Cheng wrote:
The idea being that common cases where a pair moves can be turned into a
single wider move without having to write target code to make that happen
much of the time. ie 2xQI->HI, 2xHI->SI, 2xSI->DI 2xSF->DF. For things
outside those simple cases, fall back to
This is an updated version of the patch to replace void_zero_node with
void_node. The main controversial point/hack in the original was that
it handled void_node in the gimplifier as a zero constant. Jason sent
me a patch that traps the dummy object in the front end, meaning that
it no longer esc
On 05/20/14 02:06, Richard Biener wrote:
We're still going to have problems if SSA names are re-used
(as even released SSA names have to remain valid lattice
entries after this). But currently nothing will create new
SSA names (fingers crossing) during eliminate. A followup
patch of mine will
On 05/20/14 08:10, Zamyatin, Igor wrote:
Please look then on the following patch.
Regtested successfully on x86_64.
Is it ok for trunk and 4.9?
gcc/cp/ChangeLog:
2014-05-20 Igor Zamyatin
PR c/60189
* parser.c (cp_parser_postfix_expression): Move handling of cilk_sync
from here to...
(cp_pa
> The test in get_call_reg_set_usage for flag_use_caller_save and the hook is
> strictly speaking not necessary. But it's the interface function to retrieve
> the collected register usage information, so it seems a good location to do
> an early-out. I've left it in for now.
But the test for targe
On 05/20/14 09:27, H.J. Lu wrote:
On Tue, May 20, 2014 at 7:31 AM, Zamyatin, Igor wrote:
Hi all!
The following patch fixes the ICE for the cilk code with syntax error.
Regtested on x86_64.
Ok for trunk and 4.9?
Thanks,
Igor
gcc/c/ChangeLog:
2014-05-20 Igor Zamyatin
* c-array-notation.
Hi,
libgo currently does not build because of duplicated symbols. This is because of
confusion in the output machinery, where comdat local is output as a global
symbol. This is because of confusing use of DECL_ONE_ONLY. This bug seems to be
there since introduction of comdat locals in gcc 4.9, but
On 05/20/14 17:05, Alexey Merzlyakov wrote:
Hi all,
This is a fix for thumb1 build fail on trunk appeared since
rev.210515(pr60758).
On thumb1 targets the LR can not be used as argument of POP instruction.
To keep a support of __cxa_end_cleanup backtracing on thumb1
we can make additional regis
Hi,
On 05/20/2014 05:08 PM, Jason Merrill wrote:
On 05/20/2014 10:50 AM, Paolo Carlini wrote:
+ if (TREE_TYPE (init) != type)
+{
+ if (BRACE_ENCLOSED_INITIALIZER_P (init)
+ && CP_AGGREGATE_TYPE_P (type))
+init = reshape_init (type, init, tf_warning_or_error);
+
On 20.05.2014 17:16, Richard Earnshaw wrote:
On 20/05/14 14:12, Ramana Radhakrishnan wrote:
The following PR is opened for this problem:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61223
Thumb1 failure was also detected and reported in pr60758.
I've proposed a thumb1 bugfix there. Regtest for
Hi all,
This is a fix for thumb1 build fail on trunk appeared since
rev.210515(pr60758).
On thumb1 targets the LR can not be used as argument of POP instruction.
To keep a support of __cxa_end_cleanup backtracing on thumb1
we can make additional register operations before push/pop.
But I guess
On Tue, May 20, 2014 at 6:32 AM, Teresa Johnson wrote:
> On Mon, May 19, 2014 at 11:51 PM, Xinliang David Li
> wrote:
>> Why duplicating the merger functions in dyn-ipa.c? Should those in
>> libgcov-merge.c be reused?
>
> The merger functions in libgcov-merge.c use a macro to either read the
> c
PR 61208 is a wrong code bug in Thumb2 where we can generate out of
range branches due to incorrect instruction size calculations. It's
mostly gone latent on 4.9 and trunk (though could still happen at -O0
where splitting is done during final instruction generation).
Complicating things slightly
On Tue, May 20, 2014 at 7:31 AM, Zamyatin, Igor wrote:
> Hi all!
>
> The following patch fixes the ICE for the cilk code with syntax error.
>
> Regtested on x86_64.
> Ok for trunk and 4.9?
>
> Thanks,
> Igor
>
>
> gcc/c/ChangeLog:
>
> 2014-05-20 Igor Zamyatin
>
> * c-array-notation.c (fix_built
On May 20, 2014, at 6:03 AM, Rainer Orth wrote:
> The following patch implements what I've outlined there
> Ok for mainline and 4.9 branch if those pass?
For the test suite bits, Ok.
On Tue, May 20, 2014 at 5:00 AM, Kirill Yukhin wrote:
> Hello,
> On 19 May 09:58, H.J. Lu wrote:
>> On Mon, May 19, 2014 at 9:45 AM, Uros Bizjak wrote:
>> > On Mon, May 19, 2014 at 6:42 PM, H.J. Lu wrote:
>> >
>> Uros,
>> I am looking into libreoffice size and the data alignment seems
On Tue, 20 May 2014, Eric Botcazou wrote:
> > Make the code base easier to understand for newcomers. It's also a
> > documentation improvement (you see what a HOST_WIDE_INT really is),
> > alongside with [u]int64_t being less to type ...
>
> I personally find the abstraction and the separation w
On 05/20/2014 10:50 AM, Paolo Carlini wrote:
+ if (TREE_TYPE (init) != type)
+ {
+ if (BRACE_ENCLOSED_INITIALIZER_P (init)
+ && CP_AGGREGATE_TYPE_P (type))
+ init = reshape_init (type, init, tf_warning_or_error);
+ init = digest_init (type, init, t
On 20/05/14 15:53 +0100, Jonathan Wakely wrote:
The first part of this patch fixes the PR, we should be adding
target-libstdc++-v3 to noconfigdirs, not libstdc++-v3.
The second part disables the target libs that can't be bootstrapped
without libstdc++.
libcilkrts and libsanitizer use lots of st
On Tue, May 20, 2014 at 03:53:14PM +0100, Jonathan Wakely wrote:
> The first part of this patch fixes the PR, we should be adding
> target-libstdc++-v3 to noconfigdirs, not libstdc++-v3.
>
> The second part disables the target libs that can't be bootstrapped
> without libstdc++.
>
> libcilkrts an
The first part of this patch fixes the PR, we should be adding
target-libstdc++-v3 to noconfigdirs, not libstdc++-v3.
The second part disables the target libs that can't be bootstrapped
without libstdc++.
libcilkrts and libsanitizer use lots of std:: stuff.
libitm only uses std::atomic, but in a
Hi,
On 05/19/2014 08:28 PM, Jason Merrill wrote:
How about doing digest_init in get_nsdmi, so that the conversion is
also exposed to walk_field_subobs?
Thus, good news: something as simple as the below passes testing, works
for the 2 bugs and for c++/58704 too. Thus, what else? Personally, I'm
On 05/20/2014 02:55 AM, Mark Wielaard wrote:
On Mon, May 19, 2014 at 04:50:35PM -0400, Jason Merrill wrote:
On 05/13/2014 03:21 AM, Mark Wielaard wrote:
So the debugger doesn't have to guess the properties of the enum's
underlying base type, like size, encoding and signedness.
Well, the enum
On May 20, 2014, at 1:17 AM, Richard Sandiford
wrote:
>> The patch gives a consistent compile-time improvement of about ~3.5%
>> on the -O0 fold-const.ii test case.
3.5 alone is really nice and 3.5 on top of 3.5 is amazing.
On 05/19/2014 05:37 PM, James Greenhalgh wrote:
> On Fri, May 16, 2014 at 06:49:45PM +0100, Vladimir Makarov wrote:
>>The following patch fixes
>>
>> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=60969
>>
>> The patch was bootstrapped and tested on x86/x86-64.
>>
>> Committed as rev. 210519 to g
Hi all!
The following patch fixes the ICE for the cilk code with syntax error.
Regtested on x86_64.
Ok for trunk and 4.9?
Thanks,
Igor
gcc/c/ChangeLog:
2014-05-20 Igor Zamyatin
* c-array-notation.c (fix_builtin_array_notation_fn): Check invalid
function parameters.
diff --git a/gcc/c/
Hi,all
There are some small problems that block the ira to make the best
choice, as PR61241 shows.
This patch fix PR61241. I have done "make bootstrap" and "make check"
on x86, nothing changed after the patch.
Is the patch OK for trunk?
Thanks
Ma Jiang
2014-05-20 Ma Jiang
PR rtl-opti
Please look then on the following patch.
Regtested successfully on x86_64.
Is it ok for trunk and 4.9?
gcc/cp/ChangeLog:
2014-05-20 Igor Zamyatin
PR c/60189
* parser.c (cp_parser_postfix_expression): Move handling of cilk_sync
from here to...
(cp_parser_statement): ...here. Make sure only s
OK.
Jason
OK.
Jason
This makes the dumps easier to follow.
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied.
Richard.
2014-05-20 Richard Biener
* tree-ssa-sccvn.c (process_scc): Dump SCC here, when
iterating,
(extract_and_process_scc_for_name): not here.
(cond_dom_wa
On Mon, May 19, 2014 at 11:51 PM, Xinliang David Li wrote:
> Why duplicating the merger functions in dyn-ipa.c? Should those in
> libgcov-merge.c be reused?
The merger functions in libgcov-merge.c use a macro to either read the
counter to merge from a buffer in memory (when IN_GCOV_TOOL) or from
Hi,
On 05/13/2014 02:19 AM, Paolo Carlini wrote:
Hi,
in this issue Marc noticed that the warning for ignored attribute
visibility is truncated when the previous declaration belongs to a
system header. Luckily the problem can be neatly fixed because it's
just a consequence of not using warnin
On 20/05/14 14:12, Ramana Radhakrishnan wrote:
>
>>
>> The following PR is opened for this problem:
>> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61223
>>
>> Thumb1 failure was also detected and reported in pr60758.
>> I've proposed a thumb1 bugfix there. Regtest for the fix currently is in
>> p
PR 61210 is about a case where a templated wi:: function had "x | y"
and where both "x" and "y" needed temporary variables. The recursive
instantation of x and y were done as two arguments to the same function
call, which meant that the order of instantiation depended on the host
compiler. The ui
The following PR is opened for this problem:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61223
Thumb1 failure was also detected and reported in pr60758.
I've proposed a thumb1 bugfix there. Regtest for the fix currently is in
progress.
Patches must be proposed on gcc-patches and / or in thi
On 20.05.2014 16:25, Richard Earnshaw wrote:
On 16/05/14 14:56, Alexey Merzlyakov wrote:
On 07.05.2014 13:28, Ramana Radhakrishnan wrote:
On 05/07/14 09:19, Yury Gribov wrote:
Original Message
Subject: [PING] [PATCH] Fix for PR libstdc++/60758
Date: Thu, 17 Apr 2014 17:48:12
On 15/05/14 09:47, Kyrill Tkachov wrote:
> Hi all,
>
> Shifted arithmetic operations can never be encoded in 16-bits in and
> therefore
> can not appear in Thumb2 IT blocks under ARMv8-A rules (and the -mrestrict-it
> rules). This patch adjusts the relevant pattern for that purpose.
>
> Tested
This adds $subject.
Committed to branch.
Richard.
2014-05-20 Richard Biener
* genmatch.c (error_cb, fatal_at): New functions.
(expect, parse_expr, parse_op, parse_match_and_simplify,
main): Provide error locations.
Index: gcc/genmatch.c
=
Rainer Orth writes:
> Prompted by the recent failures of c-c++-common/gomp/pr60823-2.c on
> Solaris/x86 with Sun as
>
> http://gcc.gnu.org/ml/gcc-patches/2014-05/msg00943.html
>
> I've reworked the clearing of hardware capabilities via linker maps for
> Sun ld, which is currently replicated
OK.
Jason
On 20/05/14 08:57, Richard Biener wrote:
> On Tue, 20 May 2014, Maciej W. Rozycki wrote:
>
>> Ian,
>>
>> On Sat, 17 May 2014, Richard Biener wrote:
>>
>>> On May 17, 2014 12:22:23 AM CEST, "Maciej W. Rozycki"
>>> wrote:
On Fri, 16 May 2014, Joseph S. Myers wrote:
>> 2014-05-16 Mac
On 16/05/14 14:56, Alexey Merzlyakov wrote:
> On 07.05.2014 13:28, Ramana Radhakrishnan wrote:
>> On 05/07/14 09:19, Yury Gribov wrote:
>>> Original Message
>>> Subject: [PING] [PATCH] Fix for PR libstdc++/60758
>>> Date: Thu, 17 Apr 2014 17:48:12 +0400
>>> From: Alexey Merzlyakov
We can't really code-generate those (and thus such patterns are
simply skipped during code-gen ...). Reject them early.
Committed to the branch.
Richard.
2014-05-20 Richard Biener
* genmatch.c (parse_match_and_simplify): Reject outermost
expressions that are captured or not
Hello,
On 19 May 09:58, H.J. Lu wrote:
> On Mon, May 19, 2014 at 9:45 AM, Uros Bizjak wrote:
> > On Mon, May 19, 2014 at 6:42 PM, H.J. Lu wrote:
> >
> Uros,
> I am looking into libreoffice size and the data alignment seems to make
> huge
> difference. Data section has grown f
On Mon, May 19, 2014 at 5:24 PM, Xinliang David Li wrote:
> Sorry about it. Here is the patch. There is one remaining case where
> cgraph_dump_file and dump_enable_p are checked separately --
> cgraph_dump_file is set up differently from 'dump_file'.
But there you check with an else if, so if yo
On Tue, 20 May 2014, Eric Botcazou wrote:
> > Same as for going C++.
>
> Not to the same extent, this will be worse because done en masse throughout
> the code instead of gradually.
Like the gimple -> gimple * change pending or the various
gimple -> gswitch,glabel,etc. stuff? It's on a similar
> Same as for going C++.
Not to the same extent, this will be worse because done en masse throughout
the code instead of gradually.
> Make the code base easier to understand for newcomers. It's also a
> documentation improvement (you see what a HOST_WIDE_INT really is),
> alongside with [u]int6
On Tue, 20 May 2014, Eric Botcazou wrote:
> > The following is my current idea on progressing on the HOST_WIDE_INT
> > removal
> >
> > 1) https://gcc.gnu.org/ml/gcc-patches/2014-05/msg00381.html (ping)
> >
> > 2) make sure [u]int64_t is available and use that to define HOST_WIDE_INT
> >
> > 3)
1 - 100 of 123 matches
Mail list logo
