Changes since v9:
* Do not store (declare static) the nop pattern template string.
In the future, it might depend on the particular function
being emitted. Fetch it freshly each time instead.
* On platforms without named sections, simply omit the recording
of the nop locations. Run-time ins
On Tue, Jun 27, 2017 at 6:39 AM, Wilco Dijkstra wrote:
> This patch fixes a failure in gcc.target/aarch64/reload-valid-spoff.c
> triggered by https://gcc.gnu.org/ml/gcc-patches/2017-06/msg01367.html -
> it supersedes https://gcc.gnu.org/ml/gcc-patches/2017-06/msg01907.html
> as this fixes the root
Hi Rainer.
> This patch serie adds support for the SPARC M8 processor to GCC.
> The SPARC M8 processor implements the Oracle SPARC Architecture 2017.
[...]
> Note that full binutils support for M8 was upstreamed in May 19.
> Bootstrapped and tested in sparc64-linux-gnu. N
Hello,
Ping for patch proposed here:
https://gcc.gnu.org/ml/gcc-patches/2017-06/msg00579.html
Thanks much in advance,
With Kind Regards,
Olivier
> On Jun 9, 2017, at 10:42 , Olivier Hainque wrote:
>
> Hello,
>
> This is a follow-up improvement over the change
> introduced from
>
> https://
On 04.07.2017 20:11, Richard Sandiford wrote:
Georg-Johann Lay writes:
Hi,
This patch adds support for devices that can access flash memory
by LD* instructions, hence there is no need to put .rodata in RAM.
The default linker script for the new multilib versions already
supports this feature,
Hi,
this is a patch with fixed issues for previous review. Tested and
bootstrapped on x86_64-unknown-linux-gnu and ppc64le-redhat-linux.
Could you take a look?
-Maxim
gcc/ChangeLog:
2017-07-05 Maxim Ostapenko
* asan.c: Include gimple-fold.h.
(get_last_alloca_addr): New function.
(handl
Hi Jose,
> > This patch serie adds support for the SPARC M8 processor to GCC.
> > The SPARC M8 processor implements the Oracle SPARC Architecture 2017.
> [...]
> > Note that full binutils support for M8 was upstreamed in May 19.
> > Bootstrapped and tested in sparc64-linux-gnu.
> While the initialization value doesn't matter (wi::add will overwrite it)
> better initialize both to false ;) Ah, you mean because we want to
> transform only if get_range_info returned VR_RANGE. Indeed somewhat
> unintuitive (but still the best variant for now).
> so I'm still missing a comm
[3/3] Tests
--
gcc/testsuite/ChangeLog:
2017-07-05 Robin Dapp
* gcc.dg/wrapped-binop-simplify-signed-1.c: New test.
* gcc.dg/wrapped-binop-simplify-signed-2.c: New test.
* gcc.dg/wrapped-binop-simplify-unsigned-1.c: New test.
* gcc.dg/wrapped-binop-simplify-un
Hi all,
I've rebased the previous patch to trunk per Andrew's suggestion.
Original patch description/motivation/questions are in
https://gcc.gnu.org/ml/gcc-patches/2017-06/msg01869.html
-Y
safe-unwind-2.patch
Description: Binary data
#include
#include
struct _Unwind_Context;
typedef int (*_
On Wed, Jul 05, 2017 at 11:24:15AM +0300, Maxim Ostapenko wrote:
> + In general, can't we use new_sp as bot parameter because on some
s/can't we/we can't/
> + /* new_alloca = new_alloca_with_rz + align. */
> + g = gimple_build_assign (make_ssa_name (ptr_type), POINTER_PLUS_EXPR,
> +
On 19/05/17 22:11, Michael Collison wrote:
> Christophe,
>
> I had a type in the two test cases: "addcs" should have been "adcs". I caught
> this previously but submitted the previous patch incorrectly. Updated patch
> attached.
>
> Okay for trunk?
>
Apologies for the delay responding, I've b
Andrew Pinski wrote:
>
> This looks related to PR 80266 in that one was crashing due to the
> store pair instruction like what was reported.
Yes it's the same bug. I've now finally reproduced it, it seems many stack
addresses in Ada are SImode which is incorrect (and ultimately can trigger
the LD
On 05.07.2017 10:17, Georg-Johann Lay wrote:
On 04.07.2017 20:11, Richard Sandiford wrote:
Georg-Johann Lay writes:
Hi,
This patch adds support for devices that can access flash memory
by LD* instructions, hence there is no need to put .rodata in RAM.
The default linker script for the new mu
Ping #3
http://gcc.gnu.org/ml/gcc-patches/2017-06/msg01029.html
As avr maintainers are off-line, would a global maintainer have
a look at this?
Thanks,
Johann
On 27.06.2017 12:01, Georg-Johann Lay wrote:
Ping #2
http://gcc.gnu.org/ml/gcc-patches/2017-06/msg01029.html
On 14.06.2017 14:03,
On 05/07/17 12:34, Jakub Jelinek wrote:
On Wed, Jul 05, 2017 at 11:24:15AM +0300, Maxim Ostapenko wrote:
+ In general, can't we use new_sp as bot parameter because on some
s/can't we/we can't/
+ /* new_alloca = new_alloca_with_rz + align. */
+ g = gimple_build_assign (make_ssa_name (ptr_
On Wed, Jul 05, 2017 at 01:19:27PM +0300, Maxim Ostapenko wrote:
> But won't we need to replace all alloca uses manually in this case? E.g. to
> change str.1_18 value to _27?
>
> str.1_18 = __builtin_alloca_with_align (_16, 256);
> *str.1_18[index_19(D)] ={v} 49;
>
> to
>
> _26 = __builtin
Georg-Johann Lay writes:
> On 05.07.2017 10:17, Georg-Johann Lay wrote:
>> On 04.07.2017 20:11, Richard Sandiford wrote:
>>> Georg-Johann Lay writes:
Hi,
This patch adds support for devices that can access flash memory
by LD* instructions, hence there is no need to put .rodata
On 05 Jul 06:38, Peryt, Sebastian wrote:
> Hi Kirill,
>
> Sorry for this confusion. I meant to write MDs for intrinsics. Those
> intrinsics are all masked ones for ADD[SD,SS], SUB[SD,SS], MUL[SD,SS],
> DIV[SD,SS],
> MIN[SD,SS] and MAX[SD,SS]. What I found is that for mask equal 0 they were
> pr
> I am preparing a second version of the patch serie that I will be
> submitting today.
Ok. I was a bit astonished lately to find that a couple of SPARC
patches to binutils and gdb had only been tested (or even only
supported) Linux/SPARC, not Solaris/SPARC.
For binu
Ok, I've fixed comments (not sure the note about optimization is
well-formatted) and moved replace_call_with_value.
Looks better now?
On 05/07/17 13:28, Jakub Jelinek wrote:
Anyway, this is not a strong requirement, the most important is to
fix the comments, then move the replace_call_with_valu
Advanced SIMD patterns are not predicable, thus they should not have %?
in their output templates. Found when auditing the code for something
else. This has been in my tree for sometime , bootstrapped and
regression tested on armhf for armv7ve+simd as the architectural base.
Applied to trunk
Hi,
Instruction selection must not depend on "optimize" because
LDS / STS range might not cover range of IN / OUT.
This lead to wrong ISR code for avrtiny.
Applied as obvious.
Also added some test coverage for ISRs which we didn't have
at all to date.
Johann
gcc/
PR target/81305
Tests were added. I also updated Changelog and set the max line length to be
equal to 79 characters.
gcc/
* config/i386/subst.md (mask_scalar, round_scalar,
round_saeonly_scalar): New meta-templates.
(mask_scalar_name, mask_scalar_operand3, round_scalar_name,
round
Well, it's all only avr stuff... and I decided that it's obvious :-)
Applied the addendum to PR81072 / trunk r249124 from below.
Sorry for the inconvenience.
Johann
On 05.07.2017 12:30, Richard Sandiford wrote:
Georg-Johann Lay writes:
On 05.07.2017 10:17, Georg-Johann Lay wrote:
On 04.07
This patch introduces a new value for the insn type attribute viscmp.
VIS comparison insn are adapted to use it, and finally the DFA
schedulers are updated accordingly.
gcc/ChangeLog:
* config/sparc/sparc.md ("type"): New insn type viscmp.
("fcmp_vis"): Set insn type to
vi
This patch introduces a new value for the insn type attribute bmask.
bmask instructions, which were previously typed as `array', are
adapted to use it, and finally the several DFA schedulers are updated
accordingly.
gcc/ChangeLog:
* config/sparc/sparc.md: New instruction type `bmask'.
[Changes from the previons version:
- Fixed two typos on the definition of the built-ins for fpcmple32shl
and fpcmpde8shl in 32-bit mode.
- Bootstrapped and regtested in sparc-sun-solaris2.12.
- Rebased to today's master.]
This patch serie adds support for the SPARC M8 processor to GCC.
The SPAR
This patch adds a DFA scheduler modelling the core S5 in the SPARC M8
processors.
gcc/ChangeLog:
* config/sparc/m8.md: New file.
* config/sparc/sparc.md: Include m8.md.
---
gcc/ChangeLog | 5 +
gcc/config/sparc/m8.md| 242
This patch adds suppport for the following VIS instructions, which are
introduced in the Oracle SPARC Architecture 2017 and implemented by the
SPARC M8 cpu:
- Dictionary unpack.
- Partitioned compare with shifted result.
- Unsigned partitioned compare with shifted result.
- Partitioned dual-equal
This patch reworks the M7 DFA scheduler to use instruction subtypes. It
also removes the v3pipe insn attribute from sparc.md, as it is no longer
needed.
gcc/ChangeLog:
* config/sparc/niagara7.md: Rework the DFA scheduler to use insn
subtypes.
* config/sparc/sparc.md: Remo
This patch adds the following support for the SPARC M8 cpu, which
implements the Oracle SPARC Architecture 2017:
- Support for -mcpu=m8 and -mtune=m8.
- Definition of cpu target macros and specs in the backend.
- Tuning of backend parameters for the M8.
- Addition of a new cpu type m8 in the machi
This patch introduces a new insn attribute `subtype', and marks
existing insns appropriately. The resulting instruction hierarchy is
documented in a comment.
gcc/ChangeLog:
* config/sparc/sparc.md ("subtype"): New insn attribute.
("*wrgsr_sp64"): Set insn subtype.
("*rdgs
On Mon, Jul 3, 2017 at 12:58 PM, Nathan Sidwell wrote:
> I found classtype_has_nothrow_assign_or_copy_p confusing, trying to figure
> out when it should return false and when true.
I'm curious why you were looking at it? It's only used by obsolete
trait built-ins.
Jason
On 07/05/2017 10:41 AM, Jason Merrill wrote:
On Mon, Jul 3, 2017 at 12:58 PM, Nathan Sidwell wrote:
I found classtype_has_nothrow_assign_or_copy_p confusing, trying to figure
out when it should return false and when true.
I'm curious why you were looking at it? It's only used by obsolete
tra
On 27/06/17 23:29, Jeff Law wrote:
On 06/06/2017 02:25 AM, Kyrill Tkachov wrote:
Hi all,
I'm trying to improve some of the RTL-level handling of vector lane
operations on aarch64 and that
involves dealing with a lot of vec_merge operations. One simplification
that I noticed missing
from simpli
On 27/06/17 23:28, Jeff Law wrote:
On 06/06/2017 02:35 AM, Kyrill Tkachov wrote:
Hi all,
Another vec_merge simplification that's missing is transforming:
(vec_merge (vec_duplicate x) (vec_concat (y) (z)) (const_int N))
into
(vec_concat x z) if N == 1 (0b01) or
(vec_concat y x) if N == 2 (0b10)
On 26/06/17 21:29 +0200, François Dumont wrote:
Hi
Here is the patch to default implementation of std::list default
and move constructors.
I introduce _List_node_header to take care of the move
implementation and also isolate management of the optional list size
storage. I prefer it t
r216834 did a mass removal of "enum" before "machine_mode". This patch
removes some new uses that have been added since then.
Tested on aarch64-linux-gnu and x86_64-linux-gnu, and also by building
at least one target per config/cpu. Applied as obvious on the basis
that r216834 was OK.
Richard
Applied as obvious after testing on aarch64-linux-gnu and x86_64-linux-gnu.
Richard
2017-07-05 Richard Sandiford
libcc1/
* libcp1plugin.cc (plugin_build_field): Use SET_DECL_MODE.
Index: libcc1/libcp1plugin.cc
===
--- l
On Mon, 2017-07-03 at 23:01 +, Joseph Myers wrote:
> Does the changed location fix bug 7356?
The patch as-written doesn't affect that bug, since the patch only
affects sites that use c_parser_require and cp_parser_require with
certain token types, and the diagnostic in PR 7356 is emitted by th
This patch implements TARGET_FIXED_CONDITION_CODE_REGS on ARM.
We have two main cases to consider: in Thumb1 code there are no
condition code registers, so we simply return false. For other
cases we set the the first pointer to CC_REGNUM and the second to
VFPCC_REGNUM iff generating hard-float c
On 07/05/2017 01:36 AM, Torsten Duwe wrote:
Changes since v9:
* Do not store (declare static) the nop pattern template string.
In the future, it might depend on the particular function
being emitted. Fetch it freshly each time instead.
* On platforms without named sections, simply omit th
On 05/07/17 16:38, Sandra Loosemore wrote:
> On 07/05/2017 01:36 AM, Torsten Duwe wrote:
>> Changes since v9:
>>
>> * Do not store (declare static) the nop pattern template string.
>>In the future, it might depend on the particular function
>>being emitted. Fetch it freshly each time instea
On Wed, 5 Jul 2017, Olivier Hainque wrote:
> Hello,
>
> Ping for patch proposed here:
> https://gcc.gnu.org/ml/gcc-patches/2017-06/msg00579.html
This patch is OK.
--
Joseph S. Myers
jos...@codesourcery.com
On 19/06/17 22:48 +0200, François Dumont wrote:
Hi
Here is the patch to default the default and move constructors on
the std::forward_list. Putting a move constructor on
_Fwd_list_node_base helped limiting the code impact of this patch. It
doesn't have any side effect as iterator types usi
On Wed, 5 Jul 2017, David Malcolm wrote:
> This error message might be better to be worded in terms of the
> syntactic thing that came before, which would yield:
>
> pr7356.c:1:2: error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or ‘__attribute__’
> after declaration
> a//sample
> ^
>
> or somesuch. Doi
On 05 Jul 13:51, Peryt, Sebastian wrote:
> Tests were added. I also updated Changelog and set the max line length to be
> equal to 79 characters.
Thanks!
>
> Is it ok for trunk?
Your changes are OK for trunk. I've committed the patch.
--
Thanks, K
>
> Thanks,
> Sebastian
>
> -Original Mess
Ping...
On 06/14/17 14:33, Bernd Edlinger wrote:
> Ping...
>
> On 06/01/17 18:00, Bernd Edlinger wrote:
>> Ping...
>>
>> On 05/12/17 18:49, Bernd Edlinger wrote:
>>> Ping...
>>>
>>> On 04/29/17 19:21, Bernd Edlinger wrote:
Ping...
On 04/20/17 20:11, Bernd Edlinger wrote:
> Ping
Ping...
The latest version of this patch was here:
https://gcc.gnu.org/ml/gcc-patches/2017-04/msg01567.html
Thanks
Bernd.
On 06/14/17 14:34, Bernd Edlinger wrote:
> Ping...
>
> On 06/01/17 18:01, Bernd Edlinger wrote:
>> Ping...
>>
>> On 05/12/17 18:49, Bernd Edlinger wrote:
>>> Ping...
>>>
>>>
On Wed, Jul 5, 2017 at 11:03 AM, Nathan Sidwell wrote:
> On 07/05/2017 10:41 AM, Jason Merrill wrote:
>>
>> On Mon, Jul 3, 2017 at 12:58 PM, Nathan Sidwell wrote:
>>>
>>> I found classtype_has_nothrow_assign_or_copy_p confusing, trying to
>>> figure
>>> out when it should return false and when tr
The patch below has not been applied to the best of my knowledge.
Just a reminder for whoever cares.
Harald
On 05/04/17 20:19, Harald Anlauf wrote:
> On 05/04/17 18:15, Steve Kargl wrote:
>> On Thu, May 04, 2017 at 05:26:17PM +0200, Harald Anlauf wrote:
>>> While trying to clean up my working co
This patch adds AddressSanitizer annotations to std::vector, so that
ASan can detect out-of-bounds accesses to the unused capacity of a
vector. e.g.
std::vector v(2);
int* p = v.data();
v.pop_back();
return p[1]; // ERROR
This cannot be detected by Debug Mode, but with these annotations ASa
On 05/07/17 20:00 +0100, Jonathan Wakely wrote:
We can probably do similar annotations for std::deque, so that
partially filled pages are annotated. I also have a patch for
shared_ptr so that objects created by make_shared can be marked as
invalid after they're destroyed.
This is the make_share
On Wed, Jul 5, 2017 at 8:00 PM, Jonathan Wakely wrote:
> This patch adds AddressSanitizer annotations to std::vector, so that
> ASan can detect out-of-bounds accesses to the unused capacity of a
> vector. e.g.
>
> std::vector v(2);
> int* p = v.data();
> v.pop_back();
> return p[1]; // ERROR
Currently, the nvptx libgomp plugin indiscriminately sets num_gangs to
32 regardless of the underlying CUDA hardware. Depending on the GPU,
this value can be extremely conservative. The attached patch implements
a more sophisticated approach, which probes the hardware at run time to
calculate the n
Hi Harald,
thanks for the reminder. I can take care of committing the patch for
you. Just give me a day or two ...
Cheers,
Janus
2017-07-05 20:44 GMT+02:00 Harald Anlauf :
> The patch below has not been applied to the best of my knowledge.
>
> Just a reminder for whoever cares.
>
> Harald
>
>
On 05/07/17 20:44 +0100, Yuri Gribov wrote:
On Wed, Jul 5, 2017 at 8:00 PM, Jonathan Wakely wrote:
This patch adds AddressSanitizer annotations to std::vector, so that
ASan can detect out-of-bounds accesses to the unused capacity of a
vector. e.g.
std::vector v(2);
int* p = v.data();
v.pop_
Segher Boessenkool writes:
> On Fri, Jun 30, 2017 at 11:53:48AM -0500, Peter Bergner wrote:
>> >> Not use an installed header, that's not what I'm asking. Share the
>> >> source file, i.e., just copy it over from the glibc source tree (it
>> >> should probably hold the master copy). Fewer typos
On Sun, Apr 30, 2017 at 1:02 PM, Pedro Alves wrote:
> Hi Martin,
>
> Thanks much for doing this. A few comments below, in light of my
> experience doing the equivalent checks in the gdb patch linked below,
> using standard C++11.
>
> On 04/29/2017 09:09 PM, Martin Sebor wrote:
>> Calling memset,
On 4 July 2017 at 10:50, Christophe Lyon wrote:
> On 3 July 2017 at 17:30, Jeff Law wrote:
>> On 07/03/2017 09:00 AM, Christophe Lyon wrote:
>>> Hi,
>>>
>>> This is a follow-up to
>>> https://gcc.gnu.org/ml/gcc-patches/2017-06/msg01791.html
>>>
>>> This patch adds dg-require-stack-check and updat
On 07/05/2017 02:58 PM, Andrew Pinski wrote:
On Sun, Apr 30, 2017 at 1:02 PM, Pedro Alves wrote:
Hi Martin,
Thanks much for doing this. A few comments below, in light of my
experience doing the equivalent checks in the gdb patch linked below,
using standard C++11.
On 04/29/2017 09:09 PM, Mar
On Fri, 23 Jun 2017, Prachi Godbole wrote:
> Index: gcc/config/mips/mips.md
> ===
> --- gcc/config/mips/mips.md (revision 246899)
> +++ gcc/config/mips/mips.md (working copy)
> @@ -6578,6 +6581,20 @@
>[(set_attr "type" "jump"
On 07/04/2017 04:19 AM, Jan Hubicka wrote:
> Hi,
> this is another bug I noticed while looking into Itanium rgression.
> There is no profile attached to recovery blocks in scheduler.
> I made them very unlikely, but I wonder if we can do better? After all
> we probably know the probability of path
On 07/04/2017 02:50 AM, Christophe Lyon wrote:
> On 3 July 2017 at 17:30, Jeff Law wrote:
>> On 07/03/2017 09:00 AM, Christophe Lyon wrote:
>>> Hi,
>>>
>>> This is a follow-up to
>>> https://gcc.gnu.org/ml/gcc-patches/2017-06/msg01791.html
>>>
>>> This patch adds dg-require-stack-check and updates
NetBSD/Arm requires that DEFAULT_STRUCTURE_SIZE_BOUNDARY (see
config/arm/netbsd-elf.h for details). This patch disallows
-mstructure-size-boundary on netbsd if the value is not equal to the
DEFAULT_STRUCTURE_SIZE_BOUNDARY.
Okay for trunk?
2017-07-05 Michael Collison
* config/arm/ar
66 matches
Mail list logo
