Re: [RFA] Add --with-libz-prefix option in config/zlib.m4

Hi! On Thu, 19 Feb 2015 08:54:46 +0100, I wrote: > On Wed, 7 Jan 2015 17:00:59 +0100, Tristan Gingold > wrote: > > > > > On 07 Jan 2015, at 15:45, Joel Brobecker wrote: > > > This patch enhances config/zlib.m4 to introduce an extra option > > > --with-libz-prefix which allows us to provide the

Re: [RFA] Add --with-libz-prefix option in config/zlib.m4

Hi! On Wed, 7 Jan 2015 17:00:59 +0100, Tristan Gingold wrote: > > > On 07 Jan 2015, at 15:45, Joel Brobecker wrote: > > This patch enhances config/zlib.m4 to introduce an extra option > > --with-libz-prefix which allows us to provide the location of > > the zlib library we want to use during th

Re: ping #3: [RFA] Add --with-libz-prefix option in config/zlib.m4

On Wed, Feb 18, 2015 at 01:54:17PM -0800, H.J. Lu wrote: > On Wed, Feb 18, 2015 at 1:40 PM, Mark Wielaard wrote: > > On Wed, 2015-02-18 at 12:53 -0800, H.J. Lu wrote: > >> On Wed, Feb 18, 2015 at 12:32 PM, Mark Wielaard wrote: > >> > That doesn't seem like a smart default. And why is is Linux/x86

[PATCH] Fix for PR c++/60269

Hello Everyone,     Attached, please find a patch that is a fix for PR c++/60269. Tested on x86_64 and have no regression issues. Is this OK for trunk? Thanks, Balaji V. Iyer. +2015-02-18  Balaji V. Iyer  + +   PR c++/60269 +   * parser.c (cp_parser_cilk_simd_vectorlength)

Re: ping #3: [RFA] Add --with-libz-prefix option in config/zlib.m4

> Why do you want to turn off zlib? On Linux/x86, zlib is required > for assembler. At least, you should issue an error when --without-libz > is used in binutils for Linux/x86 target. I am trying to do the exact opposite, which is to provide an option to compile WITH zlib, but using an install a

[PATCH] sem_function::bb_dict_test should take a vec *

From: Trevor Saunders bb_dict_test () ment to operate on the callers vector, not a copy of it. Otherwise it either does nothing or crashes. approved by Honza off list committing to trunk (bootstrapped + regtested x86_64-linux-gnu). Trev gcc/ChangeLog: 2015-02-18 Trevor Saunders *

Re: ping #3: [RFA] Add --with-libz-prefix option in config/zlib.m4

On 18 Feb 2015 14:24, H.J. Lu wrote: > On Wed, Feb 18, 2015 at 2:21 PM, Mike Frysinger wrote: > > i think we already have the reports: multiple people don't think it should > > be > > (1) x86-specific or (2) required. don't get me wrong -- i think having > > support > > like this is great. that

Re: [patch] fix PR65048: check that jump-thread paths are still valid

Jeff Law wrote: > These kinds of situations are normally pruned out in mark_threaded_blocks. I added the FSM code generation before calling mark_threaded_blocks. > > The dumps for the FSM threads are a bit sparse -- they don't show > the entire path. That makes it much harder to see what's goin

Re: ping #3: [RFA] Add --with-libz-prefix option in config/zlib.m4

On Wed, Feb 18, 2015 at 2:21 PM, Mike Frysinger wrote: > On 18 Feb 2015 13:54, H.J. Lu wrote: >> On Wed, Feb 18, 2015 at 1:40 PM, Mark Wielaard wrote: >> > On Wed, 2015-02-18 at 12:53 -0800, H.J. Lu wrote: >> >> On Wed, Feb 18, 2015 at 12:32 PM, Mark Wielaard wrote: >> >> > That doesn't seem like

Re: ping #3: [RFA] Add --with-libz-prefix option in config/zlib.m4

On 18 Feb 2015 13:54, H.J. Lu wrote: > On Wed, Feb 18, 2015 at 1:40 PM, Mark Wielaard wrote: > > On Wed, 2015-02-18 at 12:53 -0800, H.J. Lu wrote: > >> On Wed, Feb 18, 2015 at 12:32 PM, Mark Wielaard wrote: > >> > That doesn't seem like a smart default. And why is is Linux/x86 only? > >> > Shouldn'

Re: [PATCH] rtl-optimization/64935: Sorting of ready list is different with/without DEBUG_INSNs.

On 02/18/15 09:02, Maxim Kuvyrkov wrote: Hi, This patch fixes PR64935, which is triggered when ready list at the start of a basic block is greater than --param=max-sched-ready-insns. Sorting the ready list when it has more than max-sched-ready-insns elements is special in that we want to sor

Re: ping #3: [RFA] Add --with-libz-prefix option in config/zlib.m4

On Wed, Feb 18, 2015 at 1:40 PM, Mark Wielaard wrote: > On Wed, 2015-02-18 at 12:53 -0800, H.J. Lu wrote: >> On Wed, Feb 18, 2015 at 12:32 PM, Mark Wielaard wrote: >> > That doesn't seem like a smart default. And why is is Linux/x86 only? >> > Shouldn't that be something that is done explicitly b

