Re: [PING 4] [PATCH][AArch64] Add neon_pairwise_add & neon_pairwise_add_q types

Hi,  Please consider this as a personal reminder to review the patch at following link and let me know your comments on the same.  https://gcc.gnu.org/ml/gcc-patches/2017-03/msg00505.html Thanks, Naveen    

Re: [PATCH] [PowerPC/RTEMS] Add 64-bit support using ELFv2 ABI

On 26/07/17 19:56, Segher Boessenkool wrote: On Tue, Jul 25, 2017 at 02:25:21PM +0200, Sebastian Huber wrote: Add 64-bit support for RTEMS using the ELFv2 ABI with 64-bit long double. diff --git a/gcc/config.gcc b/gcc/config.gcc index 2ae0218b5fc..aab7f65c1df 100644 --- a/gcc/config.gcc +++ b/

Re: [PATCH 15/17] Language Server Protocol: add lsp::server abstract base class

On Mon, Jul 24, 2017 at 04:05:12PM -0400, David Malcolm wrote: > This patch adds an lsp::server abstract base class for implementing > servers for the Language Server Protocol: > https://github.com/Microsoft/language-server-protocol > > along with supporting classes mirroring those from the prot

Re: [PING] [PATCH v4 0/12] [i386] Improve 64-bit Microsoft to System V ABI pro/epilogues

On 07/26/2017 02:03 PM, H.J. Lu wrote: This patch caused: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=81563 Yes, I discovered this flaw while working on PR 80969 but I hadn't found an actual testcase where it caused a problem yet. I'm about to submit my patchset for review, so sorry I didn

[COMMITTED/AARCH64] Improve thunderx_vect_cost some more

Looked into tuning the cost model for ThunderX 1 and I noticed I had too high cost of the unaligned load/store. This reduces the cost there and now the loops in linpack are able to vectorize and perform the best. Also tested on SPEC CPU 2006 to make sure we don't regress the vectorizer there. Co

Re: [PATCH] Improve alloca alignment

