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
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.
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
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
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'
This patch fixes pr44735 and pr60593. Otherwise I have regstrapped the
three patches without regression.
Thanks,
Dominique
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
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
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
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
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
> 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 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
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
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
> 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/
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/
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
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
> 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
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.
> 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
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
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 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
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
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 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
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 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
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 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
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/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/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
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
> 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
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,
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: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 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: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
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 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
> 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/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 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
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
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.
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.
> > * 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
> > * expr.c (convert_move): If the target has an explicit converter,
> > use it.
> OK.
Thanks! Committed.
> > 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
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 '-
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:
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
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 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/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/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/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/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/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
63 matches
Mail list logo