Re: [PATCH] Put cleanups of cleanups after cleanups (PR gcov-profile/64634)

On 02/18/15 11:40, Jakub Jelinek wrote: Hi! Richard's GIMPLE EH rewrite in r151696 regressed following testcase. The problem is that when lowering: [gcov-15.C:14:5] try { [gcov-15.C:18:12] D.2335 = __cxa_allocate_exception (4); [gcov-15.C:18:12] try { [g

Re: ping #3: [RFA] Add --with-libz-prefix option in config/zlib.m4

On Wed, 2015-02-18 at 12:53 -0800, H.J. Lu wrote: > On Wed, Feb 18, 2015 at 12:32 PM, Mark Wielaard wrote: > > That doesn't seem like a smart default. And why is is Linux/x86 only? > > Shouldn't that be something that is done explicitly by a distro > > configuring binutils after making sure it act

Re: [Patch] Add option ftree-stdarg-opt

On 17-02-15 13:26, Richard Biener wrote: On Tue, Feb 17, 2015 at 1:12 PM, Tom de Vries wrote: Hi, this patch adds option ftree-stdarg-opt, which switches pass_stdarg on or off. Pass_stdarg does an optimization on cfun->va_list_gpr/fpr_size, and since it's an optimization, it's useful to be ab

[committed] Add missing cleanup in gfortran.dg/coarray_35a.f90

Hi, After running gfortran tests, I found a global_coarrays.mod file in my test directory. Fixed by this patch which adds a missing cleanup-modules. Tested by running the testcase and checking that the file does not occur anymore in the test directory. Committed as obvious. Thanks, - Tom 2

Re: [PATCH] Fix testsuite race on additional_sources

On 02/18/15 08:05, Maxim Kuvyrkov wrote: Hi, This testsuite patch fixes race on additional_source testsuite variable. When a test has both dg-additional-sources and "dg-do run { target FOO }" directives, it may occur that the FOO test will attempt to use additional_sources, which will result

[committed] Add missing cleanup in gfortran.dg/pr37287-1.f90

Hi, After running gfortran tests, I found a pr37287_2.mod in my test directory. Fixed by this patch which adds a missing cleanup-modules. Tested by running the testcase and checking that the file does not occur anymore in the test directory. Committed as obvious. Thanks, - Tom 2015-02-18 To

Re: ping #3: [RFA] Add --with-libz-prefix option in config/zlib.m4

On Wed, Feb 18, 2015 at 12:32 PM, Mark Wielaard wrote: > On Wed, 2015-02-18 at 11:52 -0800, H.J. Lu wrote: >> On Wed, Feb 18, 2015 at 11:44 AM, Mike Frysinger wrote: >> > On 18 Feb 2015 08:58, H.J. Lu wrote: >> >> On Wed, Feb 18, 2015 at 8:54 AM, Mike Frysinger wrote: >> >> >> Why do you want to

[committed] Add missing cleanup in gfortran.dg/finalize_28.f90

Hi, I found finalize_28.f90.003t.original in a gfortran test directory. This patch adds the missing cleanup-tree-dump. Tested by running the test-case and checking that the file does not occur anymore in the test directory. Committed as obvious. Thanks, - Tom 2015-02-18 Tom de Vries *

Re: ping #3: [RFA] Add --with-libz-prefix option in config/zlib.m4

