Re: [PATCH] Add pow -> exp hack for SPEC2k17 628.pop2_s (PR tree-optimization/82004)

2018-03-28 Thread Richard Biener
On Tue, 27 Mar 2018, Jakub Jelinek wrote: > Hi! > > As mentioned in the PR, sw_absorption.fppized.f90 relies on pow in > x = log10 (something) - y; > for (...) > { > x = x + y; > z = pow (10.0, x); > } > where x + y in the first iteration is exactly -3 to be >= 0.001, > unfortunately

Re: [PATCH] [PR c++/84979] improve auto handling in explicit tmpl args for concepts

2018-03-28 Thread Alexandre Oliva
On Mar 23, 2018, Jason Merrill wrote: > On Fri, Mar 23, 2018 at 3:38 PM, Alexandre Oliva wrote: >> + /* Concepts allows 'auto' in template arguments, even multiple >> + 'auto's in a single argument. > I think that's only intended for class templates; Aah, that explains a lot! Dang, it wa

Re: [PATCH PR81647][AARCH64] Fix handling of Unordered Comparisons in aarch64-simd.md

2018-03-28 Thread Sudakshina Das
Hi On 20/03/18 10:57, Sudakshina Das wrote: Hi On 20/03/18 08:13, Christophe Lyon wrote: On 19 March 2018 at 19:55, Sudakshina Das wrote: Hi On 19/03/18 14:29, James Greenhalgh wrote: On Fri, Dec 15, 2017 at 11:57:46AM +, Sudi Das wrote: Hi This patch fixes the inconsistent behavi

Re: [PR middle-end/70359] uncoalesce IVs outside of loops

2018-03-28 Thread Richard Biener
On Tue, Mar 20, 2018 at 2:36 PM, Richard Biener wrote: > On Mon, Mar 19, 2018 at 6:08 PM, Aldy Hernandez wrote: >> Hi Richard. >> >> As discussed in the PR, the problem here is that we have two different >> iterations of an IV live outside of a loop. This inhibits us from using >> autoinc/dec ad

Re: [PATCH] Fix compile-time hog in MPX boundary checking (PR target/84988).

2018-03-28 Thread Jakub Jelinek
On Wed, Mar 28, 2018 at 08:31:38AM +0200, Martin Liška wrote: > Ok, so should we make the set of cum->bnds_in_bt based on > flag_check_pointer_bounds flag? > > If so, I've got patch that I've tested on my x86_64-linux-gnu machin. > > Martin > >From 7b5978e61305c5098a084c2352fcbacb4c347158 Mon S

Re: [PATCH][i386,AVX] Fix PR84783 - backport missing permutexvar to GCC7

2018-03-28 Thread Jakub Jelinek
On Tue, Mar 27, 2018 at 03:18:12PM +, Peryt, Sebastian wrote: > Hi Jakub, > > Gentle ping. Ok for 7.4. > > > 2018-03-22 Sebastian Peryt > > > > > > gcc: > > > PR84783 > > > * config/i386/avx512vlintrin.h (_mm256_permutexvar_epi64) > > > (_mm256_permutexvar_epi32, _mm256_permutex_epi64)

[C++ Patch] PR 85028 ("[8 Regression] ICE on invalid C++ code: in tsubst_default_argument, at cp/pt.c:12340")

2018-03-28 Thread Paolo Carlini
Hi, as I said in the audit trail, in its way this error recovery issue is somewhat interesting: Jason's r251422 added some code at the beginning of tsubst_default_argument, included the gcc_assert that triggers here. In fact, parmtype is only used in the assertion thus the error recovery chec

Re: [PATCH,nvptx] Fix PR85056

2018-03-28 Thread Cesar Philippidis
On 03/27/2018 01:17 AM, Tom de Vries wrote: > On 03/26/2018 11:57 PM, Cesar Philippidis wrote: >> As noted in PR85056, the nvptx BE isn't declaring external arrays using >> PTX array notation. Specifically, it's emitting code that's missing the >> empty angle brackets '[]'. > > [ FYI, see https:/

Re: [PATCH,nvptx] Fix PR85056

2018-03-28 Thread Tom de Vries
On 03/28/2018 03:43 PM, Cesar Philippidis wrote: OK for stage4 trunk. Can I backport this patch to GCC 6 and 7? Yes please. Thanks, - Tom

Re: [PATCH] Add pow -> exp hack for SPEC2k17 628.pop2_s (PR tree-optimization/82004, take 2)

2018-03-28 Thread Richard Biener
On Wed, 28 Mar 2018, Jakub Jelinek wrote: > On Wed, Mar 28, 2018 at 09:46:52AM +0200, Richard Biener wrote: > > > +/* Return true if pow(cst, x) should be optimized into exp(log(cst) * > > > x). */ > > > + > > > +static bool > > > +optimize_pow_to_exp (tree arg0, tree arg1) > > > +{ > > > + ret

[PATCH, GCC-7, GCC-6][ARM][PR target/84826] Backport Fix ICE in extract_insn, at recog.c:2304 on arm-linux-gnueabihf

2018-03-28 Thread Sudakshina Das
Hi This patch is a request to backport r258777 and r258805 to gcc-7-branch and gcc-6-branch. The same ICE occurs in both the branches with -fstack-check. Thus the test case directive has been changed. The discussion on the patch that went into trunk is: https://gcc.gnu.org/ml/gcc-patches/2018-03

[PATCH] Fix wrong use-after-scope sanitization for omp variable (PR sanitizer/85081).

2018-03-28 Thread Martin Liška
Hi. I'm sending Jakub's patch, where I removed the guard in asan_poison_variable. Patch can bootstrap on ppc64le-redhat-linux and survives regression tests. Martin gcc/ChangeLog: 2018-03-28 Jakub Jelinek Martin Liska PR sanitizer/85081 * gimplify.c (asan_poiso

Re: [PATCH] Improve adc discovery during combine on x86 (PR target/85095, take 2)

2018-03-28 Thread Uros Bizjak
On Wed, Mar 28, 2018 at 2:54 PM, Jakub Jelinek wrote: > On Wed, Mar 28, 2018 at 08:51:07AM +0200, Uros Bizjak wrote: >> > --- gcc/config/i386/i386.md.jj 2018-03-27 12:54:54.685244368 +0200 >> > +++ gcc/config/i386/i386.md 2018-03-27 19:38:43.891451026 +0200 >> > @@ -6854,6 +6854,23 @@ (define

Re: [PATCH] Fix wrong use-after-scope sanitization for omp variable (PR sanitizer/85081).

2018-03-28 Thread Jakub Jelinek
On Wed, Mar 28, 2018 at 04:14:45PM +0200, Martin Liška wrote: > Hi. > > I'm sending Jakub's patch, where I removed the guard in asan_poison_variable. > Patch can bootstrap on ppc64le-redhat-linux and survives regression tests. > > Martin > > gcc/ChangeLog: > > 2018-03-28 Jakub Jelinek >

Re: [PATCH] Prefer mempcpy to memcpy on x86_64 target (PR middle-end/81657).

2018-03-28 Thread Martin Liška
On 03/21/2018 11:34 AM, Jakub Jelinek wrote: On Wed, Mar 14, 2018 at 02:54:00PM +0100, Martin Liška wrote: The variable is not named very well, shouldn't it be avoid_libcall or something similar? Perhaps the variable should have a comment describing what it is. Do you need separate argument fo

Re: [C++ Patch] PR 85028 ("[8 Regression] ICE on invalid C++ code: in tsubst_default_argument, at cp/pt.c:12340")

2018-03-28 Thread Jason Merrill
OK. On Wed, Mar 28, 2018 at 8:25 AM, Paolo Carlini wrote: > Hi, > > as I said in the audit trail, in its way this error recovery issue is > somewhat interesting: Jason's r251422 added some code at the beginning of > tsubst_default_argument, included the gcc_assert that triggers here. In > fact, p

Re: [PR middle-end/70359] uncoalesce IVs outside of loops

2018-03-28 Thread Richard Biener
On Wed, Mar 28, 2018 at 1:37 PM, Richard Biener wrote: > On Tue, Mar 20, 2018 at 2:36 PM, Richard Biener > wrote: >> On Mon, Mar 19, 2018 at 6:08 PM, Aldy Hernandez wrote: >>> Hi Richard. >>> >>> As discussed in the PR, the problem here is that we have two different >>> iterations of an IV live

Re: [PATCH] Prefer mempcpy to memcpy on x86_64 target (PR middle-end/81657).

2018-03-28 Thread Jakub Jelinek
On Wed, Mar 28, 2018 at 04:18:45PM +0200, Martin Liška wrote: > 2018-03-14 Martin Liska > > PR middle-end/81657 > * builtins.c (expand_builtin_memory_copy_args): Handle situation > when libc library provides a fast mempcpy implementation/ > * config/i386/i386-protos.h (g

[PATCH] Improve adc discovery during combine on x86 (PR target/85095, take 2)

2018-03-28 Thread Jakub Jelinek
On Wed, Mar 28, 2018 at 08:51:07AM +0200, Uros Bizjak wrote: > > --- gcc/config/i386/i386.md.jj 2018-03-27 12:54:54.685244368 +0200 > > +++ gcc/config/i386/i386.md 2018-03-27 19:38:43.891451026 +0200 > > @@ -6854,6 +6854,23 @@ (define_insn "add3_carry" > > (set_attr "pent_pair" "pu") > >

[PATCH] Add pow -> exp hack for SPEC2k17 628.pop2_s (PR tree-optimization/82004, take 2)

2018-03-28 Thread Jakub Jelinek
On Wed, Mar 28, 2018 at 09:46:52AM +0200, Richard Biener wrote: > > +/* Return true if pow(cst, x) should be optimized into exp(log(cst) * x). > > */ > > + > > +static bool > > +optimize_pow_to_exp (tree arg0, tree arg1) > > +{ > > + return false; > > +} > > So instead of this simply guard the

Re: [PATCH] Fix wrong use-after-scope sanitization for omp variable (PR sanitizer/85081).

2018-03-28 Thread Martin Liška
On 03/28/2018 04:17 PM, Jakub Jelinek wrote: On Wed, Mar 28, 2018 at 04:14:45PM +0200, Martin Liška wrote: Hi. I'm sending Jakub's patch, where I removed the guard in asan_poison_variable. Patch can bootstrap on ppc64le-redhat-linux and survives regression tests. Martin gcc/ChangeLog: 2018-0

Re: [PATCH] Fix typos (PR other/84819).

2018-03-28 Thread Martin Liška
On 03/20/2018 04:49 PM, Martin Liška wrote: On 03/20/2018 04:28 PM, Gerald Pfeifer wrote: On Tue, 20 Mar 2018, Martin Liška wrote: There's patch for couple of documentation issues reported in the PR. Looks good to me, thanks! Just, should there be an "a" in "use jump table"? Looks good. An

Hear from Columbia University Medical Center, UCLA, Colorado State Universtiy & More

2018-03-28 Thread CannaTherapeutics
CANNABINOID THERAPEUTICS SYMPOSIUM June 6 - 7, 2018 | Crowne Plaza Redondo Beach & Marina Hotel | Redondo Beach, CA --- SIGN UP TODAY: http://links.infocastevents.mkt8115.com/ctt?kn=5&ms=MzM2NTI0NzUS1&r=NjkyMTk1NzM3MTk0S0&b=2&j=MTI0MzQyNDM2NgS2&mt=1&rt=0 --- Join the evolving conversation arou

Re: Full range of PCBs inquiry

2018-03-28 Thread Mr. Aaron
Dear Friends, I am Mr. Aaron from FullyGold Holdings, we can provide high-end PCBs(Printed Circuit Boards) to our customer side. Specialize in mulitlayer(4L~50L) PCB fields more than 13 years, also including rigid, rigid-flex, FPC, and hybrid lam., and high performance PCB, our many clients

Re: [PATCH. rs6000] Fix PR84912: ICE using -m32 on __builtin_divde*, patch #2

2018-03-28 Thread Peter Bergner
On 3/27/18 5:02 PM, Segher Boessenkool wrote: >> @@ -15952,6 +15953,10 @@ rs6000_invalid_builtin (enum rs6000_buil >> name); >>else if ((fnmask & RS6000_BTM_FLOAT128) != 0) >> error ("builtin function %qs requires the %qs option", name, >> "-mfloat128"); >> + else if ((fnmask & (

Re: [patch, fortran] Simplify constants which come from parameter arrays

2018-03-28 Thread Dominique d'Humières
Hi Thomas, If I am not mistaken, the patch causes: FAIL: gfortran.dg/pr71935.f90 -O (test for excess errors) FAIL: gfortran.dg/substr_6.f90 -O0 execution test FAIL: gfortran.dg/substr_6.f90 -O1 execution test FAIL: gfortran.dg/substr_6.f90 -O2 execution test FAIL: gfortran.dg/substr_6

Re: [PATCH] Prefer mempcpy to memcpy on x86_64 target (PR middle-end/81657).

2018-03-28 Thread Martin Liška
On 03/28/2018 04:31 PM, Jakub Jelinek wrote: On Wed, Mar 28, 2018 at 04:18:45PM +0200, Martin Liška wrote: 2018-03-14 Martin Liska PR middle-end/81657 * builtins.c (expand_builtin_memory_copy_args): Handle situation when libc library provides a fast mempcpy implementa

Re: [PATCH. rs6000] Fix PR84912: ICE using -m32 on __builtin_divde*, patch #2

2018-03-28 Thread Segher Boessenkool
On Wed, Mar 28, 2018 at 10:38:49AM -0500, Peter Bergner wrote: > On 3/27/18 5:02 PM, Segher Boessenkool wrote: > >> @@ -15952,6 +15953,10 @@ rs6000_invalid_builtin (enum rs6000_buil > >> name); > >>else if ((fnmask & RS6000_BTM_FLOAT128) != 0) > >> error ("builtin function %qs requir

Re: [PATCH] Prefer mempcpy to memcpy on x86_64 target (PR middle-end/81657).

2018-03-28 Thread Jakub Jelinek
On Wed, Mar 28, 2018 at 06:30:21PM +0200, Martin Liška wrote: > --- a/gcc/config/linux.c > +++ b/gcc/config/linux.c > @@ -37,3 +37,24 @@ linux_libc_has_function (enum function_class fn_class) > >return false; > } > + > +/* This hook determines whether a function from libc has a fast > imple

Re: [PATCH] [PR c++/84979] improve auto handling in explicit tmpl args for concepts

2018-03-28 Thread Jason Merrill
On Wed, Mar 28, 2018 at 5:06 AM, Alexandre Oliva wrote: > On Mar 23, 2018, Jason Merrill wrote: > >> On Fri, Mar 23, 2018 at 3:38 PM, Alexandre Oliva wrote: >>> + /* Concepts allows 'auto' in template arguments, even multiple >>> + 'auto's in a single argument. > >> I think that's only inte

[PATCH,rs6000] Remove __builtin_fctid and __builtin_fctiw

2018-03-28 Thread Carl Love
GCC Maintainers: The following patch reverts commit 253238 to add support for the fctid and fctiw builtins for Steve Munroe. gcc/Chan

Re: [PATCH. rs6000] Fix PR84912: ICE using -m32 on __builtin_divde*, patch #2

2018-03-28 Thread Peter Bergner
On 3/28/18 12:59 PM, Segher Boessenkool wrote: > It should be something like > > +error ("builtin function %qs requires the %qs (or newer), and " > "%qs or %qs options", > +name, "-mcpu=power7", "-m64", "-mpowerpc64"); > > I don't see other such strings that quote incorrectly

Re: [PATCH] [PR c++/84943] allow folding of array indexing indirect_ref

2018-03-28 Thread Jason Merrill
On Wed, Mar 28, 2018 at 2:18 AM, Alexandre Oliva wrote: > On Mar 23, 2018, Jason Merrill wrote: > >> On Fri, Mar 23, 2018 at 4:55 PM, Jason Merrill wrote: >>> On Fri, Mar 23, 2018 at 12:44 PM, Jason Merrill wrote: Seems like cp_fold should update CALL_EXPR_FN with "callee" if non-null. >>>

[PATCH] Fix -Wduplicated-branches with -g (PR c/85094)

2018-03-28 Thread Jakub Jelinek
Hi! With the introduction of DEBUG_BEGIN_STMT/-gstatement-frontiers on by default, the -Wduplicated-branches warning doesn't work anymore with -g, because operand_equal_p in OEP_LEXICOGRAPHIC mode doesn't consider DEBUG_BEGIN_STMTs as equal unless they are pointer equal. For the warning we either

[PATCH] Fix _blendm

2018-03-28 Thread Jakub Jelinek
Hi! When looking at PR85090, I've looked into tmp-mddump.md and noticed there some instructions that can't assemble - vblendmd etc., there is only vpblendmd or vblendmps. I couldn't make a testcase that would reproduce this though, seems the masked *mov_internal is used instead. Bootstrapped/reg

[PATCH 2/2] Show pertinent parameter (PR c++/85110)

2018-03-28 Thread David Malcolm
This followup patch updates the specific error-handling path to add a note showing the pertinent parameter decl, taking the output from: test.cc: In function 'void caller(const char*)': test.cc:6:14: error: cannot convert 'const char*' to 'const char**' for argument '2' to 'void callee(int, const

[PATCH 1/2] More underlining of bad arguments (PR c++/85110)

2018-03-28 Thread David Malcolm
As of r256448, the C++ frontend underlines many bad arguments in its diagnostics; those where perform_overload_resolution returns a non-NULL candidate, but there's a failure in convert_like_real. However, for the case where perform_overload_resolution fails, but there's a single non-viable candida

Re: [PATCH,rs6000] Remove __builtin_fctid and __builtin_fctiw

2018-03-28 Thread Segher Boessenkool
Hi Carl, On Wed, Mar 28, 2018 at 11:51:09AM -0700, Carl Love wrote: > gcc/ChangeLog: > > 2018-03-20 Carl Love > > Reverting patch: > 2017-09-27 Carl Love > > * config/rs6000/rs6000-builtin.def: Remove macro expansion for > FCTIW and FCTID. Remove macro definition B

Re: [PATCH. rs6000] Fix PR84912: ICE using -m32 on __builtin_divde*, patch #2

2018-03-28 Thread Segher Boessenkool
On Wed, Mar 28, 2018 at 01:57:34PM -0500, Peter Bergner wrote: > On 3/28/18 12:59 PM, Segher Boessenkool wrote: > > It should be something like > > > > +error ("builtin function %qs requires the %qs (or newer), and " > >"%qs or %qs options", > > + name, "-mcpu=power7", "-m64", "-m

Re: [PATCH. rs6000] Fix PR84912: ICE using -m32 on __builtin_divde*, patch #2

2018-03-28 Thread Peter Bergner
On 3/28/18 4:13 PM, Segher Boessenkool wrote: > On Wed, Mar 28, 2018 at 01:57:34PM -0500, Peter Bergner wrote: >> On 3/28/18 12:59 PM, Segher Boessenkool wrote: >>> It should be something like >>> >>> +error ("builtin function %qs requires the %qs (or newer), and " >>>"%qs or %qs option

Re: [PATCH. rs6000] Fix PR84912: ICE using -m32 on __builtin_divde*, patch #2

2018-03-28 Thread Segher Boessenkool
On Wed, Mar 28, 2018 at 07:13:36PM -0500, Peter Bergner wrote: > On 3/28/18 4:13 PM, Segher Boessenkool wrote: > > On Wed, Mar 28, 2018 at 01:57:34PM -0500, Peter Bergner wrote: > >> On 3/28/18 12:59 PM, Segher Boessenkool wrote: > >>> It should be something like > >>> > >>> +error ("builtin fu

Unauthorized sign in attempt!

2018-03-28 Thread Apple