Jeff Law wrote: > +  if (required_align > MAX_SUPPORTED_STACK_ALIGNMENT) > +    { > +  extra = (required_align - MAX_SUPPORTED_STACK_ALIGNMENT) > + / BITS_PER_UNIT; > +  size = plus_constant (Pmode, size, extra); > +  size = force_operand (size, NULL_RTX); >  > -  if (extra && siz

Re: [PATCH #4, cleanup, committed] Remove PowerPC -mvsx-small-integer

This is the final patch that I committed. [gcc] 2017-07-26 Michael Meissner * config/rs6000/rs6000-cpus.def (ISA_2_7_MASKS_SERVER): Delete -mvsx-small-integer option. (ISA_3_0_MASKS_IEEE): Likewise. (OTHER_VSX_VECTOR_MASKS): Likewise. (POWERPC_MASKS): Li

Re: [PATCH,AIX] Enable libffi for AIX

On Wed, Jul 26, 2017 at 8:17 AM, REIX, Tony wrote: > Description: > * This patch enables libffi on AIX. > > Tests: > * Fedora25/x86_64 + GCC trunk : Configure/Build: SUCCESS >- build made by means of gmake. > > ChangeLog: > * configure.ac, configure: Enable libffi for AIX This is fine with

Re: [PATCH] PR libstdc++/53984 handle exceptions in basic_istream::sentry

On 26/07/17 20:14 +0200, Paolo Carlini wrote: Hi again, On 26/07/2017 16:27, Paolo Carlini wrote: Hi, On 26/07/2017 16:21, Andreas Schwab wrote: ERROR: 27_io/basic_fstream/53984.cc: unknown dg option: dg-require-file-io 18 {} for " dg-require-file-io 18 "" " Should be already fixed, a trivia

gotools: remove *.sent for make mostlyclean

I have no idea why this is necessary, but remove *.sent when running make mostlyclean, for https://gcc.gnu.org/PR81548. Bootstrapped on x86_64-pc-linux-gnu. Committed to mainline. Ian 2017-07-26 Ian Lance Taylor PR go/81548 * Makefile.am (MOSTLYCLEANFILES): Add *.sent. * Makefile.in: Rebuil

[Committed/AARCH64] Fix ThunderX fp vectorizer cost model

Hi, When I went and split the vectorizer cost model to support thunderx2t99, I did not go back and fix up thunderx to be correct after the split. This patch fixes the problem by changing the vector cost for fp to be the same as scalar fp cost which is good enough and brings us back to where we we

Re: [PATCH,AIX] Fully enable XCOFF in libbacktrace on AIX

On Wed, Jul 26, 2017 at 12:54 AM, REIX, Tony wrote: > Description: > * This patch fully enables XCOFF in libbacktrace on AIX. > > Tests: > * Fedora25/x86_64 + GCC v7.1.0 : Configure/Build: SUCCESS >- build made by means of gmake. > > ChangeLog: > * configure.ac, filetype.awk: Separate AIX

Re: [PATCH 0/2] Python testcases to check DWARF output

On Jul 26, 2017, at 9:00 AM, Pierre-Marie de Rodat wrote: > At the last GNU Cauldron, Richard Biener and I talked about DWARF output > testing. Except for guality tests, which are disabled on several > targets, the only way tests check the DWARF is scanning the annotated > assembly (-dA), making i

Re: [PATCH 12/17] Add server.h and server.c

On Jul 26, 2017, at 7:50 AM, David Malcolm wrote: > > On Wed, 2017-07-26 at 23:35 +0900, Oleg Endo wrote: >> On Mon, 2017-07-24 at 16:05 -0400, David Malcolm wrote: >>> >>> + >>> +You should have received a copy of the GNU General Public License >>> +along with GCC; see the file COPYING3. If no

Re: [PATCH #4, cleanup] Remove PowerPC -mvsx-small-integer

On Wed, Jul 26, 2017 at 03:02:15PM -0500, Segher Boessenkool wrote: > Hi Mike, > > On Wed, Jul 26, 2017 at 12:24:17AM -0400, Michael Meissner wrote: > > * config/rs6000/rs6000-cpus.def (ISA_2_7_MASKS_SERVER): Delete > > -mvsx-small-integer option. > > (ISA_3_0_MASKS_IEEE): Likewise. >

Re: [PATCH 1/2] simplify-rtx: The truncation of an IOR can have all bits set (PR81423)

On Jul 25, 2017, at 3:10 PM, Segher Boessenkool wrote: > > On Tue, Jul 25, 2017 at 12:30:13PM +0100, Kyrill Tkachov wrote: >> We sometimes use the __mode__ attribute to force certain sizes in C types. >> For example: typedef int ditype __attribute__ ((mode (DI))); >> Maybe you can do this to for

Re: [PATCH] PR libstdc++/53984 handle exceptions in basic_istream::sentry

On 26 July 2017 at 21:14, Paolo Carlini wrote: > Hi again, > > On 26/07/2017 16:27, Paolo Carlini wrote: >> >> Hi, >> >> On 26/07/2017 16:21, Andreas Schwab wrote: >>> >>> ERROR: 27_io/basic_fstream/53984.cc: unknown dg option: >>> dg-require-file-io 18 {} for " dg-require-file-io 18 "" " >> >> Sh

Re: [PATCH #4, cleanup] Remove PowerPC -mvsx-small-integer

On Wed, Jul 26, 2017 at 12:28:02AM -0400, Michael Meissner wrote: > And not only the patches to the compiler, I forgot to include the testsuite > patches: > > [gcc/testsuite] > 2017-07-25 Michael Meissner > > * gcc.target/powerpc/vsx-himode.c: Delete -mvsx-small-integer > option. >

Re: gotools patch committed: Test runtime, misc/cgo/{test,testcarchive}

On Sat, Jul 22, 2017 at 11:08 AM, Uros Bizjak wrote: >> This patch to the gotools Makefile adds tests to `make check`. We now >> test the runtime package using the newly built go tool, and test that >> cgo works by running the misc/cgo/test and misc/cgo/testcarchive >> tests. Bootstrapped and ra

Re: [PATCH] dwarf2out.c patch for AIX

On 07/22/2017 08:29 PM, David Edelsohn wrote: This patch mirrors the earlier patch to copy debug_section_label into dl_section_ref and append the adjustment when necessary. With this patch, GDB is able to report correct macro information. Bootstrapped on powerpc-ibm-aix7.2.0.0 Debug related p

Re: [PATCH, alpha]: libgo patch: also recognize alpha in signal handler

On Sat, Jul 22, 2017 at 10:59 AM, Uros Bizjak wrote: > Similar patch to the one for PPC [1]. > > [1] https://gcc.gnu.org/ml/gcc-patches/2017-07/msg01334.html > > Tested on alphaev68-pc-linux-gnu. Thanks. Committed to mainline. Ian

Re: [PATCH 1/2] x86,s390: add compiler memory barriers when expanding atomic_thread_fence (PR 80640)

On Wed, 26 Jul 2017, Alexander Monakov wrote: > On Wed, 26 Jul 2017, Jeff Law wrote: > > I'm not sure what you mean by extraneous compiler barriers -- isn't the > > worst case scenario here that the target emits them as well? So there > > would be an extraneous one in that case, but that ought to

Re: [PATCH #4, cleanup] Remove PowerPC -mvsx-small-integer

Hi Mike, On Wed, Jul 26, 2017 at 12:24:17AM -0400, Michael Meissner wrote: > * config/rs6000/rs6000-cpus.def (ISA_2_7_MASKS_SERVER): Delete > -mvsx-small-integer option. > (ISA_3_0_MASKS_IEEE): Likewise. > (POWERPC_MASKS): Likewise. (OTHER_VSX_VECTOR_MASKS): Likewise. > @

Re: [C++ Patch] PR 71570 ("[6/7/8 regression] ICE on invalid variable capture in cxx_incomplete_type_diagnostic...")

OK; this is trivial enough for 6/7/8. On Wed, Jul 26, 2017 at 2:49 PM, Paolo Carlini wrote: > Hi, > > avoiding the error recovery issues noticed in the bug seems just matter of > early returning error_mark_node from add_capture, like we already do a few > lines below in a similar case of ill-form

[PATCH] Fix PR middle-end/81564: ICE in group_case_labels_stmt()

The test case for PR81564 exposes an issue where the case labels for a switch statement point to blocks that have already been removed by an earlier call to cleanup_tree_cfg(). In that case, the code in group_case_labels_stmt() that does: base_bb = label_to_block (CASE_LABEL (base_case)); ...r

Re: [PATCH] enhance -Wrestrict for sprintf %s arguments

On 07/19/2017 10:10 AM, Martin Sebor wrote: > On 07/19/2017 12:42 AM, Jeff Law wrote: >> On 07/02/2017 02:00 PM, Martin Sebor wrote: >>> The attached patch enhances the -Wrestrict warning to detect more >>> than just trivial instances of overlapping copying by sprintf and >>> related functions. >>>

Re: [PATCH][AArch64] Simplify frame layout for stack probing

On 07/25/2017 07:58 AM, Wilco Dijkstra wrote: > This patch makes some changes to the frame layout in order to simplify > stack probing. We want to use the save of LR as a probe in any non-leaf > function. With shrinkwrapping we may only save LR before a call, so it > is useful to define a fixed l

Re: [PATCH] x86: Properly check register CFA offset

On Wed, Jul 26, 2017 at 12:04 PM, Uros Bizjak wrote: > On Wed, Jul 26, 2017 at 6:14 PM, H.J. Lu wrote: >> >> X86 epilogue saves register at CFA offset. Since its location on stack >> is computed as CFA - its CFA_OFFSET, CFA_OFFSET points the end of the >> saved register location on stack. This

Re: [PATCH] Improve alloca alignment

On 07/26/2017 11:39 AM, Wilco Dijkstra wrote: > This patch improves alloca alignment. Currently alloca reserves > too much space as it aligns twice, and generates unnecessary stack > alignment code. For example alloca (16) generates: > > sub sp, sp, #32 ??? > mov x1, sp >

[PATCH v5] aarch64: Add split-stack initial support

This is an update patch based on my previous submission [1]. The changes from previous version are: - Update aarch64_supports_split_stack to return true an let the loader to actually emit an error if it does not provide the required TCB field. The TCB field support is planed for GLIBC

Re: [PING] [PATCH v4 0/12] [i386] Improve 64-bit Microsoft to System V ABI pro/epilogues

On Sun, May 14, 2017 at 3:23 AM, Uros Bizjak wrote: > On Sun, May 14, 2017 at 12:34 AM, Daniel Santos > wrote: >> On 05/13/2017 11:52 AM, Uros Bizjak wrote: >>> >>> On Sat, May 13, 2017 at 1:01 AM, Daniel Santos >>> wrote: Ping? I have posted revisions of the following in patch set:

Re: [PATCH] x86: Properly check register CFA offset

On Wed, Jul 26, 2017 at 6:14 PM, H.J. Lu wrote: > > X86 epilogue saves register at CFA offset. Since its location on stack > is computed as CFA - its CFA_OFFSET, CFA_OFFSET points the end of the > saved register location on stack. This patch updates sp_valid_at and > fp_valid_at to properly chec

[C++ Patch] PR 71570 ("[6/7/8 regression] ICE on invalid variable capture in cxx_incomplete_type_diagnostic...")

Hi, avoiding the error recovery issues noticed in the bug seems just matter of early returning error_mark_node from add_capture, like we already do a few lines below in a similar case of ill-formed capture. Tested x86_64-linux. If we are going to fix this in a simply way, maybe we could also

Re: [PATCH] PR libstdc++/53984 handle exceptions in basic_istream::sentry

Hi again, On 26/07/2017 16:27, Paolo Carlini wrote: Hi, On 26/07/2017 16:21, Andreas Schwab wrote: ERROR: 27_io/basic_fstream/53984.cc: unknown dg option: dg-require-file-io 18 {} for " dg-require-file-io 18 "" " Should be already fixed, a trivial typo. ... but now the new test simply fails

Re: [PATCH 1/2] x86,s390: add compiler memory barriers when expanding atomic_thread_fence (PR 80640)

On Wed, 26 Jul 2017, Jeff Law wrote: > I'm not sure what you mean by extraneous compiler barriers -- isn't the > worst case scenario here that the target emits them as well? So there > would be an extraneous one in that case, but that ought to be a "don't > care". Yes, exactly this. > In the mid

Re: [PATCH,AIX] Changes for linking gotools on AIX.

On Wed, Jul 26, 2017 at 9:58 AM, David Edelsohn wrote: > On Wed, Jul 26, 2017 at 12:41 PM, REIX, Tony wrote: >> Description: >> * This patch adds linker options for gotools for AIX. >> >> Tests: >> * Fedora25/x86_64 + GCC trunk : Configure/Build: SUCCESS >>- build remade by means of gmake.

Re: [rs6000] Avoid rotates of floating-point modes

Hi! On Tue, Jul 25, 2017 at 04:10:19PM +0100, Richard Sandiford wrote: > Segher Boessenkool writes: > --- gcc/config/rs6000/rs6000.c2017-07-13 09:25:13.909213921 +0100 > +++ gcc/config/rs6000/rs6000.c2017-07-25 11:14:27.692739547 +0100 > @@ -10503,17 +10503,28 @@ rs6000_const_vec

Re: [PATCH] [PowerPC/RTEMS] Add 64-bit support using ELFv2 ABI

Hi! On Tue, Jul 25, 2017 at 02:25:21PM +0200, Sebastian Huber wrote: > Add 64-bit support for RTEMS using the ELFv2 ABI with 64-bit long > double. > > diff --git a/gcc/config.gcc b/gcc/config.gcc > index 2ae0218b5fc..aab7f65c1df 100644 > --- a/gcc/config.gcc > +++ b/gcc/config.gcc > @@ -2424,7 +2

Re: [PATCH] Validate that Init value is within range defined by IntegerRange.

On 07/13/2017 02:03 AM, Martin Liška wrote: > Hello. > > I'm sending the patch that does validation of values of Init. > Patch can bootstrap on ppc64le-redhat-linux and survives regression tests. > > Ready to be installed? > Martin > > > 0001-Validate-that-Init-value-is-within-range-defined-by-

Re: [PATCH] Fix when -lssp is added by driver (PR middle-end/81400).

On 07/12/2017 07:38 AM, Martin Liška wrote: > Hi. > > Following patch adds -lspp when one uses -mstack-protector-guard=global. > > Patch can bootstrap on ppc64le-redhat-linux and survives regression tests. > > Ready to be installed? > > Martin > > gcc/ChangeLog: > > 2017-07-12 Martin Liska

Re: [PATCH] Initialize counters in autoFDO to zero, not to uninitialized.

On 07/11/2017 04:35 AM, Martin Liška wrote: > Hello. > > This fixes majority of autoFDO test-cases. > > Patch can boostrap and survives regression tests. > > Ready for trunk? > Thanks, > Martin > > gcc/ChangeLog: > > 2017-07-11 Martin Liska > > * auto-profile.c (afdo_annotate_cfg): Ass

[Patch AArch64 obvious] Unify address costs to generic_addrcost_table

Hi, The special case address cost tables for Cortex-A57 and qdf24xx are no different from the generic address cost table. We should just use the generic address cost table directly. If this changes in future, a core is welcome to add new address cost tables. Comitted as obvious as revision 25058

[Patch AArch64 Obvious] Unify branch costs to generic_branch_cost

Hi, All the cores in AArch64 use the pair {1, 3} for their branch costs. As that is covered by generic_branch_cost, we can just use that directly and save the tiny amount of redundant code. If in future any core wants to modify this, they can always add a special-case branch-cost back. Comitted

[PATCH] Improve alloca alignment

This patch improves alloca alignment. Currently alloca reserves too much space as it aligns twice, and generates unnecessary stack alignment code. For example alloca (16) generates: sub sp, sp, #32 ??? mov x1, sp Similarly alloca (x) generates: add x0, x0,

C++ PATCH to implement P0702R1, list-deduction of vector

At the Toronto meeting we adjusted the semantics of class template argument deduction for a class such as std::vector that has an initializer-list constructor: previously, we would get vector v1 { 42 }; // vector vector v2 { v1 }; // vector> but now v2 is also deduced to vector. Tested x86_64-pc

Re: [PATCH 1/2] x86,s390: add compiler memory barriers when expanding atomic_thread_fence (PR 80640)

On 07/26/2017 11:19 AM, Alexander Monakov wrote: > On Wed, 26 Jul 2017, Jeff Law wrote: >> So I think this is up to the target maintainers. I have no concerns >> with enabling use of expand_asm_memory_barrier to be used outside of >> optabs. So if the s390/x86 maintainers want to go forward, the

Re: Patch ping

On Wed, Jul 26, 2017 at 04:13:30PM +0200, Richard Biener wrote: > > > You don't seem to use 'size' anywhere. > > > > size I thought about but then decided not to do anything with it. > > There are two cases, one is where there is no ADDR_EXPR and it actually > > a memory reference. > > In that

Re: [PATCH] toplev: avoid recursive emergency_dump_function

On Sat, 22 Jul 2017, Segher Boessenkool wrote: > On Thu, Jul 20, 2017 at 05:40:28PM +0300, Alexander Monakov wrote: > > Segher pointed out on IRC that ICE reporting with dumps enabled got worse: > > if emergency_dump_function itself leads to an ICE (e.g. by segfaulting), > > nested ICE reporting wi

Re: [PATCH v2] New C++ warning option '-Waccess-specifiers'

On 7/24/17, Franz Sirl wrote: > Am 2017-07-24 um 00:19 schrieb Volker Reichelt: >> On 23 Jul, Eric Gallager wrote: >>> On 7/23/17, Volker Reichelt wrote: Hi again, here is an updated patch for a new warning about redundant access-specifiers. It takes Dave's various comments in

Re: [PATCH] Fix indirect call optimization done by autoFDO.

On 07/11/2017 04:37 AM, Martin Liška wrote: > Hello. > > Following is a typo fix which nobody has noticed during testing of > e.g. gcc/testsuite/gcc.dg/tree-prof/indir-call-prof.c. > > > Patch can bootstrap and survives regression tests. > > Ready for trunk? > Thanks, > Martin > > > gcc/Chang

Re: [PATCH 1/2] x86,s390: add compiler memory barriers when expanding atomic_thread_fence (PR 80640)

On Wed, 26 Jul 2017, Jeff Law wrote: > So I think this is up to the target maintainers. I have no concerns > with enabling use of expand_asm_memory_barrier to be used outside of > optabs. So if the s390/x86 maintainers want to go forward, the optabs > changes are pre-approved. Please see the alt

Re: [PATCH] Improve extraction of changed file in contrib/mklog

On 07/09/2017 01:03 PM, Yuri Gribov wrote: > Hi, > > Currently mklog will fail to analyze lines like this in patches: > diff -rupN gcc/gcc/testsuite/lib/profopt.exp > gcc-compare-checks/gcc/testsuite/lib/profopt.exp > (it fails with "Error: failed to parse diff for ... and ..."). > > This patch f

Re: [PATCH 00/17] RFC: New source-location representation; Language Server Protocol

On 07/24/2017 01:04 PM, David Malcolm wrote: * The LSP implementation is a just a proof-of-concept, to further motivate capturing the extra data. Turning it into a "proper" LSP server implementation would be a *lot* more work, and I'm unlikely to actually do that (but maybe someone on t

Re: [PATCH 2/2] Introduce Python testcases to check DWARF output

On Wed, 2017-07-26 at 18:00 +0200, Pierre-Marie de Rodat wrote: [...] > diff --git a/gcc/testsuite/python/dwarfutils/__init__.py > b/gcc/testsuite/python/dwarfutils/__init__.py > new file mode 100644 > index 000..246fbbd15be > --- /dev/null > +++ b/gcc/testsuite/python/dwarfutils/__init__.p

Re: [PATCH 1/2] x86,s390: add compiler memory barriers when expanding atomic_thread_fence (PR 80640)

On 05/26/2017 04:58 AM, Alexander Monakov wrote: > On Wed, 17 May 2017, Alexander Monakov wrote: > >> Ping. > > Ping^2? > >> (to be clear, patch 2/2 is my previous followup in this thread, I forgot to >> adjust the subject line; it should have said: >> "[PATCH 2/2] x86: add compiler memory barri

Re: [PATCH,AIX] Changes for linking gotools on AIX.

On Wed, Jul 26, 2017 at 12:41 PM, REIX, Tony wrote: > Description: > * This patch adds linker options for gotools for AIX. > > Tests: > * Fedora25/x86_64 + GCC trunk : Configure/Build: SUCCESS >- build remade by means of gmake. >- some test redone in libgo (gmake check) > * AIX + GCC 7.

Re: ping [PATCH] [MSP430] Fix PR78849: ICE on initialization of global struct containing __int20 array

On 05/19/2017 07:35 AM, Jozef Lawrynowicz wrote: > Original post: https://gcc.gnu.org/ml/gcc-patches/2017-04/msg01030.html > > The attached patch fixes an issue for the msp430 target where the TYPE_SIZE of > the __int20 type was set using the precision (20 bits) instead of the > in-memory > size

Re: [PATCH] PR c++/67054 Allow inheriting constructor with non-default-constructible members

OK, thanks. On Wed, Jul 26, 2017 at 12:45 PM, Leonid Koppel wrote: > This patch addresses PR 67054 (duplicates 62310, 80851). An implicitly-defined > inheriting constructor was wrongly considered deleted when it would > initialize a non-default-constructible member, even when a > brace-or-equal

Re: [PATCH 1/2] Introduce testsuite support to run Python tests

On Wed, 2017-07-26 at 18:35 +0200, Pierre-Marie de Rodat wrote: > On 07/26/2017 06:25 PM, David Malcolm wrote: > > str.format was introduced in Python 2.6, so presumably the minimum > > python 2 version here is at least 2.6+; for Python 3 I believe it > > was > > present in Python 3.0 onwards. > >

[PATCH] PR c++/67054 Allow inheriting constructor with non-default-constructible members

This patch addresses PR 67054 (duplicates 62310, 80851). An implicitly-defined inheriting constructor was wrongly considered deleted when it would initialize a non-default-constructible member, even when a brace-or-equal-initializer was present. The bug only affects deduction of the constructor'

[PATCH,AIX] Changes for linking gotools on AIX.

Description: * This patch adds linker options for gotools for AIX. Tests: * Fedora25/x86_64 + GCC trunk : Configure/Build: SUCCESS - build remade by means of gmake. - some test redone in libgo (gmake check) * AIX + GCC 7.1.0 : - build remade by means of gmake. - some test redone in

Re: [PATCH 1/2] Introduce testsuite support to run Python tests

On 07/26/2017 06:25 PM, David Malcolm wrote: str.format was introduced in Python 2.6, so presumably the minimum python 2 version here is at least 2.6+; for Python 3 I believe it was present in Python 3.0 onwards. Hm… Python 2.6 is fairly old: last binary release was ages ago, last source relea

Re: [PATCH 0/2] Python testcases to check DWARF output

On 07/26/2017 06:15 PM, David Malcolm wrote: (FWIW I'm a big fan of Python, so am happy to see this proposal) Me too. :-) Which version of Python did you test against? As far as I can see you've coded this using the common subset of Python 2 and Python 3; it's worth spelling out what the as

Re: [PATCH 1/2] Introduce testsuite support to run Python tests

On Wed, 2017-07-26 at 18:00 +0200, Pierre-Marie de Rodat wrote: [...snip...] > diff --git a/gcc/testsuite/python/testutils.py > b/gcc/testsuite/python/testutils.py > new file mode 100644 > index 000..503105ad9d0 > --- /dev/null > +++ b/gcc/testsuite/python/testutils.py > @@ -0,0 +1,45 @@ >

Re: [PATCH 0/2] Python testcases to check DWARF output

On Wed, 2017-07-26 at 18:00 +0200, Pierre-Marie de Rodat wrote: > Hello, > > At the last GNU Cauldron, Richard Biener and I talked about DWARF > output > testing. Except for guality tests, which are disabled on several > targets, the only way tests check the DWARF is scanning the annotated > assem

[PATCH] x86: Properly check register CFA offset

X86 epilogue saves register at CFA offset. Since its location on stack is computed as CFA - its CFA_OFFSET, CFA_OFFSET points the end of the saved register location on stack. This patch updates sp_valid_at and fp_valid_at to properly check register CFA offset. Tested on x86-64. OK for trunk if

[PATCH 2/2] Introduce Python testcases to check DWARF output

For now, this supports only platforms that have an objdump available for the corresponding target. There are several things that would be nico to have in the future: * add support for more DWARF dumping tools, such as otool on Darwin; * have a DWARF location expression decoder, to be able to

[PATCH 1/2] Introduce testsuite support to run Python tests

gcc/testsuite/ * lib/gcc-python.exp: New test library. * python/testutils.py: New Python helper. --- gcc/testsuite/lib/gcc-python.exp | 95 +++ gcc/testsuite/python/testutils.py | 45 +++ 2 files changed, 140 insertions(+) crea

[PATCH 0/2] Python testcases to check DWARF output

Hello, At the last GNU Cauldron, Richard Biener and I talked about DWARF output testing. Except for guality tests, which are disabled on several targets, the only way tests check the DWARF is scanning the annotated assembly (-dA), making it hard to write reliable tests. For instance, checking the

RE: [PATCH][GCC][AArch64] optimize float immediate moves (1 /4) - infrastructure.

Hi James, I have updated the patch and have responded to your question blow. Ok for trunk? Thanks, Tamar > > static bool > > @@ -5857,12 +5955,6 @@ aarch64_preferred_reload_class (rtx x, > reg_class_t regclass) > >return NO_REGS; > > } > > > > - /* If it's an integer immediate th

[PATCH,AIX] Enable libffi for AIX

Description: * This patch enables libffi on AIX. Tests: * Fedora25/x86_64 + GCC trunk : Configure/Build: SUCCESS - build made by means of gmake. ChangeLog: * configure.ac, configure: Enable libffi for AIX Cordialement, Tony Reix Bull - ATOS IBM Coop Architect & Technical Leader Office :

Re: [PATCH 12/17] Add server.h and server.c

On Wed, 2017-07-26 at 23:35 +0900, Oleg Endo wrote: > On Mon, 2017-07-24 at 16:05 -0400, David Malcolm wrote: > > > > + > > +You should have received a copy of the GNU General Public License > > +along with GCC; see the file COPYING3. If not see > > +.  */ > > + > >

Re: [PATCH 12/17] Add server.h and server.c

On Mon, 2017-07-24 at 16:05 -0400, David Malcolm wrote: >  > + > +You should have received a copy of the GNU General Public License > +along with GCC; see the file COPYING3.  If not see > +.  */ > + > +#ifndef GCC_SERVER_H > +#define GCC_SERVER_H > + > +/* Wrapper arou

Re: [PATCH v12] add -fpatchable-function-entry=N,M option

On Jul 26 2017, Torsten Duwe wrote: > On Wed, Jul 26, 2017 at 04:16:25PM +0200, Andreas Schwab wrote: >> On Jul 07 2017, Torsten Duwe wrote: >> >> > diff --git a/gcc/testsuite/c-c++-common/patchable_function_entry-decl.c >> > b/gcc/testsuite/c-c++-common/patchable_function_entry-decl.c >> > ne

Re: [PATCH] PR libstdc++/53984 handle exceptions in basic_istream::sentry

Hi, On 26/07/2017 16:21, Andreas Schwab wrote: ERROR: 27_io/basic_fstream/53984.cc: unknown dg option: dg-require-file-io 18 {} for " dg-require-file-io 18 "" " Should be already fixed, a trivial typo. Thanks, Paolo.

Re: [PATCH v12] add -fpatchable-function-entry=N,M option

On Wed, Jul 26, 2017 at 04:16:25PM +0200, Andreas Schwab wrote: > On Jul 07 2017, Torsten Duwe wrote: > > > diff --git a/gcc/testsuite/c-c++-common/patchable_function_entry-decl.c > > b/gcc/testsuite/c-c++-common/patchable_function_entry-decl.c > > new file mode 100644 > > index 000..851

Re: [PATCH] PR libstdc++/53984 handle exceptions in basic_istream::sentry

On Jul 25 2017, Jonathan Wakely wrote: > diff --git a/libstdc++-v3/testsuite/27_io/basic_fstream/53984.cc > b/libstdc++-v3/testsuite/27_io/basic_fstream/53984.cc > new file mode 100644 > index 000..e84072e > --- /dev/null > +++ b/libstdc++-v3/testsuite/27_io/basic_fstream/53984.cc > @@ -0,0

Re: [PATCH v12] add -fpatchable-function-entry=N,M option

On Jul 07 2017, Torsten Duwe wrote: > diff --git a/gcc/testsuite/c-c++-common/patchable_function_entry-decl.c > b/gcc/testsuite/c-c++-common/patchable_function_entry-decl.c > new file mode 100644 > index 000..8514b10e820 > --- /dev/null > +++ b/gcc/testsuite/c-c++-common/patchable_functi

Re: Patch ping

On Wed, 26 Jul 2017, Jakub Jelinek wrote: > On Wed, Jul 26, 2017 at 12:34:10PM +0200, Richard Biener wrote: > > On Tue, 25 Jul 2017, Jakub Jelinek wrote: > > > > > Hi! > > > > > > I'd like to ping 2 patches: > > > > > > - UBSAN -fsanitize=pointer-overflow support > > > - http://gcc.gnu.org/ml

Re: [PATCH v2] [SPARC] Add -mfsmuld option

> I would like to add this to the web site: > > Index: htdocs/gcc-7/changes.html > === > RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-7/changes.html,v > retrieving revision 1.88 > diff -r1.88 changes.html > 1248a1249,1251 > > > Use of

[PATCH] Backport to GCC7

Hi. I'm going to install following 3 revision in order to fix GCC.x branch: 0001-Backport-r249728.patch 0002-Backport-r249833.patch 0003-Backport-r250561.patch Patch can bootstrap on ppc64le-redhat-linux and survives regression tests. Martin >From ce5f2421462ebf6bc02eec9729502671c3483883 Mon Se

Re: Patch ping

On Wed, Jul 26, 2017 at 12:34:10PM +0200, Richard Biener wrote: > On Tue, 25 Jul 2017, Jakub Jelinek wrote: > > > Hi! > > > > I'd like to ping 2 patches: > > > > - UBSAN -fsanitize=pointer-overflow support > > - http://gcc.gnu.org/ml/gcc-patches/2017-06/msg01365.html > > The probablility stuf

[PATCH][AArch64] Remove '*' from movsi/di/ti patterns

Remove the remaining uses of '*' from the movsi/di/ti patterns. Using '*' in alternatives is typically incorrect at it tells the register allocator to ignore those alternatives. So remove these from all the integer move patterns. This removes unnecessary int to float moves, for example gcc.target

Re: [PATCH v2] [SPARC] Add -mfsmuld option

On 26/07/17 14:44, Sebastian Huber wrote: On 26/07/17 14:13, Eric Botcazou wrote: Add the -mfsmuld option to control the generation of the FsMULd instruction. In general, this instruction is available in architecture version V8 and V9 CPUs with FPU. Some CPUs of this category do not support

[PATCH,AIX] Enable Go for AIX

Description: * This patch enables Go on AIX. Tests: * Fedora25/x86_64 + GCC trunk : Configure/Build: SUCCESS - build made by means of gmake. ChangeLog: * configure.ac, configure: Enable Go for AIX * contrib/config-list.mk: Enable Go for AIX Cordialement, Tony Reix Bull - ATOS IBM Coop

Re: [PATCH v2] [SPARC] Add -mfsmuld option

On 26/07/17 14:13, Eric Botcazou wrote: Add the -mfsmuld option to control the generation of the FsMULd instruction. In general, this instruction is available in architecture version V8 and V9 CPUs with FPU. Some CPUs of this category do not support this instruction properly, e.g. AT697E, AT69

[PATCH 2/2, i386]: Introduce ASM_FPRINTF_EXTENSIONS and simplify ASM_OUTPUT_REG_{PUSH|POP}

Hello! Attached patch introduces a couple of asm_fprintf extensions to simplify handling of word-mode operands. 2017-07-26 Uros Bizjak * config/i386/i386.h (ASM_PRINTF_EXTENSIONS): New macro. (ASM_OUTPUT_REG_PUSH): Rewrite with new operand modifiers. (ASM_OUTPUT_REG_POP): Ditto.

[PATCH 1/2, c-family]: Add 'z' to asm_fprintf_char_table

Hello! This patch is the prerequisite for my next patch. It enables %z extension for asm_fprintf that will emit 'q' or 'l' suffixes for instructions with word-sized operands. 2017-07-26 Uros Bizjak * c-format.c (asm_fprintf_char_table): Add 'z' to format_chars. Bootstrapped and regressio

Re: [PATCH v2] [SPARC] Add -mfsmuld option

> Add the -mfsmuld option to control the generation of the FsMULd > instruction. In general, this instruction is available in architecture > version V8 and V9 CPUs with FPU. Some CPUs of this category do not > support this instruction properly, e.g. AT697E, AT697F and UT699. Some > CPUs of this

Re: [COMMITED][AArch64] Fix PR79041

Andreas Schwab wrote: > That fails in ILP32 mode. Well -mabi-ilp32 and -mcmodel=large make no sense at all, that should really give an error... I've committed a patch to trunk and GCC7 to disable this test with ILP32. Wilco

Re: [PATCH GCC][1/2]Feed bound computation to folder in loop split

On Wed, Jul 26, 2017 at 9:48 AM, Richard Biener wrote: > On Tue, Jul 25, 2017 at 7:45 PM, Marc Glisse wrote: >> On Tue, 25 Jul 2017, Richard Biener wrote: >> I think we need Richard to say what the intent is for the valueization function. It is used both to stop looking at defining stmt

Re: [PATCH] Switch vec_init and vec_extract optabs to 2 mode optab to allow extraction of vector from vector or initialization of vector from smaller vectors (PR target/80846)

On Wed, Jul 26, 2017 at 09:09:04AM +0200, Jakub Jelinek wrote: > On Tue, Jul 25, 2017 at 03:52:56PM -0500, Segher Boessenkool wrote: > > On Tue, Jul 25, 2017 at 11:14:32AM +0200, Jakub Jelinek wrote: > > > This patch only adds new vector from vector extract and init patterns to > > > the i386 backe

Re: [PATCH] Fix infinite recursion with div-by-zero (PR middle-end/70992)

On Wed, Jul 26, 2017 at 1:35 PM, Marek Polacek wrote: > On Tue, Jul 25, 2017 at 03:47:31PM +0200, Richard Biener wrote: >> On Tue, Jul 25, 2017 at 3:30 PM, Eric Botcazou wrote: >> >> Eric, any comments? >> > >> > No objection for the build2_stat hunk, I think it's in keeping with the Ada >> > sem

Re: [PATCH] Fix infinite recursion with div-by-zero (PR middle-end/70992)

On Tue, Jul 25, 2017 at 03:47:31PM +0200, Richard Biener wrote: > On Tue, Jul 25, 2017 at 3:30 PM, Eric Botcazou wrote: > >> Eric, any comments? > > > > No objection for the build2_stat hunk, I think it's in keeping with the Ada > > semantics. But the tree_could_trap_p hunk is certainly an abomin

Re: PING^2: Fwd: SSA range class and removal of VR_ANTI_RANGEs

On Tue, Jul 25, 2017 at 4:50 PM, Andrew MacLeod wrote: > On 07/25/2017 03:12 AM, Richard Biener wrote: >> >> On Fri, Jul 21, 2017 at 9:30 PM, Aldy Hernandez wrote: >>> >>> On Mon, Jul 17, 2017 at 6:23 AM, Richard Biener >>> wrote: On Mon, Jul 17, 2017 at 8:51 AM, Aldy Hernandez w

Re: [PATCH GCC][1/2]Feed bound computation to folder in loop split

On Wed, Jul 26, 2017 at 11:57 AM, Marc Glisse wrote: > On Wed, 26 Jul 2017, Richard Sandiford wrote: > >> Marc Glisse writes: >>> >>> On Wed, 26 Jul 2017, Richard Sandiford wrote: Richard Biener writes: > > On Tue, Jul 25, 2017 at 7:45 PM, Marc Glisse > wrote: >> >

Re: [PATCH] Switch vec_init and vec_extract optabs to 2 mode optab to allow extraction of vector from vector or initialization of vector from smaller vectors (PR target/80846)

On Tue, Jul 25, 2017 at 11:14 AM, Jakub Jelinek wrote: > Hi! > > The following patch adjusts the vec_init and vec_extract optabs, so that > they don't have in the expander names just the vector mode, but also another > mode, for vec_extract the mode of the result and for vec_init the mode of > the

Re: [PATCH] Switch vec_init and vec_extract optabs to 2 mode optab to allow extraction of vector from vector or initialization of vector from smaller vectors (PR target/80846)

On Tue, 25 Jul 2017, Jakub Jelinek wrote: > Hi! > > The following patch adjusts the vec_init and vec_extract optabs, so that > they don't have in the expander names just the vector mode, but also another > mode, for vec_extract the mode of the result and for vec_init the mode of > the elts of the

Re: Patch ping

On Tue, 25 Jul 2017, Jakub Jelinek wrote: > Hi! > > I'd like to ping 2 patches: > > - UBSAN -fsanitize=pointer-overflow support > - http://gcc.gnu.org/ml/gcc-patches/2017-06/msg01365.html The probablility stuff might need updating? Can you put the TYPE_PRECISION (sizetype) != POINTER_SIZE ch

[PATCH,AIX] Manage .go_export section for AIX

Description: * This patch manages the .go_export section as an EXCLUDE section on AIX. Tests: * Fedora25/x86_64 + GCC trunk : Configure/Build: SUCCESS - build made by means of gmake. ChangeLog: * go-backend.c (go_write_export_data): Use EXCLUDE section for AIX. Cordialement, Tony Reix Bu

  1   2   >