On 08/27/14 16:30, Yi Yang wrote:
Ping
On Mon, Aug 11, 2014 at 3:10 PM, Yi Yang wrote:
Sorry, it is a typo :(
Patch v2:
--
2014-08-11 Yi Yang
gcc:
* bb-reorder.c (pass_partition_blocks::gate): Replace check.
* c-family/c-common.c (handle_section_attribute): Remove
user_defined
On 08/18/14 06:07, Ilya Enkovich wrote:
On 15 Aug 23:08, Jan Hubicka wrote:
Hi,
I get a segafult in decl_maybe_in_construction_p during function versioning.
We have following steps in clone creation (e.g. as in
create_version_clone_with_body):
1. Create function decl
2. Create clone of c
On 08/21/14 09:49, Olivier Hainque wrote:
Hello,
Experiments with custom install programs exposed that the INSTALL series of
Makefile variables aren't propagated down from the gcc subdir.
This patch fixes this. Checked that it addressed the unexpected behavior we
were observing + bootstrapped &
On 08/21/14 11:53, Aldy Hernandez wrote:
Well, whadayaknow... Tom Tromey pointed me at --regex which we can use
to add patterns for not only the .opt files, but for a bunch of other
files/languages we define in GCC.
The following patch adds support for common.opt, rtl.def, tree.def, and
gimple.d
On 08/07/14 09:33, David Malcolm wrote:
On Wed, 2014-08-06 at 21:29 -0400, Trevor Saunders wrote:
On Wed, Aug 06, 2014 at 01:22:58PM -0400, David Malcolm wrote:
+class GTY(()) rtx_insn_list : public rtx_def
+{
+ /* No extra fields, but adds invariant: (GET_CODE (X) == INSN_LIST).
some nice f
On 08/27/14 23:04, Bin.Cheng wrote:
On Wed, Aug 27, 2014 at 6:34 PM, Richard Earnshaw wrote:
On 27/08/14 11:08, Bin Cheng wrote:
Hi
As reported in bug pr62151, combine pass may wrongly delete necessary
instruction in function distribute_notes thus leaving register
uninitialized. This patch is
On 08/29/14 12:07, David Malcolm wrote:
Yes: I made various mistakes in reorg.c and resource.c where I assumed
that a JUMP_LABEL(insn) was an insn, whereas the existing code is set up
to handle RETURN nodes.
Well, it would seem to me that reorg is being totally braindead in
mixing and matching
On 8/28/14 6:58, Chen Gang wrote:
> On 08/28/2014 06:51 AM, Konstantin Serebryany wrote:
>> But, what makes you think there is a bug here?
>> The comment in sanitizer_common/sanitizer_common.h says:
>> // name should have space for at least max_len+1 bytes.
>>
>
> Oh, really, but for me, I still p
Hi,
The loop-19.c test case has regressed from 4.8 to 4.9 and trunk on
classic FPU Power targets, these failures are now seen:
FAIL: gcc.dg/tree-ssa/loop-19.c scan-tree-dump-times optimized "MEM.(base:
&|symbol: )a," 2
FAIL: gcc.dg/tree-ssa/loop-19.c scan-tree-dump-times optimized "MEM.(base:
Hi,
This change fixes ugly confusing error messages like:
Executing on host: powerpc-linux-gnu-gcc arm_cortex_m25641.c
-fno-diagnostics-show-caret -fdiagnostics-color=never -mthumb -S -o
arm_cortex_m25641.s(timeout = 300)
powerpc-linux-gnu-gcc: error: unrecognized command line option '-
> > gcc/
> > * cppbuiltin.c (define_builtin_macros_for_type_sizes): Round
> > pointer size up to a power of two.
> > * defaults.h (DWARF2_ADDR_SIZE): Round up.
> > (POINTER_SIZE_UNITS): New, rounded up value.
> > * dwarf2asm.c (size_of_encoded_value): Use it.
> > (dw2_outpu
> > * expr.c (convert_move): If the target has an explicit converter,
> > use it.
> OK.
Thanks! Committed.
> > * testsuite/
> > * lib/target-supports.exp (check_effective_target_size32plus):
> > Increase size to avoid false positives on 24-bit address spaces.
> > * gcc.c-torture/compile/limits-stringlit.c: Skip if msp430.
> > * gcc.dg/lto/pr54709_1.c: Fix memcpy prototype.
> > * gcc
On Aug 28, 2014, at 11:03 PM, Honggyu Kim wrote:
> This fixes some typo errors in the gcc internal document.
These errors are on purpose.
> -There are many places in which this document is incomplet and incorrekt.
> +There are many places in which this document is incomplete or incorrect.
2014-08-29 Wei Mi
* tree-cfg.c (struct locus_discrim_map): New field needs_increment.
(next_discriminator_for_locus): Increase discriminator only when
return_next or needs_increment are true.
(assign_discriminator): Add an actual for next_discriminator_for_locus.
Hi,
when removing a file we can instead use the remove() function
specified in ISO C instead of unlink(), as found in POSIX, Windows and
probably other systems as well. Thus giving us a bit better guarantee
about the provided semantics, which header file it's found in etc.
Committed to trunk as o
On 08/14/14 10:37, Jakub Jelinek wrote:
Hi!
Right now when -fsanitize=leak adds -llsan, it adds it late on the command
line, so e.g. -lstdc++ comes after it, which seems to be bad.
The following patch puts it early on the link command line like we do for
-lasan or -ltsan. Bootstrapped/regtested
On 08/14/14 07:16, Ilya Tocar wrote:
I've observed SPEC2006 failure on avx512-vlbwdq branch.
It was caused by hardreg_cprop. In maybe_mode_change it was
assumed, that all values of the same register class and same mode.
are ok. This is not the case for i386/avx512. We need to honor
HARD_REGNO_MO
> On Fri, Aug 29, 2014 at 10:11 AM, Cary Coutant wrote:
>>> To avoid the unused new discriminator value, I added a map
>>> "found_call_this_line" to track whether a call is the first call in a
>>> source line seen when assigning discriminators. For the first call in
>>> a source line, its discrimi
On 08/13/14 20:55, Zhenqiang Chen wrote:
Hi,
Function split_live_ranges_for_shrink_wrap has code
if (!flag_shrink_wrap)
return false;
But flag_shrink_wrap is TRUE by default when optimize > 0 even if the
port does not support shrink-wrap. To make sure shrink-wrap is
enabled, "HAVE_simp
Hi,
per the discussion in the PR, committed the attached patch to trunk
and 4.9, which fixes a regression I introduced when implementing the
zlib compressed module files stuff:
Index: ChangeLog
===
--- ChangeLog (revision 214738)
> On 08/13/14 16:10, DJ Delorie wrote:
> > The purpose of this set of changes is to remove assumptions in GCC
> > about type sizes. Previous to this patch, GCC assumed that all types
> > were powers-of-two in size, and used naive math accordingly.
> >
> > Old:
> > POINTER_SIZE / BITS_PER_UNIT
On 08/13/14 16:10, DJ Delorie wrote:
The following five patches are the latest in my ongoing work to
replace the hard-coded __int128 type with a more flexible __int
system that allows target-specific types that correspond to
partial-int modes. Specifically, this will allow targets to have
pointe
On 08/13/14 16:11, DJ Delorie wrote:
Changes to the testsuite to make tests more portable to targets with
unusual address spaces.
* testsuite/
* lib/target-supports.exp (check_effective_target_size32plus):
Increase size to avoid false positives on 24-bit address spaces.
*
On 08/13/14 16:10, DJ Delorie wrote:
The purpose of this set of changes is to remove assumptions in GCC
about type sizes. Previous to this patch, GCC assumed that all types
were powers-of-two in size, and used naive math accordingly.
Old:
POINTER_SIZE / BITS_PER_UNIT
TYPE_SIZE
On 08/13/14 16:10, DJ Delorie wrote:
This patch is part of the __intN series, but is independent. It
provides an additional optimization opportunity, since the MSP430 does
a lot of conversions between HImode and PSImode.
* expr.c (convert_move): If the target has an explicit converter,
> From: David Malcolm
> Date: Fri, 29 Aug 2014 20:07:04 +0200
> BTW, in another email in the thread you said:
>
> > Thanks for the heads-up. BTW, the ChangeLog entries should say
> > "what" not "why"; that goes into a comment in the source.
>
> OK. Where possible I've added comments in the n
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=62309
It seems with gcc-4.8.3 -fno-automatic prevents initializers from
being applied to automatic variables. The following does not behave as
I would expect it to with gcc-4.8.3 on linux-x86-64:
function f (x)
implicit none
integ
On 08/29/14 13:48, David Malcolm wrote:
In my rtx-classes patches I converted a number of macros in rtl.h into
inline functions, many of them trivial and frequently called. The
attached patch adds "skip" commands for them to gdbinit.in, so that gdb
skips them when single-stepping (assuming that
On 08/25/14 13:21, Marek Polacek wrote:
The following ought to fix two spots where an undefined behavior
can occur when compiling pr28045.c with instrumented compiler.
It does so by changing the type of V to an unsigned HOST_WIDE_INT
and performing the shift on unsigned HOST_WIDE_INT.
Hopefully i
On 08/25/14 16:42, Easwaran Raman wrote:
This patch deletes REG_EQUAL note when a src register is replaced by a
constant in an assignment. This is to prevent spurious equivalences
between the constant and the expression in the REG_EQUAL note. In the
bug reported in PR 62146, an assignment in one
On 08/28/14 08:27, Bernd Schmidt wrote:
On 08/22/2014 10:39 PM, Mike Stump wrote:
On Aug 22, 2014, at 5:18 AM, Bernd Schmidt
wrote:
Here's another attempt.
Ok?
Ok. Thanks a ton for doing the work.
The next question would be what to do with gcc.c-torture/unsorted. As
far as I can tell
In my rtx-classes patches I converted a number of macros in rtl.h into
inline functions, many of them trivial and frequently called. The
attached patch adds "skip" commands for them to gdbinit.in, so that gdb
skips them when single-stepping (assuming that you've set up ~/.gdbinit
with an add-auto-
On 08/28/14 02:37, Ilya Enkovich wrote:
2014-08-28 1:39 GMT+04:00 Jeff Law :
On 08/26/14 15:42, Ilya Enkovich wrote:
diff --git a/gcc/calls.c b/gcc/calls.c
index 4285ec1..85dae6b 100644
--- a/gcc/calls.c
+++ b/gcc/calls.c
@@ -1122,6 +1122,14 @@ initialize_argument_information (int num_actuals
On 08/28/14 07:01, Uros Bizjak wrote:
On Fri, Aug 22, 2014 at 2:21 PM, Ilya Enkovich wrote:
Hi,
On Cauldron 2014 we had a couple of talks about relaxation of ebx usage in
32bit PIC mode. It was decided that the best approach would be to not fix ebx
register, use speudo register for GOT base
On 08/28/14 12:58, Uros Bizjak wrote:
On Fri, Aug 22, 2014 at 2:21 PM, Ilya Enkovich wrote:
On Cauldron 2014 we had a couple of talks about relaxation of ebx usage in
32bit PIC mode. It was decided that the best approach would be to not fix ebx
register, use speudo register for GOT base add
Hi,
This is the last in the current series of new vector built-ins. This
group adds vec_ctf, vec_cts, and vec_ctu for vector double and vector
long long. Additionally, it adds documentation for the built-ins added
in my last patch, since I forgot to add it then... /oops
Bootstrapped and tested
On Fri, 2014-08-29 at 18:15 +0200, Hans-Peter Nilsson wrote:
> > From: David Malcolm
> > Date: Fri, 29 Aug 2014 17:33:54 +0200
>
> > On Fri, 2014-08-29 at 16:48 +0200, Hans-Peter Nilsson wrote:
> > > Sorry, but that didn't help. I still get the exact same error.
> > > (Yep, I double-checked that
On 08/06/2014 03:05 AM, Varvara Rainchik wrote:
> * libgomp.h (gomp_thread): For non TLS case create thread data.
> * team.c (create_non_tls_thread_data): New function.
>
>
> ---
> diff --git a/libgomp/libgomp.h b/libgomp/libgomp.h
> index a1482cc..cf3ec8f 100644
> --- a/libgomp/l
Thanks, that is ellegant. Will paste a new patch in this way soon.
Wei.
On Fri, Aug 29, 2014 at 10:11 AM, Cary Coutant wrote:
>> To avoid the unused new discriminator value, I added a map
>> "found_call_this_line" to track whether a call is the first call in a
>> source line seen when assigning
On 19 August 2014 00:06, Joseph S. Myers wrote:
> On Tue, 12 Aug 2014, Marek Polacek wrote:
>
>> This then is the version with both issues fixed (and new test).
>>
>> Bootstrapped/regtested on x86_64-linux, ok for trunk?
>>
>> 2014-08-12 Marek Polacek
>>
>> gcc/c-family/
>> * c-opts.c (sa
> To avoid the unused new discriminator value, I added a map
> "found_call_this_line" to track whether a call is the first call in a
> source line seen when assigning discriminators. For the first call in
> a source line, its discriminator is 0. For the following calls in the
> same source line, a
We have confusing comments in that seem to imply that a template
(mis-labelled as the copy constructor) allows the compiler to define the
implicit copy constructor:
// Lets the compiler synthesize the copy constructor
// complex (const complex<_Tp>&);
/// Copy constructor.
> From: David Malcolm
> Date: Fri, 29 Aug 2014 17:33:54 +0200
> On Fri, 2014-08-29 at 16:48 +0200, Hans-Peter Nilsson wrote:
> > Sorry, but that didn't help. I still get the exact same error.
> > (Yep, I double-checked that I didn't goof testing...)
Famous last words...
> Fully identical, or j
Hi Richard:
>> >> -fno-builtin is seem not only for the c family front-end, but also
>> >> used in LTO now, so move it to common.opt and change it to `Common`.
>> >
>> > Please leave it in c-family and just add LTO to the set of supported
>> > languages. -fno-builtin isn't meaningful for other fr
This patch fixes a stupid typo in:
2014-08-28 Richard Sandiford
* rtl.h (get_pool_constant, rtx_referenced_p): Replace rtx parameters
with const_rtx parameters.
* varasm.c (get_pool_constant): Likewise.
* rtlanal.c (rtx_referenced_p_1): Delete.
(rtx_referenced_p): Use FOR_E
On Fri, 2014-08-29 at 16:48 +0200, Hans-Peter Nilsson wrote:
> > From: David Malcolm
> > Date: Fri, 29 Aug 2014 13:26:59 +0200
> > On Fri, 2014-08-29 at 06:13 +0200, Hans-Peter Nilsson wrote:
> > > /tmp/hpautotest-gcc1/cris-elf/gccobj/./gcc/xgcc
> > > -B/tmp/hpautotest-gcc1/cris-elf/gccobj/./gcc/
> From: David Malcolm
> Date: Fri, 29 Aug 2014 13:26:59 +0200
> On Fri, 2014-08-29 at 06:13 +0200, Hans-Peter Nilsson wrote:
> > /tmp/hpautotest-gcc1/cris-elf/gccobj/./gcc/xgcc
> > -B/tmp/hpautotest-gcc1/cris-elf/gccobj/./gcc/ -nostdinc
> > -B/tmp/hpautotest-gcc1/cris-elf/gccobj/cris-elf/newlib/
Hi!
The patch is another attempt to enable Cilk_for (see eg
https://www.cilkplus.org/sites/default/files/open_specifications/Intel_Cilk_plus_lang_spec_1.2.htm)
in the GCC compiler.
Bootstrapped and regtested on x86_64.
Is it ok for the trunk?
Thanks,
Igor
Changelogs:
gcc/
2014-08-29 Jakub
Hello,
Patch in the bottom extends support of compress and expand
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_mode_iterator VI48F): New.
(define_insn "avx512f_compress_mask"): Dele
Hello,
Patch in the bottom adds support for reduce,range,fpclass.
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 avx512dq_rangepv8df_mask_round,
avx512dq_rangepv16
> From: David Malcolm
> Date: Fri, 29 Aug 2014 13:40:49 +0200
> > Patch attached, which fixes the above testcase; bootstrap in progress:
> >
> > gcc/
> > * resource.h (mark_target_live_regs): Undo erroneous conversion
> > of second param of r214693, converting it back from rtx_insn * to
Hello,
Patch in the bottom updates few UNSPEC insn patterns w/ new
mode iterator. Additionally names were slightly changed.
Bootstrapped.
AVX-512* tests on top of patch-set all pass
under simulator.
Is it ok for trunk?
gcc/
* config/i386/i386.c
(avx512f_getmantv2df_round): Rename
On Fri, 2014-08-29 at 07:26 -0400, David Malcolm wrote:
> On Fri, 2014-08-29 at 06:13 +0200, Hans-Peter Nilsson wrote:
> > Sorry for the context-less mail but I didn't find a proper
> > obvious gcc-patches-message to reply to. (Also, I can't log
> > into bugzilla because to enter a PR as there app
On Fri, 2014-08-29 at 06:13 +0200, Hans-Peter Nilsson wrote:
> Sorry for the context-less mail but I didn't find a proper
> obvious gcc-patches-message to reply to. (Also, I can't log
> into bugzilla because to enter a PR as there appears to have
> been some SSL changes such that my old firefox an
Hi again! I want to remind that issue is urgent for Android.
2014-08-13 12:13 GMT+04:00 Varvara Rainchik :
> *Ping*
>
> Thanks,
> Varvara
>
> 2014-08-06 14:05 GMT+04:00 Varvara Rainchik :
>> Hi,
>>
>> The issue was firstly observed on NDK gcc since TLS is not supported
>> in Android bionic. I also
Hello Uroš,
On 28 Aug 20:44, Uros Bizjak wrote:
> On Thu, Aug 28, 2014 at 3:48 PM, Kirill Yukhin
> wrote:
> > Is it ok for trunk?
> I'd suggest to put noavx512 at the beginning, so:
>
> "noavx512_fma_fmsub_"
This is not possible currently since subst attribute is
expanded to `*' when mask not a
This patch fixes pr44735 and pr60593. Otherwise I have regstrapped the
three patches without regression.
Thanks,
Dominique
This is a first patch towards fixing PR62291 - it removes the
need of having AVAIL_OUT computed for all blocks during
compute_[partial_]antic_aux (or 'clean'). NAMEs are
available by construction and there is no need to check that
(changing the code to an assert passes bootstrap as expected).
I'
On Thu, Aug 28, 2014 at 10:14 PM, Jason Merrill wrote:
> On 08/28/2014 03:13 PM, Richard Biener wrote:
knowledge of scopes and such, only available in the FE. Is there a
generic way of determining if a DECL is in global scope?
>>
>>
>> Via DECL_CONTEXT and the global scope macro pr
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied.
Richard.
2014-08-28 Richard Biener
* tree-cfg.c (verify_gimple_assign_unary): Do not allow
NON_LVALUE_EXPR in gimple.
Index: gcc/tree-cfg.c
===
--- g
The refactoring had two errors, fixed as below.
Boostrapped and tested on x86_64-unknown-linux-gnu, applied.
Richard.
2014-08-29 Richard Biener
* gimple-fold.c (gimple_fold_builtin_strcpy): Fix error
from previous refactoring.
(gimple_fold_builtin_strncpy): Likewise.
On Thu, 28 Aug 2014, Kito Cheng wrote:
> Hi Richard:
>
> >> -fno-builtin is seem not only for the c family front-end, but also
> >> used in LTO now, so move it to common.opt and change it to `Common`.
> >
> > Please leave it in c-family and just add LTO to the set of supported
> > languages. -fn
63 matches
Mail list logo