On Wed, Feb 18, 2015 at 09:32:21PM +0100, Mark Wielaard wrote: > > Can you elaborate? > > That doesn't seem like a smart default. And why is is Linux/x86 only? > Shouldn't that be something that is done explicitly by a distro > configuring binutils after making sure it actually is beneficial > (de

[committed] Add missing cleanup in gfortran.dg/fmt_cache_1.f

Hi, I found a fort.10 file in the test directories, and using the contents tracked it back to fmt_cache_1.f, which creates fort.10 but doesn't remove it. This patch fixes that. Tested by running the test-case and checking that fort.10 doesn't appear anymore in the test directory. Committed

Re: ping #3: [RFA] Add --with-libz-prefix option in config/zlib.m4

On Wed, 2015-02-18 at 11:52 -0800, H.J. Lu wrote: > On Wed, Feb 18, 2015 at 11:44 AM, Mike Frysinger wrote: > > On 18 Feb 2015 08:58, H.J. Lu wrote: > >> On Wed, Feb 18, 2015 at 8:54 AM, Mike Frysinger wrote: > >> >> Why do you want to turn off zlib? On Linux/x86, zlib is required > >> >> for ass

Re: [patch] Fix codecvt

On 18/02/15 19:07 +, Jonathan Wakely wrote: While working on PR64797 I discovered that the codecvt specialization was, erm, completely broken when creating UTF-16 surrogate pairs. This fixes it and adds a test, based on the char32_t one I added to the testsuite yesterday. Tested x86_64-linux

[committed][PR65107] Add missing cleanup in gfortran.dg/read_eof_8.f90

Hi, I ran into a failure of gfortran.dg/eof_4.f90, due to the presence of test.dat. The contents of test.dat pointed to read_eof_8.f90, which indeed does not cleanup the test.dat it uses. This patch fixes that. Tested by running the test and checking that test.dat was not present anymore in

Re: [PATCH 09/36] floatformat.h: Wrap in extern "C".

On Thu, Feb 12, 2015 at 11:49:01AM +, Pedro Alves wrote: > On 02/09/2015 11:49 PM, Pedro Alves wrote: > > On 02/09/2015 11:35 PM, Andrew Pinski wrote: > >> On Mon, Feb 9, 2015 at 3:20 PM, Pedro Alves wrote: > >>> Just like libiberty.h. So that C++ programs, such as GDB when built > >>> as a C

Re: ping #3: [RFA] Add --with-libz-prefix option in config/zlib.m4

On Wed, Feb 18, 2015 at 11:44 AM, Mike Frysinger wrote: > On 18 Feb 2015 08:58, H.J. Lu wrote: >> On Wed, Feb 18, 2015 at 8:54 AM, Mike Frysinger wrote: >> > On 18 Feb 2015 04:56, H.J. Lu wrote: >> >> On Wed, Feb 18, 2015 at 4:08 AM, Joel Brobecker wrote: >> >> > On Wed, Jan 07, 2015 at 06:45:48PM

Re: ping #3: [RFA] Add --with-libz-prefix option in config/zlib.m4

On 18 Feb 2015 08:58, H.J. Lu wrote: > On Wed, Feb 18, 2015 at 8:54 AM, Mike Frysinger wrote: > > On 18 Feb 2015 04:56, H.J. Lu wrote: > >> On Wed, Feb 18, 2015 at 4:08 AM, Joel Brobecker wrote: > >> > On Wed, Jan 07, 2015 at 06:45:48PM +0400, Joel Brobecker wrote: > >> >> This patch enhances confi

Re: [patch] Fix codecvt

On Wed, Feb 18, 2015 at 07:07:05PM +, Jonathan Wakely wrote: > While working on PR64797 I discovered that the codecvt > specialization was, erm, completely broken when creating UTF-16 > surrogate pairs. > > This fixes it and adds a test, based on the char32_t one I added to > the testsuite yes

[patch] Fix codecvt

While working on PR64797 I discovered that the codecvt specialization was, erm, completely broken when creating UTF-16 surrogate pairs. This fixes it and adds a test, based on the char32_t one I added to the testsuite yesterday. Tested x86_64-linux (little-endian) and powerpc64-linux (big-endian)

Re: [PATCH] Use !implicit_section in the recent set_section change (PR ipa/65087)

On 2015.02.18 at 10:17 +0100, Jan Hubicka wrote: > > On 2015.02.17 at 22:00 +0100, Jan Hubicka wrote: > > > > Hi! > > > > > > > > Markus reported an ICE, that is fixed by following patch, which limits > > > > the earlier change to !implicit_section only (which I assume is the user > > > > supplied

Re: [PATCH] PR64959: SFINAE in UDLs

On 13/02/15 22:21, Andrea Azzarone wrote: We can use the same trick used in the other tests. Patch attached. Sorry about that! 2015-02-13 20:45 GMT+01:00 Jakub Jelinek : On Wed, Feb 11, 2015 at 12:26:33AM +0100, Andrea Azzarone wrote: * gcc/testsuite/g++.dg/cpp1y/udlit-char-template-vs-

[PATCH] Put cleanups of cleanups after cleanups (PR gcov-profile/64634)

Hi! Richard's GIMPLE EH rewrite in r151696 regressed following testcase. The problem is that when lowering: [gcov-15.C:14:5] try { [gcov-15.C:18:12] D.2335 = __cxa_allocate_exception (4); [gcov-15.C:18:12] try { [gcov-15.C:18:12] [gcov-15.C:18:12] MEM[(int *)D.2

Re: [PATCH] PR64959: SFINAE in UDLs

On Wed, Feb 18, 2015 at 06:29:34PM +, Alex Velenko wrote: > this patch also fixes issues for arm-none-eabi. > Could someone add this patch? ENOPATCH Jakub

Re: [Haifa Scheduler] Fix latent bug in macro-fusion/instruction grouping

On 02/18/15 01:03, Maxim Kuvyrkov wrote: The way SCHED_GROUP_P instructions have been handled historically is by combination of two artifacts: (1) removing all dependencies for instructions inside SCHED_GROUP sequence but the one to next insn, and (2) maintaining a fast track for SCHED_GROUP ins

[PATCH, 4.8] Backport "Fix debug-insn sensitivity in RA" patch to 4.8

Hello! Richard's patch at [1] is needed to fix a bootstrap failure on alpha-linux-gnu on 4.8 branch. Without the patch, IRA creates different sequences, depending on the presence of -g option. One of the many comparison failures is in expr.c, where -fdebug-compare fails with: --- expr.gkd201

Re: [PATCH 6/n] OpenMP 4.0 offloading infrastructure: option handling

Hi! On Mon, 13 Oct 2014 14:33:11 +0400, Ilya Verbin wrote: > On 13 Oct 12:19, Jakub Jelinek wrote: > > On Sat, Oct 11, 2014 at 06:49:00PM +0400, Ilya Verbin wrote: > > > 2. -foffload-abi=[lp64|ilp32] > > >This option is supposed to tell mkoffload (and offload compiler) which > > > ABI is > >

Re: [PATCH] PR target/65064: Return false for COMMON symbols

On 02/18/2015 05:18 AM, H.J. Lu wrote: > PR target/65064 > * config/ia64/predicates.md (sdata_symbolic_operand): Return false > for common symbols. Ok. r~

Re: ping #3: [RFA] Add --with-libz-prefix option in config/zlib.m4

On 2/18/2015 10:54 AM, Mike Frysinger wrote: > On 18 Feb 2015 04:56, H.J. Lu wrote: >> On Wed, Feb 18, 2015 at 4:08 AM, Joel Brobecker >> wrote: >>> On Wed, Jan 07, 2015 at 06:45:48PM +0400, Joel Brobecker wrote: This patch enhances config/zlib.m4 to introduce an extra option --with-li

Re: ping #3: [RFA] Add --with-libz-prefix option in config/zlib.m4

On Wed, Feb 18, 2015 at 8:54 AM, Mike Frysinger wrote: > On 18 Feb 2015 04:56, H.J. Lu wrote: >> On Wed, Feb 18, 2015 at 4:08 AM, Joel Brobecker >> wrote: >> > On Wed, Jan 07, 2015 at 06:45:48PM +0400, Joel Brobecker wrote: >> >> This patch enhances config/zlib.m4 to introduce an extra option >>

Re: ping #3: [RFA] Add --with-libz-prefix option in config/zlib.m4

On 18 Feb 2015 04:56, H.J. Lu wrote: > On Wed, Feb 18, 2015 at 4:08 AM, Joel Brobecker wrote: > > On Wed, Jan 07, 2015 at 06:45:48PM +0400, Joel Brobecker wrote: > >> This patch enhances config/zlib.m4 to introduce an extra option > >> --with-libz-prefix which allows us to provide the location of

Type comparing TLC

Hi, looking across the ODR violation messages in libreoffice and Chromium I found some false positives and some confused messages. This patch fixes them. In partiuclar - I introduced nasty vtable corruption when breaking out my type merging patches, so we ended up creating separate entries f

Re: [PATCH][ARM] PR target/64600 Fix another ICE with -mtune=xscale: properly sign-extend mask during constant splitting

Ping. Thanks, Kyrill On 10/02/15 09:25, Kyrill Tkachov wrote: Ping. https://gcc.gnu.org/ml/gcc-patches/2015-02/msg00141.html Thanks, Kyrill On 03/02/15 15:18, Kyrill Tkachov wrote: Hi all, The ICE in this PR occurs when -mtune=xscale triggers a particular path through arm_gen_constant durin

[PATCH] rtl-optimization/64935: Sorting of ready list is different with/without DEBUG_INSNs.

Hi, This patch fixes PR64935, which is triggered when ready list at the start of a basic block is greater than --param=max-sched-ready-insns. Sorting the ready list when it has more than max-sched-ready-insns elements is special in that we want to sort normal insns even if there are debug insn

Re: [PATCH][AArch64] Testcase fix for __ATOMIC_CONSUME

On 12/02/15 18:38, Mike Stump wrote: On Feb 11, 2015, at 12:16 PM, Torvald Riegel wrote: On Mon, 2015-02-09 at 09:10 -0800, Mike Stump wrote: On Feb 9, 2015, at 7:11 AM, Alex Velenko wrote: The following patch makes atomic-op-consume.c XFAIL Is this patch ok? Ok. I’d shorten the comment

Re: patch to fix rtl documentation for new floating point comparisons

On Wed, 18 Feb 2015, Kenneth Zadeck wrote: > > I think the norm is dynamic, because that's what was in IEEE 754-1985, > > with static rounding added more recently on some processors, because of > > IEEE 754-2008. (There are other variants - IA64 having multiple dynamic > > rounding mode regist

[PATCH] Fix testsuite race on additional_sources

Hi, This testsuite patch fixes race on additional_source testsuite variable. When a test has both dg-additional-sources and "dg-do run { target FOO }" directives, it may occur that the FOO test will attempt to use additional_sources, which will result in failure to compile FOO test. It often

Re: patch to fix rtl documentation for new floating point comparisons

> On Feb 18, 2015, at 3:23 AM, Joseph Myers wrote: > >> On Tue, 17 Feb 2015, Kenneth Zadeck wrote: >> >> The fp exceptions raise some very tricky issues with respect to gcc and >> optimization. On many machines, noisy does not mean to throw an >> exception, it means that you set a bit and

Re: [PATCH] PR rtl-optimization/32219: optimizer causees wrong code in pic/hidden/weak symbol checking

On 13/02/15 05:11, Richard Henderson wrote: On 02/12/2015 08:14 PM, H.J. Lu wrote: I tried the second patch. Results look good on Linux/x86-64. Thanks. My results concurr. I went ahead and installed the patch as posted. r~ 2015-02-12 H.J. Lu Richard Henderson

Re: [RFC, PATCH] LTO: IPA inline speed up for large apps (Chrome)

722 MB perf.data (~643202 samples) ] marxin@marxinbox:~/Programming/gecko-dev/obj-x86_64-unknown-linux-gnu/toolkit/library> perf report marxin@marxinbox:~/Programming/gecko-dev/obj-x86_64-unknown-linux-gnu/toolkit/library> gcc -v Using built-in specs. COLLECT_GCC=gcc COLLECT_LTO_WRAPPER=

Re: [patch] Warn on undefined loop exit

On Thu, Nov 20, 2014 at 05:27:35PM +0100, Richard Biener wrote: > On Wed, Nov 19, 2014 at 9:19 PM, Andrew Stubbs wrote: > > On 19/11/14 16:39, Marek Polacek wrote: > >> > >> On Wed, Nov 19, 2014 at 04:32:43PM +, Andrew Stubbs wrote: > >>> > >>> +if (warning_at (gimple_locat

Re: [RFC, PATCH] LTO: IPA inline speed up for large apps (Chrome)

On 02/17/2015 10:03 PM, Jan Hubicka wrote: Hi, this patch should chase away the expensive thunks and aliases walks from most of analysis code. I think only real use left is local_p predicate that needs to stay because i386 expect local flag to match between caller and callee when expanding assemb

Re: [PATCH][AArch64] Fix wrong-code bug in right-shift SISD patterns

On 18/02/15 13:46, Maxim Kuvyrkov wrote: On Feb 18, 2015, at 4:42 PM, Kyrill Tkachov wrote: On 18/02/15 12:32, Maxim Kuvyrkov wrote: On Feb 18, 2015, at 2:35 PM, Kyrill Tkachov wrote: Hi all, This patch fixes a wrong-code bug with the *aarch64_lshr_sisd_or_int_3 pattern and its associate

Re: [PATCH][AArch64] Fix wrong-code bug in right-shift SISD patterns

On Feb 18, 2015, at 4:42 PM, Kyrill Tkachov wrote: > > On 18/02/15 12:32, Maxim Kuvyrkov wrote: >> On Feb 18, 2015, at 2:35 PM, Kyrill Tkachov wrote: >> >>> Hi all, >>> >>> This patch fixes a wrong-code bug with the *aarch64_lshr_sisd_or_int_3 >>> pattern and its associated splitters. The pro

Re: [PATCH][AArch64] Fix wrong-code bug in right-shift SISD patterns

On 18/02/15 12:32, Maxim Kuvyrkov wrote: On Feb 18, 2015, at 2:35 PM, Kyrill Tkachov wrote: Hi all, This patch fixes a wrong-code bug with the *aarch64_lshr_sisd_or_int_3 pattern and its associated splitters. The problem is that for the 2nd alternative it will split a right-shift into a SISD

Re: [wwwdocs] Porting to again

On Wed, Feb 18, 2015 at 01:16:55PM +0100, Jakub Jelinek wrote: > On Wed, Feb 18, 2015 at 01:04:30PM +0100, Marek Polacek wrote: > > --- porting_to.html 10 Feb 2015 11:12:20 - 1.3 > > +++ porting_to.html 18 Feb 2015 12:01:50 - > > @@ -24,6 +24,17 @@ > > manner. Additions and suggestion

Re: [PATCH] PR target/65064: Return false for COMMON symbols

On Wed, Feb 18, 2015 at 5:18 AM, H.J. Lu wrote: > On Sun, Feb 15, 2015 at 06:19:21AM -0800, H.J. Lu wrote: >> Hi, >> >> r220674 exposed a bug in ia64_in_small_data_p. After r220674, COMMON >> symbols binds locally for executables. But ia64_in_small_data_p returns >> true for COMMON symbols which

Re: [PATCH] PR target/65064: Return false for COMMON symbols

On Sun, Feb 15, 2015 at 06:19:21AM -0800, H.J. Lu wrote: > Hi, > > r220674 exposed a bug in ia64_in_small_data_p. After r220674, COMMON > symbols binds locally for executables. But ia64_in_small_data_p returns > true for COMMON symbols which are never in small data section. This patch > fixes i

Re: [PATCH] Use automatic dependencies for mkoffload.o

On Wed, 18 Feb 2015, Jakub Jelinek wrote: > On Wed, Feb 18, 2015 at 02:56:07PM +0300, Ilya Verbin wrote: > > On Wed, Feb 18, 2015 at 12:48:21 +0100, Thomas Schwinge wrote: > > > What is the rationale for the insn-modes.h order-only prerequisites for > > > mkoffload.o? Is this simply to get past t

Re: ping #3: [RFA] Add --with-libz-prefix option in config/zlib.m4

On Wed, Feb 18, 2015 at 4:08 AM, Joel Brobecker wrote: > Yay? Nay? > > Thank you. > > On Wed, Jan 07, 2015 at 06:45:48PM +0400, Joel Brobecker wrote: >> Hello, >> >> This patch enhances config/zlib.m4 to introduce an extra option >> --with-libz-prefix which allows us to provide the location of >>

Re: [patch, avr] Fix ICE PR64452 pushing eliminated rtxes

2015-02-18 14:59 GMT+03:00 Georg-Johann Lay : > Am 02/17/2015 um 03:34 PM schrieb Denis Chertykov: > >> 2015-02-17 14:12 GMT+03:00 Georg-Johann Lay : >>> >>> Byte-wise pushing virtual regs like arg pointer migth result in patterns >>> like >>> >>> (set (mem:QI (post_dec:HI (reg:HI 32 SP))) >>>

Re: [PATCH][AArch64] Fix wrong-code bug in right-shift SISD patterns

On Feb 18, 2015, at 3:32 PM, Maxim Kuvyrkov wrote: > First of all, applauses! I realize how difficult it was to reduce this > problem. > > Your patch looks OK to me, but I can't shake off feeling that it will > pessimize cases when d8 is not used afterwards. In particular, your patch > make

Re: nvptx offloading patches [3/n], RFD

On Wed, Feb 18, 2015 at 01:09:53PM +0100, Thomas Schwinge wrote: > On Wed, 18 Feb 2015 12:34:38 +0100, Jakub Jelinek wrote: > > On Wed, Feb 18, 2015 at 10:12:19AM +0100, Thomas Schwinge wrote: > > > Do you literally have »nvptx-newlib symlinked into the gcc tree as > > > newlib«? If yes, then tha

Re: [PATCH][AArch64] Fix wrong-code bug in right-shift SISD patterns

On Feb 18, 2015, at 2:35 PM, Kyrill Tkachov wrote: > Hi all, > > This patch fixes a wrong-code bug with the *aarch64_lshr_sisd_or_int_3 > pattern and its associated splitters. The problem is that for the 2nd > alternative it will split a right-shift into a SISD left-shift by the negated > amount

Re: Merge current set of OpenACC changes from gomp-4_0-branch

On Wed, Feb 04, 2015 at 15:05:45 +, Julian Brown wrote: > This (WIP) patch is based on top of a version of your patch that I > merged to our internal branch: that's still the easiest way for me to > test the PTX backend (with unloading support) at present, and it passes > libgomp testing that w

[PATCH] Use automatic dependencies for mkoffload.o

On Wed, Feb 18, 2015 at 02:56:07PM +0300, Ilya Verbin wrote: > On Wed, Feb 18, 2015 at 12:48:21 +0100, Thomas Schwinge wrote: > > What is the rationale for the insn-modes.h order-only prerequisites for > > mkoffload.o? Is this simply to get past the build issue which, for > > example, Jakub also r

Re: [wwwdocs] Porting to again

On Wed, Feb 18, 2015 at 01:04:30PM +0100, Marek Polacek wrote: > --- porting_to.html 10 Feb 2015 11:12:20 - 1.3 > +++ porting_to.html 18 Feb 2015 12:01:50 - > @@ -24,6 +24,17 @@ > manner. Additions and suggestions for improvement are welcome. > > > +Preprocessor issues > + > +

Re: [libstdc++/65033] Give alignment info to libatomic

On 12/02/15 13:23 -0800, Richard Henderson wrote: When we fixed PR54005, making sure that atomic_is_lock_free returns the same value for all objects of a given type, we probably should have changed the interface so that we would pass size and alignment rather than size and object pointer. Instea

Re: nvptx offloading patches [3/n], RFD

Hi Jakub! (Will respond to your other questions later.) On Wed, 18 Feb 2015 12:34:38 +0100, Jakub Jelinek wrote: > On Wed, Feb 18, 2015 at 10:12:19AM +0100, Thomas Schwinge wrote: > > Do you literally have »nvptx-newlib symlinked into the gcc tree as > > newlib«? If yes, then that should expla

[wwwdocs] Porting to again

This is a revised version. I reworded the paragraph dealing with __STDC_VERSION__, made some clarifications wrt %a, and added some text wrt cpp -P issue. Ok? Index: porting_to.html === RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-5/porting

Re: [PATCH 1/4] Add mkoffload for Intel MIC

On Wed, Feb 18, 2015 at 12:48:21PM +0100, Thomas Schwinge wrote: > > --- /dev/null > > +++ b/gcc/config/i386/t-intelmic > > @@ -0,0 +1,9 @@ > > +mkoffload.o: $(srcdir)/config/i386/intelmic-mkoffload.c | insn-modes.h > > + $(COMPILER) -c $(ALL_COMPILERFLAGS) $(ALL_CPPFLAGS) $(INCLUDES) \ > > +

Re: [patch, avr] Fix ICE PR64452 pushing eliminated rtxes

Am 02/17/2015 um 03:34 PM schrieb Denis Chertykov: 2015-02-17 14:12 GMT+03:00 Georg-Johann Lay : Byte-wise pushing virtual regs like arg pointer migth result in patterns like (set (mem:QI (post_dec:HI (reg:HI 32 SP))) (subreg:QI (plus:HI (reg:HI 28) (const_in

Re: [PATCH 1/4] Add mkoffload for Intel MIC

On Wed, Feb 18, 2015 at 12:48:21 +0100, Thomas Schwinge wrote: > What is the rationale for the insn-modes.h order-only prerequisites for > mkoffload.o? Is this simply to get past the build issue which, for > example, Jakub also reported for the nvptx mkoffload, >

Re: [PATCH 1/4] Add mkoffload for Intel MIC

Hi! On Wed, 22 Oct 2014 22:57:01 +0400, Ilya Verbin wrote: > --- /dev/null > +++ b/gcc/config/i386/intelmic-mkoffload.c > +[...] > +#include "config.h" > +#include > +#include "system.h" > +#include "coretypes.h" > +#include "obstack.h" > +#include "intl.h" > +#include "diagnostic.h" > +#include

[PATCH][AArch64] Fix wrong-code bug in right-shift SISD patterns

Hi all, This patch fixes a wrong-code bug with the *aarch64_lshr_sisd_or_int_3 pattern and its associated splitters. The problem is that for the 2nd alternative it will split a right-shift into a SISD left-shift by the negated amount to be shifted by (the ushl instruction allows such semantics)

Re: nvptx offloading patches [3/n], RFD

On Wed, Feb 18, 2015 at 10:12:19AM +0100, Thomas Schwinge wrote: > Do you literally have »nvptx-newlib symlinked into the gcc tree as > newlib«? If yes, then that should explain the problem: as I wrote in > , > yo

Re: [Patch] Fix android build.

On Wed, Feb 18, 2015 at 01:59:25PM +0300, Ilya Tocar wrote: > Hi, > > On android dlerror returns const char*. Looks like POSIX violation. > Ok for trunk? > > libgomp/ > * target.c (gomp_load_plugin_for_device): Fix type of dlerror > return value. > (DLSYM_OPT): Ditto. Therefo

[Patch] Fix android build.

Hi, On android dlerror returns const char*. Ok for trunk? libgomp/ * target.c (gomp_load_plugin_for_device): Fix type of dlerror return value. (DLSYM_OPT): Ditto. --- libgomp/target.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libgomp/target.c

Re: [RFC, PATCH] LTO: IPA inline speed up for large apps (Chrome)

On 02/17/2015 07:38 PM, Jan Hubicka wrote: Hi, thanks for working on it. There are 3 basically indpeendent changes in the patch - The patch to make checking in lto_streamer_init ENABLE_CHECKING only that I think can be comitted as obvoius. Hello. Following email contains fix for that,

Re: nvptx offloading patches [3/n], RFD

On Wed, Feb 18, 2015 at 10:12:19AM +0100, Thomas Schwinge wrote: > On Tue, 17 Feb 2015 17:40:33 +0100, Jakub Jelinek wrote: > > On Tue, Feb 17, 2015 at 04:21:06PM +, Joseph Myers wrote: > > > On Tue, 17 Feb 2015, Jakub Jelinek wrote: > > > > I have nvptx-newlib symlinked into the gcc tree as n

Re: patch to fix rtl documentation for new floating point comparisons

On Tue, 17 Feb 2015, Kenneth Zadeck wrote: > The fp exceptions raise some very tricky issues with respect to gcc and > optimization. On many machines, noisy does not mean to throw an > exception, it means that you set a bit and then check later. If you try > to model this kind of behavior in

Re: nvptx offloading patches [3/n], RFD

On Tue, Feb 17, 2015 at 11:00:14AM +0100, Richard Biener wrote: > I'm just looking for a way to make this less of a hack (and the LTO IL > less target dependent). Not for GCC 5 for which something like your > patch is probably ok, but for the future. So, given Ilya's and Thomas' testing, is this

[PATCH] Fix PR62217

The following patch extends a heuristic in DOM that avoids propagating copies into IV increments to cover all BIV replacements. This avoids the extra loop copy complete peeling produces and would have also avoided the array bound warning had we not disabled them completely from VRP2. Bootstrappe

Re: hashtable optimization

With patch. On 18/02/2015 10:35, François Dumont wrote: Hello I am still studying hashtable performances and especially how to reduce overhead compared to tr1 implementation. Most of the overhead is coming from the additional modulo operations required with the new data model. Having a c

hashtable optimization

Hello I am still studying hashtable performances and especially how to reduce overhead compared to tr1 implementation. Most of the overhead is coming from the additional modulo operations required with the new data model. Having a closer look at PR 57885 bench I realized that we can quite

Re: [PATCH] Fix IFN_OBJECT_SIZE expansion (PR sanitizer/65081)

On Wed, Feb 18, 2015 at 10:15:03AM +0100, Marek Polacek wrote: > We're lacking the POINTER_DIFF_EXPR, which means that ptr - 1 is in fact > ptr + very_big_number. This can result in bogus run-time error when the > objsz checking is turned on. Jakub suggested to not to issue the error > if (ptr >

Re: [PATCH] Use !implicit_section in the recent set_section change (PR ipa/65087)

> On 2015.02.17 at 22:00 +0100, Jan Hubicka wrote: > > > Hi! > > > > > > Markus reported an ICE, that is fixed by following patch, which limits > > > the earlier change to !implicit_section only (which I assume is the user > > > supplied __attribute__((section (. > > > > > > Bootstrapped/regt

[PATCH] Fix IFN_OBJECT_SIZE expansion (PR sanitizer/65081)

We're lacking the POINTER_DIFF_EXPR, which means that ptr - 1 is in fact ptr + very_big_number. This can result in bogus run-time error when the objsz checking is turned on. Jakub suggested to not to issue the error if (ptr > ptr + offset) is true. So this patch attemps to do that, along with so

Re: nvptx offloading patches [3/n], RFD

Hi! On Tue, 17 Feb 2015 17:40:33 +0100, Jakub Jelinek wrote: > On Tue, Feb 17, 2015 at 04:21:06PM +, Joseph Myers wrote: > > On Tue, 17 Feb 2015, Jakub Jelinek wrote: > > > I have nvptx-newlib symlinked into the gcc tree as newlib, so I expected > > > it > > > would be built in-tree, is that

Re: nvptx offloading patches [3/n], RFD

Hi! On Mon, 16 Feb 2015 22:08:12 +0100, Jakub Jelinek wrote: > On Mon, Feb 09, 2015 at 11:20:00AM +0100, Richard Biener wrote: > > I think (also communicated that on IRC) we should instead try not streaming > > machine-modes at all but generating them at stream-in time via layout_type > > or layo

Re: nvptx-tools and nvptx-newlib (was: The nvptx port [10/11+] Target files)

On Wed, Feb 18, 2015 at 09:50:15AM +0100, Thomas Schwinge wrote: > > What about multilibs, is newlib built for both -m32 and -m64, or just the > > default option? > > So far, we have concentrated only on the 64-bit x86_64 configuration; > 32-bit has several known issues to be resolved. >

Re: nvptx-tools and nvptx-newlib (was: The nvptx port [10/11+] Target files)

Hi! On Wed, 4 Feb 2015 10:43:14 +0100, Jakub Jelinek wrote: > On Mon, Feb 02, 2015 at 04:32:34PM +0100, Thomas Schwinge wrote: > > Hi! > > > > On Tue, 23 Dec 2014 19:49:35 +0100, I wrote: > > > On Mon, 10 Nov 2014 17:19:57 +0100, Bernd Schmidt > > > wrote: > > > > The scripts (11/11) I've put

Re: patch to fix rtl documentation for new floating point comparisons

Joseph Myers writes: > For example, on MIPS the C.cond.fmt instruction has a four-bit condition > field: "In the cond field of the instruction: cond 2..1 specify the nature > of the comparison (equals, less than, and so on); cond 0 specifies whether > the comparison is ordered or unordered, i.

nvptx mkoffload: For non-installed testing, look in all COMPILER_PATHs for GCC_INSTALL_NAME (was: nvptx offloading patches [4/n])

Hi! On Sat, 1 Nov 2014 13:11:29 +0100, Bernd Schmidt wrote: > [nvptx mkoffload] To support the --enable-offload-targets=nvptx-none=[install directory] configuration option, I committed the following to trunk in r220782 (and filed ): commit a7243b5200794d53b01d59fa6

Re: [PATCH][C++] PR 65071

Ops, forgot the diff. 2015-02-18 9:19 GMT+01:00 Andrea Azzarone : > Hi all, > > this patch try to fix PR c++/65071 (ICE on valid, sizeof...() of > template template parameter pack in return type). > > 2015-2-18 Andrea Azzarone > PR c++/65071 > * gcc/cp/parser.c (cp_parser_sizeof_pack) Also co

[PATCH][C++] PR 65071

Hi all, this patch try to fix PR c++/65071 (ICE on valid, sizeof...() of template template parameter pack in return type). 2015-2-18 Andrea Azzarone PR c++/65071 * gcc/cp/parser.c (cp_parser_sizeof_pack) Also consider template template parameters. Thanks. -- Andrea Azzarone

Re: [PATCH, AArch64] Handle SYMBOL_SMALL_TPREL appropriately

On 18 February 2015 at 04:45, Hurugalawadi, Naveen wrote: > Hi Marcus, > > Thanks for the review. > >>> OK, but fix the trailing white space in the patch > > Done. Committed with the modification. > >>> Can you prepare a backport into 4.9 > > ILP32 support is not completely added in 4.9 and hence

Re: [Haifa Scheduler] Fix latent bug in macro-fusion/instruction grouping

On Feb 17, 2015, at 9:43 AM, Jeff Law wrote: > On 02/11/15 02:20, James Greenhalgh wrote: >> >> On Mon, Feb 09, 2015 at 11:16:56PM +, Jeff Law wrote: >>> On 02/06/15 05:24, James Greenhalgh wrote: --- 2015-02-06 James Greenhalgh * haifa-sched.c (recompute_tod

Re: The nvptx port [0/11+]

Hi! On Mon, 20 Oct 2014 16:17:56 +0200, Bernd Schmidt wrote: > This is a patch kit that adds the nvptx port to gcc. Committed to trunk in r220781: commit 0f7695734890f93fe58179e36ac2f41bf4147d78 Author: tschwinge Date: Wed Feb 18 08:01:03 2015 + nvptx-none: Disable the lto-plugin.