Re: PATCH: Add -mlong-double-128 and make it default for 64-bit Bionic

2014-01-31 Thread Uros Bizjak
On Fri, Jan 31, 2014 at 2:59 PM, H.J. Lu hongjiu...@intel.com wrote: For 64-bit Android, long double is 128-bit IEEE-754 floating point type. This patch adds -mlong-double-128 to i386 and makes it default for 64-bit Bionic. I only added MASK_LONG_DOUBLE_128. I made -mlong-double-128,

Re: PATCH: Add -mlong-double-128 and make it default for 64-bit Bionic

2014-01-31 Thread Uros Bizjak
On Fri, Jan 31, 2014 at 5:58 PM, Andrew Pinski pins...@gmail.com wrote: On Fri, Jan 31, 2014 at 5:59 AM, H.J. Lu hongjiu...@intel.com wrote: Hi, For 64-bit Android, long double is 128-bit IEEE-754 floating point type. This patch adds -mlong-double-128 to i386 and makes it default for 64-bit

Re: [testsuite, committed] Fix unresolved vector26.C for c++98

2014-02-01 Thread Uros Bizjak
Hello! The scan-assembler test in vector26.C was showing up unresolved for c++98, since in that case we report an expected error and don't produce any asm. Tested on mips64-linux-gnu and applied as obvious. -// { dg-final { scan-assembler-not globl\[ \t]*_Z3bar } } +// { dg-final {

[PATCH, i386]: Fix PR 60017 Struct not returned correctly

2014-02-02 Thread Uros Bizjak
Hello! There is off-by-one error in classify_argument, when processing integer atomic types. When bit_offset is 64, two X86_64_INTEGER CLASS registers should be used. Attached patch fixes this off-by-one error, so we won't use size = 0 when bit_offset = 64. 2014-02-02 Uros Bizjak ubiz

Re: [PATCH, i386]: Fix PR 60017 Struct not returned correctly

2014-02-02 Thread Uros Bizjak
On Sun, Feb 2, 2014 at 5:35 PM, Jakub Jelinek ja...@redhat.com wrote: On Sun, Feb 02, 2014 at 04:31:35PM +0100, Uros Bizjak wrote: There is off-by-one error in classify_argument, when processing integer atomic types. When bit_offset is 64, two X86_64_INTEGER CLASS registers should be used

[4.7 PATCH]: Backport 'gcc_update explicit use of svn'

2014-02-02 Thread Uros Bizjak
Hello! Attached patch backports 'gcc_update explicit use of svn' patch [1] to 4.7 branch. Without this patch, contrib/gcc_update command produces corrupted gcc/REVISION on my F20 system. 2014-02-02 Uros Bizjak ubiz...@gmail.com Backport from mainline 2012-05-09 David

[PATCH, testsuite]: Lower timeout value for trivial tsan testcase check

2014-02-04 Thread Uros Bizjak
Hello! Trivial tsan testcase check (tsan_init in lib/tsan-dg.exp) does not terminate properly on CentOS 5.10. Instead of waiting for the default of 300 seconds, attached patch limits total execution time to 20 seconds. 2014-02-04 Uros Bizjak ubiz...@gmail.com * lib/tsan-dg.exp (tsan_init

Re: [testsuite] Fix gcc.target/i386/avx512f-vrndscaless-2.c on Solaris 9/x86

2014-02-05 Thread Uros Bizjak
On Wed, Feb 5, 2014 at 4:50 PM, Rainer Orth r...@cebitec.uni-bielefeld.de wrote: gcc.target/i386/avx512f-vrndscaless-2.c currently FAILs on Solaris 9/x86 with gas: FAIL: gcc.target/i386/avx512f-vrndscaless-2.c (test for excess errors) Excess errors:

Re: [testsuite] Fix gcc.target/i386/avx512f-vrndscaless-2.c on Solaris 9/x86

2014-02-05 Thread Uros Bizjak
On Wed, Feb 5, 2014 at 5:11 PM, Jakub Jelinek ja...@redhat.com wrote: gcc.target/i386/avx512f-vrndscaless-2.c currently FAILs on Solaris 9/x86 with gas: FAIL: gcc.target/i386/avx512f-vrndscaless-2.c (test for excess errors) Excess errors:

Re: [PATCH] Fix ix86_function_regparm with optimize attribute (PR target/60062, take 3)

2014-02-06 Thread Uros Bizjak
On Thu, Feb 6, 2014 at 10:45 AM, Richard Biener rguent...@suse.de wrote: On Thu, 6 Feb 2014, Jakub Jelinek wrote: On Wed, Feb 05, 2014 at 08:42:27PM +0100, Jakub Jelinek wrote: So, where do we want to do that instead? E.g. should it be e.g. in tree_versionable_function_p directly and let

Re: [PATCH 12/8] [AVX-512] Improve EAS, ICC, GCC conformance.

2014-02-07 Thread Uros Bizjak
On Fri, Feb 7, 2014 at 10:49 AM, Kirill Yukhin kirill.yuk...@gmail.com wrote: This (should be) the last patch for AVX-512 support in v4.9. It improves correspondence between ICC, SDM [1], and official intrinsics guide [2]. What was done: - Fixed shifts such as VPSLLD and friends. Actual

Re: [PATCH 12/8] [AVX-512] Improve EAS, ICC, GCC conformance.

2014-02-07 Thread Uros Bizjak
On Fri, Feb 7, 2014 at 11:13 AM, Uros Bizjak ubiz...@gmail.com wrote: This (should be) the last patch for AVX-512 support in v4.9. It improves correspondence between ICC, SDM [1], and official intrinsics guide [2]. What was done: - Fixed shifts such as VPSLLD and friends. Actual

Re: [testsuite, i386] Compile g++.dg/ext/vector26.C with -mmmx

2014-02-10 Thread Uros Bizjak
On Mon, Feb 10, 2014 at 5:14 PM, Rainer Orth r...@cebitec.uni-bielefeld.de wrote: On both Solaris 9/x86 and 32-bit Linux/x86, g++.dg/ext/vector26.C FAILs like this: FAIL: g++.dg/ext/vector26.C -std=c++11 (test for excess errors) Excess errors:

[PATCH, testsuite]: Revert PR testsuite/58630 on mainline

2014-02-11 Thread Uros Bizjak
Hello! Now that Richard fixed ms_abi limitation w.r.t. accumulate-outgoing-args, we can revert PR testsuite/58630 fix on mainline. On mainline, ms_abi works without problems with and without this option. 2014-02-11 Uros Bizjak ubiz...@gmail.com PR target/59927 Revert 2013-12-15

Re: PATCH: PR target/60151: HAVE_AS_GOTOFF_IN_DATA is mis-detected on x86-64

2014-02-11 Thread Uros Bizjak
On Tue, Feb 11, 2014 at 8:28 PM, H.J. Lu hongjiu...@intel.com wrote: HAVE_AS_GOTOFF_IN_DATA defines a 32-bit assembler feature, we need to pass --32 to assembler. Otherwise, we get the wrong result on x86-64. We already pass --32 to assembler on x86. It should be OK to do it in configure.

Re: PATCH: PR target/60151: HAVE_AS_GOTOFF_IN_DATA is mis-detected on x86-64

2014-02-12 Thread Uros Bizjak
On Tue, Feb 11, 2014 at 9:41 PM, H.J. Lu hjl.to...@gmail.com wrote: HAVE_AS_GOTOFF_IN_DATA defines a 32-bit assembler feature, we need to pass --32 to assembler. Otherwise, we get the wrong result on x86-64. We already pass --32 to assembler on x86. It should be OK to do it in configure. OK

Re: [PATCH i386 13/8] [AVX-512] Fix argument order for perm and recp intrinsics.

2014-02-13 Thread Uros Bizjak
On Thu, Feb 13, 2014 at 11:44 AM, Kirill Yukhin kirill.yuk...@gmail.com wrote: I've noticed that _mm512_permutexvar_epi[64|32] intrinsics have wrong arguments order. As per [1] first argument is index. For vmpermps/vpermpd intrinsics are fine, but I've changed tests to call CALC with same arg

Re: [PATCH i386 13/8] [AVX-512] Fix argument order for perm and recp intrinsics.

2014-02-13 Thread Uros Bizjak
On Thu, Feb 13, 2014 at 1:37 PM, Uros Bizjak ubiz...@gmail.com wrote: I've noticed that _mm512_permutexvar_epi[64|32] intrinsics have wrong arguments order. As per [1] first argument is index. For vmpermps/vpermpd intrinsics are fine, but I've changed tests to call CALC with same arg order

[PATCH, i386]: Fix xop_vmfrczmode2 expander

2014-02-13 Thread Uros Bizjak
Hello! No functional changes. 2014-02-13 Uros Bizjak ubiz...@gmail.com * config/i386/sse.md (xop_vmfrczmode2): Generate const0 in operands[2], not operands[3]. Tested on x86_64-pc-linux-gnu {,-m32} and committed to mainline SVN. Uros. Index: config/i386/sse.md

Re: [PATCH i386 13/8] [AVX-512] Fix argument order for perm and recp intrinsics.

2014-02-13 Thread Uros Bizjak
On Thu, Feb 13, 2014 at 1:55 PM, Uros Bizjak ubiz...@gmail.com wrote: I've noticed that _mm512_permutexvar_epi[64|32] intrinsics have wrong arguments order. As per [1] first argument is index. For vmpermps/vpermpd intrinsics are fine, but I've changed tests to call CALC with same arg order

Re: [PATCH] x86: Use ud2 assembly mnemonic when available.

2014-02-13 Thread Uros Bizjak
Hello! Non-ancient assemblers support the ud2 mnemonic, so there is no need to emit the literal opcode as data. OK for trunk and 4.8? You forgot to tell us how the patch tested... gcc/ 2014-02-13 Roland McGrath mcgra...@google.com * configure.ac (HAVE_AS_IX86_UD2): New test for 'ud2'

Re: [patch i386]: Fix PR/60193

2014-02-14 Thread Uros Bizjak
Hello! 2014-02-14 Kai Tietz kti...@redhat.com PR target/60193 * config/i386/i386.c (ix86_expand_prologue): Use rax register as displacement for restoring %r10, %eax. Regression-tested for x86_64-unknown-linux-gnu, and x86_64-w64-mingw32, and i686-w64-mingw32. Ok for apply?

Re: [patch i386]: Fix PR/60193

2014-02-14 Thread Uros Bizjak
On Fri, Feb 14, 2014 at 2:48 PM, Kai Tietz ktiet...@googlemail.com wrote: 2014-02-14 13:55 GMT+01:00 Uros Bizjak ubiz...@gmail.com: Hello! 2014-02-14 Kai Tietz kti...@redhat.com PR target/60193 * config/i386/i386.c (ix86_expand_prologue): Use rax register as displacement

Re: [patch i386]: Fix PR/60193

2014-02-14 Thread Uros Bizjak
On Fri, Feb 14, 2014 at 3:50 PM, Kai Tietz ktiet...@googlemail.com wrote: 2014-02-14 15:40 GMT+01:00 Uros Bizjak ubiz...@gmail.com: On Fri, Feb 14, 2014 at 2:48 PM, Kai Tietz ktiet...@googlemail.com wrote: 2014-02-14 13:55 GMT+01:00 Uros Bizjak ubiz...@gmail.com: Hello! 2014-02-14 Kai Tietz

Re: [patch i386]: Fix PR/60193

2014-02-14 Thread Uros Bizjak
On Fri, Feb 14, 2014 at 4:19 PM, Kai Tietz ktiet...@googlemail.com wrote: Adjusted my original testcase so that eax isn't redeclared and shadows. Additional moved initialization of eax_live up. ChangeLog 2014-02-14 Kai Tietz kti...@redhat.com PR target/60193 *

Re: [PATCH i386 13/8] [AVX-512] Fix argument order for perm and recp intrinsics.

2014-02-17 Thread Uros Bizjak
On Mon, Feb 17, 2014 at 1:26 PM, Kirill Yukhin kirill.yuk...@gmail.com wrote: Please don't change srcp pattern, it should be defined similar to vrcpss (aka sse_vmrcpv4sf). You need to switch operand order elsewhere. No, you are correct. Operands should be swapped as in your patch. Eh,

Re: [PATCH] Fix -march=native (PR driver/60233)

2014-02-17 Thread Uros Bizjak
if OS doesn't save YMM state was done before it was set. Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? 2014-02-17 Jakub Jelinek ja...@redhat.com Uros Bizjak ubiz...@gmail.com PR driver/60233 * config/i386/driver-i386.c

Re: [PATCH i386 13/8] [AVX-512] Fix argument order for perm and recp intrinsics.

2014-02-18 Thread Uros Bizjak
On Tue, Feb 18, 2014 at 11:06 AM, Kirill Yukhin kirill.yuk...@gmail.com wrote: Please don't change srcp pattern, it should be defined similar to vrcpss (aka sse_vmrcpv4sf). You need to switch operand order elsewhere. No, you are correct. Operands should be swapped as in your patch.

[PATCH, i386]: Fix PR 60205: No ABI warning for AVX-512

2014-02-18 Thread Uros Bizjak
Hello! Just following what AVX warnings do. 2014-02-18 Uros Bizjak ubiz...@gmail.com PR target/60205 * config/i386/i386.h (struct ix86_args): Add warn_avx512f. * config/i386/i386.c (init_cumulative_args): Initialize warn_avx512f. (type_natural_mode): Warn ABI change when %zmm

[PATCH, i386]: Warn for ABI changes only when -Wpsabi is enabled

2014-02-18 Thread Uros Bizjak
Hello! Also use boolean AND instead of bitwise AND in a couple of places. 2014-02-18 Uros Bizjak ubiz...@gmail.com PR target/59794 * config/i386/i386.c (type_natural_mode): Warn for ABI changes only when -Wpsabi is enabled. testsuite/ChangeLog: 2014-02-18 Uros Bizjak ubiz

Re: [PATCH][i386][AVX512] PR60204 - update abi for large structs.

2014-02-19 Thread Uros Bizjak
On Wed, Feb 19, 2014 at 2:30 PM, Ilya Tocar tocarip.in...@gmail.com wrote: Hi everyone, As AVX512 abi for passing/returing structs was recently changed in https://github.com/hjl-tools/x86-64-psABI/commit/6d7ccd614fe67111d2aecec853c3df0310b372d2 We need to update GCC accordingly. This patch

Re: [Patch, Fortran] PR602864 - fix INQUIRE for write= with stdout/stdin/stderr

2014-02-20 Thread Uros Bizjak
Hello! A rather simple patch. Build and regtested on x86-64-gnu-linux. OK for the trunk? 2014-02-20 Tobias Burnus bur...@net-b.de PR fortran/602864 ... we are not there yet with PR numbers ;) Uros.

Re: [PATCH][i386][AVX512] Match latest spec.

2014-02-20 Thread Uros Bizjak
On Thu, Feb 20, 2014 at 4:39 PM, Ilya Tocar tocarip.in...@gmail.com wrote: Latest version of AVX512 spec http://download-software.intel.com/sites/default/files/managed/50/1a/319433-018.pdf Has a few changes. This patch fixes first of them: Vptestnmd and vptestnmq instructions now have CPUID

[RFA, RFC PATCH, rtl-optimization]: Assert that crtl-emit.regno_pointer_align_length is non-zero when calling gen_reg_rtx

2014-02-20 Thread Uros Bizjak
of gcc_checking_assert is in consistent ICE (and consequently consistent bugreports) instead of a data corruption even for the non-checking compiler. I am aware that this assert *could* trigger some ICEs on other targets, but these will be triggered _instead_ of internal data corruption. 2014-02-20 Uros Bizjak

Re: [RFA, RFC PATCH, rtl-optimization]: Assert that crtl-emit.regno_pointer_align_length is non-zero when calling gen_reg_rtx

2014-02-20 Thread Uros Bizjak
On Thu, Feb 20, 2014 at 9:05 PM, Jakub Jelinek ja...@redhat.com wrote: On Thu, Feb 20, 2014 at 08:34:46PM +0100, Uros Bizjak wrote: 2014-02-20 Uros Bizjak ubiz...@gmail.com * emit-rtl.c (gen_reg_rtx): Assert that crtl-emit.regno_pointer_align_length is non-zero. The patch

Re: [PATCH] Fix ix86_vectorize_vec_perm_const_ok (PR target/57896)

2014-02-20 Thread Uros Bizjak
On Thu, Feb 20, 2014 at 9:08 PM, Jakub Jelinek ja...@redhat.com wrote: active) with RTL checking, further tested with GCC_TEST_RUN_EXPENSIVE=1 make -j16 -k check RUNTESTFLAGS='--target_board=unix\{-msse2,-msse3,-mssse3,-msse4,-mavx,-mavx2,-mavx512f\} dg-torture.exp=*vshuf*' (on AVX HW, so

[4.7 PATCH] Fix ix86_vectorize_vec_perm_const_ok (PR target/57896)

2014-02-20 Thread Uros Bizjak
Uros Bizjak ubiz...@gmail.com Backport from mainline 2014-02-20 Jakub Jelinek ja...@redhat.com * config/i386/i386.c (ix86_expand_vec_perm): Use V8SImode mode for mask of V8SFmode permutation. Backport from 4.8 branch 2014-02-20 Jakub Jelinek ja...@redhat.com PR

Re: [PATCH][i386][AVX512] Match latest spec. Add CPUID prefetchwt1.

2014-02-21 Thread Uros Bizjak
On Fri, Feb 21, 2014 at 4:25 PM, Ilya Tocar tocarip.in...@gmail.com wrote: Latest version of AVX512 spec http://download-software.intel.com/sites/default/files/managed/50/1a/319433-018.pdf Has a few changes. 1)PREFETCHWT1 instruction now has separate CPUID bit PREFETCHWT1. We can

[PATCH, testsuite]: Add some missing avx512 options to g++.dg/other/i386-{2,3}.C and gcc.target/i386/sse-{12,13}.c

2014-02-21 Thread Uros Bizjak
Hello! No additional testsuite failures. 2014-02-21 Uros Bizjak ubiz...@gmail.com * g++.dg/other/i386-2.C (dg-options): Add -mavx512pf. * g++.dg/other/i386-3.C (dg-options): Ditto. * gcc.target/i386/sse-12.c (dg-options): Add -msha. * gcc.target/i386/sse-13.c (dg-options): Add

Re: FW: Non-temporal move

2014-02-25 Thread Uros Bizjak
On Tue, Feb 25, 2014 at 7:10 AM, Gopalasubramanian, Ganesh ganesh.gopalasubraman...@amd.com wrote: I could see storent pattern in x86 machine descriptions (in sse.md)., but internals doc don't mention it. Should we add a description about this in the internals doc? This pattern was added way

Re: [PATCH][i386][AVX512] Match latest spec. Add CPUID prefetchwt1.

2014-02-25 Thread Uros Bizjak
On Tue, Feb 25, 2014 at 10:13 AM, Ilya Tocar tocarip.in...@gmail.com wrote: Latest version of AVX512 spec http://download-software.intel.com/sites/default/files/managed/50/1a/319433-018.pdf Has a few changes. 1)PREFETCHWT1 instruction now has separate CPUID bit PREFETCHWT1. We

Re: [PATCH][i386][AVX512] Match latest spec.

2014-02-25 Thread Uros Bizjak
On Tue, Feb 25, 2014 at 5:04 PM, Ilya Tocar tocarip.in...@gmail.com wrote: Latest version of AVX512 spec http://download-software.intel.com/sites/default/files/managed/50/1a/319433-018.pdf Has a few changes. 2)Currently for scatter/gather prefetches intrinsics we accept 1 as possible

Re: [PATCH i386 14/8] [AVX-512] Fix exp2 and sqrt tests.

2014-02-28 Thread Uros Bizjak
On Fri, Feb 28, 2014 at 1:14 PM, Kirill Yukhin kirill.yuk...@gmail.com wrote: Hello, This is relatively obvious patch which eliminates comparision of inifinities for exp2 AVX-512 test and properly comparing floats for avx512f-sqrtps-2.c. Tests pass. Is it ok for trunk? gcc/testsuite/

Re: [PATCH i386 14/8] [AVX-512] Fix exp2 and sqrt tests.

2014-03-01 Thread Uros Bizjak
On Sat, Mar 1, 2014 at 7:13 AM, Kirill Yukhin kirill.yuk...@gmail.com wrote: Hello Uroš, On 28 Feb 13:55, Uros Bizjak wrote: On Fri, Feb 28, 2014 at 1:14 PM, Kirill Yukhin kirill.yuk...@gmail.com wrote: Hello, This is relatively obvious patch which eliminates comparision of inifinities

[PATCH, i386] Fix emitting of prefetch instructions

2014-03-03 Thread Uros Bizjak
} and committed to mainline SVN. 2014-03-03 Uros Bizjak ubiz...@gmail.com * config/i386/xmmintrin.h (enum _mm_hint) _MM_HINT_ET0: Correct hint value. (_mm_prefetch): Move out of GCC target(sse) pragma. * config/i386/prfchwintrin.h (_m_prefetchw): Move out of GCC target(prfchw

Re: [PATCH, i386] Fix emitting of prefetch instructions

2014-03-03 Thread Uros Bizjak
On Tue, Mar 4, 2014 at 12:31 AM, Uros Bizjak ubiz...@gmail.com wrote: The new gcc.target/i386/prefetchwt1-1.c test currently FAILs on Solaris 9/x86: FAIL: gcc.target/i386/prefetchwt1-1.c (test for excess errors) Excess errors: /var/gcc/regression/trunk/9-gcc-gas/build/gcc/include

Re: [PATCH, i386] Fix emitting of prefetch instructions

2014-03-04 Thread Uros Bizjak
On Tue, Mar 4, 2014 at 10:04 AM, Kirill Yukhin kirill.yuk...@gmail.com wrote: Hello Uroš, On 04 Mar 01:13, Uros Bizjak wrote: On Tue, Mar 4, 2014 at 12:31 AM, Uros Bizjak ubiz...@gmail.com wrote: They are all: FAIL: gcc.target/i386/avx512pf-vscatterpf0dpd-1.c (test for excess errors

Re: [PATCH, i386] Fix emitting of prefetch instructions

2014-03-04 Thread Uros Bizjak
On Tue, Mar 4, 2014 at 1:13 AM, Uros Bizjak ubiz...@gmail.com wrote: On Tue, Mar 4, 2014 at 12:31 AM, Uros Bizjak ubiz...@gmail.com wrote: The new gcc.target/i386/prefetchwt1-1.c test currently FAILs on Solaris 9/x86: FAIL: gcc.target/i386/prefetchwt1-1.c (test for excess errors) Excess

Re: [PATCH] Install config/i386/stringop.def as a plugin header

2014-03-05 Thread Uros Bizjak
On Wed, Mar 5, 2014 at 9:48 AM, Jakub Jelinek ja...@redhat.com wrote: Hi! Pretty much all plugins fail to compile against installed plugin headers on i?86, because stringop.def isn't installed. This patch should fix that, ok if testing succeeds? 2014-03-05 Jakub Jelinek ja...@redhat.com

Re: [PATCH] Install config/i386/stringop.def as a plugin header

2014-03-05 Thread Uros Bizjak
On Wed, Mar 5, 2014 at 10:20 AM, Jakub Jelinek ja...@redhat.com wrote: Pretty much all plugins fail to compile against installed plugin headers on i?86, because stringop.def isn't installed. This patch should fix that, ok if testing succeeds? 2014-03-05 Jakub Jelinek

Re: [build, i386] Disable local dynamic TLS model on Solaris/x86 if as/ld cannot handle it

2014-03-05 Thread Uros Bizjak
On Wed, Mar 5, 2014 at 4:53 PM, Rainer Orth r...@cebitec.uni-bielefeld.de wrote: When using GNU as with Solaris ld, the TLS local dynamic tests (gcc.dg/torture/tls/run-ld.c etc.) FAIL to execute before Solaris 11: the programs crash with an illegal instruction: e.g. in the gcc.dg/lto/20090210

Re: [PATCH, x86] X86 Silvermont vector cost model tune

2014-03-05 Thread Uros Bizjak
On Wed, Mar 5, 2014 at 5:46 PM, H.J. Lu hjl.to...@gmail.com wrote: On Wed, Mar 5, 2014 at 7:58 AM, Evgeny Stupachenko evstu...@gmail.com wrote: Hi, The patch is for x86 Silvermont. It improves x86 Silvermont vector cost model. It gives +20% on facerec spec on Silvermont. It passes make

[PATCH, libgcc]: Avoid warning: array subscript is above array bounds when compiling crtstuff.c

2014-03-07 Thread Uros Bizjak
. 2014-03-08 Uros Bizjak ubiz...@gmail.com * crtstuff.c (__JCR_LIST__): Declare as zero-length array. Bootstrapped and regression tested on x86_64-pc-linux-gnu {,-m32}. OK for mainline? Uros. Index: crtstuff.c === --- crtstuff.c

Re: [PATCH, libgcc]: Avoid warning: array subscript is above array bounds when compiling crtstuff.c

2014-03-10 Thread Uros Bizjak
On Mon, Mar 10, 2014 at 8:49 AM, Jakub Jelinek ja...@redhat.com wrote: On Sun, Mar 09, 2014 at 10:38:25PM +0100, Uros Bizjak wrote: On Sun, Mar 9, 2014 at 6:31 PM, Jakub Jelinek ja...@redhat.com wrote: On Sun, Mar 09, 2014 at 09:41:59AM -0700, Ian Lance Taylor wrote: Attached patch avoids

Re: [PATCH, libgcc]: Avoid warning: array subscript is above array bounds when compiling crtstuff.c

2014-03-10 Thread Uros Bizjak
On Mon, Mar 10, 2014 at 11:14 AM, Jakub Jelinek ja...@redhat.com wrote: Well, better is non-obvious, while it is smaller (which is good for initialization and thus rarely executed code), the common case is that *jcr_list is 0 (gcj is used rarely these days) and for the common case it is

Re: [PATCH, libgcc]: Avoid warning: array subscript is above array bounds when compiling crtstuff.c

2014-03-10 Thread Uros Bizjak
On Mon, Mar 10, 2014 at 11:27 AM, Uros Bizjak ubiz...@gmail.com wrote: Well, better is non-obvious, while it is smaller (which is good for initialization and thus rarely executed code), the common case is that *jcr_list is 0 (gcj is used rarely these days) and for the common case

[PATCH, libjava]: Avoid suggest parentheses around comparison in operand of '|' in java/lang/natObject.cc

2014-03-10 Thread Uros Bizjak
); ^ It looks that parentheses are missing around (address | LOCKED), otherwise the expression is always true - LOCKED is unconditionally defined to 1. Attached patch adds these parentheses (and removes a pair of superfluous ones). 2014-03-10 Uros Bizjak ubiz...@gmail.com * java/lang

[PATCH, libjava]: Cleanup include/dwarf2-signal.h to avoid several warnings

2014-03-12 Thread Uros Bizjak
of MAKE_THROW_FRAME. 2014-03-12 Uros Bizjak ubiz...@gmail.com * include/dwarf2-signal.h: Update copyright year. (SIGNAL_HANDLER): Remove _sip argument. Mark _p argument with __attribute__ ((__unused__)). (class java::lang::Throwable): Remove declaration. (MAKE_THROW_FRAME

[PATCH, libiberty]: Avoid 'right-hand operand of comma expression has no effect' when compiling regex.c

2014-03-13 Thread Uros Bizjak
' void *t = bzero (arr, 1); ^ 2014-03-13 Uros Bizjak ubiz...@gmail.com * regex.c (bzero) [!_LIBC]: Change return value to void. Patch was bootstrapped and regression tested on x86_64-pc-linux-gnu. OK for mainline (and release branches, perhaps)? Uros. Index: regex.c

Re: [PATCH, libiberty]: Avoid 'right-hand operand of comma expression has no effect' when compiling regex.c

2014-03-13 Thread Uros Bizjak
On Thu, Mar 13, 2014 at 6:30 PM, Ian Lance Taylor i...@google.com wrote: On Thu, Mar 13, 2014 at 3:36 AM, Uros Bizjak ubiz...@gmail.com wrote: Attached patch changes the return value of the bzero macro to void, as defined in a 4.3BSD: void bzero(void *s, size_t n); As an additional

Re: [PATCH, libiberty]: Avoid 'right-hand operand of comma expression has no effect' when compiling regex.c

2014-03-13 Thread Uros Bizjak
On Thu, Mar 13, 2014 at 10:24 PM, Pedro Alves pal...@redhat.com wrote: On 03/13/2014 10:36 AM, Uros Bizjak wrote: +# define bzero(s, n)(memset (s, '\0', n), (void) 0) AFAICS, the comma operator was only needed because of the intention to return 's'. If 's' is not be returned

Re: [PATCH, i386, AVX, AVX-512] Extend ADDITION_REGISTER_NAMES to XMMs and YMMs.

2014-03-17 Thread Uros Bizjak
On Mon, Mar 17, 2014 at 4:12 PM, H.J. Lu hjl.to...@gmail.com wrote: Patch in the bottom allows to use ymmXX and zmmXX register names in inline asm statements as well as in `register` variables definitions. New tests pass. Bootstrap pass. Is it ok for trunk? Do we need to backport it to

Re: [PATCH] Fix i?86 pic thunk generation (PR target/60568)

2014-03-20 Thread Uros Bizjak
On Thu, Mar 20, 2014 at 10:05 AM, Jakub Jelinek ja...@redhat.com wrote: With -flto -fpic -m32 sometimes (unfortunately the testcase provided was in form of *.o file with LTO bytecode, so can't be reduced), if the thunk target doesn't bind locally we get ICE because we can't recog what

Re: [PATCH] x86: _mm512_set1_p[sd]

2014-03-24 Thread Uros Bizjak
Hello! On Mon, Mar 24, 2014 at 12:13 PM, Ulrich Drepper drep...@gmail.com wrote: Your patch is correct IMHO, but maybe it worst to add all missing `mm512_set1*' stuff? According to trunk and [1] we're still missing (beside mentioned by you) _mm512_set1_epi16 and _mm512_set1_epi8

Re: [PATCH] x86: _mm*_undefined_* (for real)

2014-03-24 Thread Uros Bizjak
Hello! 2014-03-19 Ulrich Drepper drep...@gmail.com * config/i386/avxintrin.h (_mm256_undefined_si256): Define. (_mm256_undefined_ps): Define. (_mm256_undefined_pd): Define. * config/i386/emmintrin.h (_mm_undefined_si128): Define. (_mm_undefined_pd):

Re: [PATCH] x86: _mm512_set1_p[sd]

2014-03-24 Thread Uros Bizjak
Hello! Another set of functions missing are those to set all elements of a 512-bit vector to the same float or double value. I think the patch below uses the optimal code sequence for that. The patch requires the previous patch introducing _mm*_undefined_*. 2014-03-19 Ulrich Drepper

Re: [PATCH] Fix ubsan i?86 {add,sub,mul}vmode4 patterns

2014-03-25 Thread Uros Bizjak
On Tue, Mar 25, 2014 at 8:24 AM, Jakub Jelinek ja...@redhat.com wrote: Hi! The bootstrap-ubsan bootstrap revealed a problem with the {add,sub,mul}vmode4 patterns. The problem is that they accept CONST_INT operands (because the instructions do accept immediates), but to model what the

Re: [PATCH] Fix ubsan i?86 {add,sub,mul}vmode4 patterns

2014-03-25 Thread Uros Bizjak
On Tue, Mar 25, 2014 at 11:17 AM, Jakub Jelinek ja...@redhat.com wrote: On Tue, Mar 25, 2014 at 09:12:14AM +0100, Uros Bizjak wrote: The bootstrap-ubsan bootstrap revealed a problem with the {add,sub,mul}vmode4 patterns. The problem is that they accept CONST_INT operands (because

Re: [PATCH] Fix and improve avx2 broadcasts (PR target/63594)

2014-10-21 Thread Uros Bizjak
On Tue, Oct 21, 2014 at 6:10 PM, Jakub Jelinek ja...@redhat.com wrote: Hi! This patch fixes a bunch of recent regressions: FAIL: gcc.target/i386/avx-1.c (internal compiler error) FAIL: gcc.target/i386/avx-1.c (test for excess errors) FAIL: gcc.target/i386/avx-2.c (internal compiler error)

Re: [PATCH i386 AVX512] [81.2/n] Add new built-ins.

2014-10-23 Thread Uros Bizjak
On Thu, Oct 23, 2014 at 2:50 PM, Kirill Yukhin kirill.yuk...@gmail.com wrote: Hello Uroš, On 20 Oct 17:41, Kirill Yukhin wrote: This patch adds (almost) all built-ins This is second, target dependent, part of the initial patch. ChangeLog is the same as in intial mail - tree* changes. Is it

Re: [testsuite] Fix gcc.dg/ipa/ipa-icf-2[18].c on Solaris

2014-10-23 Thread Uros Bizjak
Hello! Ok, will try that. Still no idea why Linux/x86 and Solaris/x86 are different here. Why do you think so? I certainly have: FAIL: gcc.dg/ipa/ipa-icf-21.c (test for excess errors) UNRESOLVED: gcc.dg/ipa/ipa-icf-21.c scan-ipa-dump icf Equal symbols: 1 UNRESOLVED:

Re: [testsuite] Fix gcc.dg/ipa/ipa-icf-2[18].c on Solaris

2014-10-23 Thread Uros Bizjak
On Thu, Oct 23, 2014 at 3:54 PM, Uros Bizjak ubiz...@gmail.com wrote: Ok, will try that. Still no idea why Linux/x86 and Solaris/x86 are different here. Why do you think so? I certainly have: FAIL: gcc.dg/ipa/ipa-icf-21.c (test for excess errors) UNRESOLVED: gcc.dg/ipa/ipa-icf-21.c scan

Re: [PATCH i386 AVX512] [82/n] Add gather built-ins.

2014-10-23 Thread Uros Bizjak
On Wed, Oct 22, 2014 at 2:17 PM, Kirill Yukhin kirill.yuk...@gmail.com wrote: Hello, This patch introduces built-ins for gather insns. Bootstrapped. AVX-512* tests on top of patch-set all pass under simulator. Is it ok for trunk? gcc/ * config/i386/i386.c (ix86_builtins): Add

Re: [PATCH i386 AVX512] [83/n] Add scatter built-ins.

2014-10-23 Thread Uros Bizjak
On Wed, Oct 22, 2014 at 2:19 PM, Kirill Yukhin kirill.yuk...@gmail.com wrote: Hello, This patch adds built-ins for scatter insns. Bootstrapped. AVX-512* tests on top of patch-set all pass under simulator. Is it ok for trunk? gcc/ * config/i386/i386.c

Re: [PATCH i386 AVX512] [84/n] Add missing immediate checks.

2014-10-23 Thread Uros Bizjak
On Wed, Oct 22, 2014 at 2:28 PM, Kirill Yukhin kirill.yuk...@gmail.com wrote: Hello, This tiny patch adds couple of missing immediate checks. Bootstrapped. AVX-512* tests on top of patch-set all pass under simulator. Is it ok for trunk? gcc/ * config/i386/i386.c

Re: [PATCH i386 AVX512] [89/n] Improve mask move insn generation.

2014-10-23 Thread Uros Bizjak
On Wed, Oct 22, 2014 at 2:58 PM, Kirill Yukhin kirill.yuk...@gmail.com wrote: Hello, This patch fixes kmov* insn generation and adds memory alternative to `movqi_internal' pattern Reg-test included. Bootstrapped. AVX-512* tests on top of patch-set all pass under simulator. Is it ok for

Re: [PATCH i386 AVX512] [85/n] Add intrinsics headers.

2014-10-23 Thread Uros Bizjak
On Wed, Oct 22, 2014 at 2:43 PM, Kirill Yukhin kirill.yuk...@gmail.com wrote: Hello, This patch introduces necessary AVX-512* intrinsics headers. Bootstrapped. AVX-512* tests on top of patch-set all pass under simulator. Is it ok for trunk? gcc/ * config/i386/avx512bwintrin.h:

Re: [PATCH i386 AVX512] [86/n] Add tests for options.

2014-10-23 Thread Uros Bizjak
On Wed, Oct 22, 2014 at 2:37 PM, Kirill Yukhin kirill.yuk...@gmail.com wrote: Hello, This patch extends sse-* and avx-* tests which checks immediates/options. Bootstrapped. AVX-512* tests on top of patch-set all pass under simulator. Is it ok for trunk? gcc/ *

Re: [PATCH i386 AVX512] [88/n] Add missed avx512bw/avx512vl cmp intrinsics.

2014-10-23 Thread Uros Bizjak
On Wed, Oct 22, 2014 at 2:51 PM, Kirill Yukhin kirill.yuk...@gmail.com wrote: Hello, This patch introduces rest of intrinsics (compare). Bootstrapped. AVX-512* tests on top of patch-set all pass under simulator. Is it ok for trunk? gcc/ * config/i386/avx512bwintrin.h: Add new

Re: [PATCH i386 AVX512] [87/n] Add rest of the tests.

2014-10-23 Thread Uros Bizjak
On Wed, Oct 22, 2014 at 2:59 PM, Kirill Yukhin kirill.yuk...@gmail.com wrote: On 22 Oct 16:48, Kirill Yukhin wrote: Hello, This patch adds tests for new intrinsics. Bootstrapped. AVX-512* tests on top of patch-set all pass under simulator. Is it ok for trunk? gcc/ *

[PATCH]: Guard division by zero in ipa-icf dumps

2014-10-24 Thread Uros Bizjak
Hello! 2014-10-24 Martin Liska mli...@suse.cz * ipa-icf.c (sem_item_optimizer::parse_nonsingleton_classes): Guard division by zero in dumps. (sem_item_optimizer::merge_classes): Ditto. Tested on alphaev68-linux-gnu, committed to mainline as obvious. Uros. diff --git

Re: [PATCH 1/2, x86, PR63534] Fix darwin bootstrap

2014-10-24 Thread Uros Bizjak
On Thu, Oct 23, 2014 at 11:19 PM, Evgeny Stupachenko evstu...@gmail.com wrote: Linux x86 bootstrap/make check passed. Darwin x86 bootstrap passed configured with --with-cpu=intel --with-cpu=corei7 issue refers to PR63620. On Fri, Oct 17, 2014 at 6:08 PM, Evgeny Stupachenko evstu...@gmail.com

[PATCH, alpha]: Fix c11 fallout in alpha testsuite

2014-10-24 Thread Uros Bizjak
2014-10-24 Uros Bizjak ubiz...@gmail.com * gcc.target/alpha/980217-1.c (main): Fix implicit int. * gcc.target/alpha/pr19518.c (_mm_setzero_si64): New function. Tested on alphaev68-linux-gnu and committed to mainline SVN. Uros. Index: gcc.target/alpha/980217-1.c

Re: Move loop peeling from RTL to gimple

2014-10-24 Thread Uros Bizjak
Hello! Hi, this is update of my 2013 update to 2012 patch to move rtl loop peeling to tree level. This is to expose optimization oppurtunities earlier. Incrementally I think I can also improve profiling to provide a histogram on loop iterations and get more sensible peeling decisions.

Re: [x86] RFA: Use new rtl iterators in ix86_check_avx256_register

2014-10-27 Thread Uros Bizjak
On Sat, Oct 25, 2014 at 11:01 AM, Richard Sandiford rdsandif...@googlemail.com wrote: This is part of a series to remove uses of for_each_rtx from the ports. Tested by making sure there were no code changes for gcc.dg, gcc.c-torture and g++.dg for x86_64-linux-gnu, and also by a boostrap. OK

Re: [x86] RFA: Use new rtl iterators in x86_extended_reg_mentioned_p

2014-10-27 Thread Uros Bizjak
On Sat, Oct 25, 2014 at 11:03 AM, Richard Sandiford rdsandif...@googlemail.com wrote: This is part of a series to remove uses of for_each_rtx from the ports. Tested by making sure there were no code changes for gcc.dg, gcc.c-torture and g++.dg for x86_64-linux-gnu, and also by a boostrap. OK

Re: [x86] RFA: Use new rtl iterators in find_constant

2014-10-27 Thread Uros Bizjak
On Sat, Oct 25, 2014 at 11:07 AM, Richard Sandiford rdsandif...@googlemail.com wrote: This is part of a series to remove uses of for_each_rtx from the ports. It's a bit hard to read, so I've attached a -b version too. Tested by making sure there were no code changes for gcc.dg, gcc.c-torture

Re: [x86] RFA: Use new rtl iterators in ix86_loop_unroll_adjust

2014-10-27 Thread Uros Bizjak
On Sat, Oct 25, 2014 at 11:11 AM, Richard Sandiford rdsandif...@googlemail.com wrote: This is part of a series to remove uses of for_each_rtx from the ports. It looks like this code should be iterating over the pattern rather than the full insn, or do MEMs in notes really count? If we iterate

Re: [PATCH 5/5] add libcc1

2014-10-28 Thread Uros Bizjak
This patch has now been committed. Also breaks bootstap on x86_64-linux-gnu, CentOS 5.11: gmake[4]: Entering directory `/home/uros/gcc-build/libcc1' /bin/sh ./libtool --tag=CXX --mode=compile g++ -DHAVE_CONFIG_H -I. -I../../gcc-svn/trunk/libcc1 -I ../../gcc-svn/trunk/libcc1/../include -I

Re: [PATCH 5/5] add libcc1

2014-10-28 Thread Uros Bizjak
On Tue, Oct 28, 2014 at 9:50 AM, Phil Muldoon pmuld...@redhat.com wrote: On 28/10/14 08:36, Uros Bizjak wrote: This patch has now been committed. Also breaks bootstap on x86_64-linux-gnu, CentOS 5.11: gmake[4]: Entering directory `/home/uros/gcc-build/libcc1' /bin/sh ./libtool --tag=CXX

Re: [PATCH 5/5] add libcc1

2014-10-28 Thread Uros Bizjak
On Tue, Oct 28, 2014 at 10:35 AM, Jakub Jelinek ja...@redhat.com wrote: On Tue, Oct 28, 2014 at 09:36:45AM +0100, Uros Bizjak wrote: This patch has now been committed. Also breaks bootstap on x86_64-linux-gnu, CentOS 5.11: For -Werror, I'd think that should fix that, WARN_FLAGS should

Re: [PATCH 5/5] add libcc1

2014-10-28 Thread Uros Bizjak
On Tue, Oct 28, 2014 at 11:35 AM, Phil Muldoon pmuld...@redhat.com wrote: This patch has now been committed. Also breaks bootstap on x86_64-linux-gnu, CentOS 5.11: For -Werror, I'd think that should fix that, WARN_FLAGS should already contain -Werror during stage2/stage3 unless

Re: [PATCH 1/X, i386, PR54232] Enable EBX for x86 in 32bits PIC code

2014-10-29 Thread Uros Bizjak
On Wed, Oct 29, 2014 at 1:28 PM, Evgeny Stupachenko evstu...@gmail.com wrote: The test passes now. So let's remove xfail. 2014-10-29 Evgeny Stupachenko evstu...@gmail.com gcc/testsuite * gcc.target/i386/pr23098.c: Remove xfail. OK. Thanks, Uros.

Re: [PATCH x86] Increase PARAM_MAX_COMPLETELY_PEELED_INSNS when branch is costly

2014-10-30 Thread Uros Bizjak
On Tue, Oct 28, 2014 at 1:07 PM, Evgeny Stupachenko evstu...@gmail.com wrote: make check for gcc passed On Mon, Oct 27, 2014 at 11:10 AM, Evgeny Stupachenko evstu...@gmail.com wrote: The results are the same for Silvermont. There are no significant changes on Haswell. So I agree with

Re: [Ping ^ 3] [PATCH] Fix PR 61225

2014-10-30 Thread Uros Bizjak
Hello! Ping? Thanks! -Zhenqiang On 9 June 2014 17:08, Zhenqiang Chen zhenqiang.c...@linaro.org wrote: Ping ^2? Thanks! -Zhenqiang On 28 May 2014 15:02, Zhenqiang Chen zhenqiang.c...@linaro.org wrote: Ping? Thanks! -Zhenqiang On 22 May 2014 17:52, Zhenqiang Chen

Re: [PATCH] x86: extend vect-args testcase to AVX flavors

2014-10-30 Thread Uros Bizjak
On Thu, Oct 30, 2014 at 8:50 AM, Jan Beulich jbeul...@suse.com wrote: gcc/testsuite: 2014-10-30 Jan Beulich jbeul...@suse.com * gcc.target/i386/i386.exp: Extend option set to test vect-args.c with to include -mavx, -mavx2, and -mavx512f. *

[4.9 RFA PATCH, RTL-optimization]: Backport recent AND-alignment alias fixes to 4.9 branch

2014-10-30 Thread Uros Bizjak
were baking for a couple of weeks in the mainline without problems. Modulo removal of old and unnecessary functionality, these changes affect only alpha target. 2014-10-30 Uros Bizjak ubiz...@gmail.com Backport from mainline: 2014-10-20 Uros Bizjak ubiz...@gmail.com * varasm.c

Re: [gofrontend-dev] Re: [PATCH 7/9] Gccgo port to s390[x] -- part I

2014-10-30 Thread Uros Bizjak
Hello! I don't mind skipping the test on other platforms, but xfail is not correct. When an xfail test passes, we get an XPASS error from the testsuite. We need dg-skip-if. I committed this patch. 2014-10-30 Dominik Vogt v...@linux.vnet.ibm.com * gcc.misc-tests/godump-1.c: Skip

Re: [PATCH AVX512] Fix dg.torture tests with avx512

2014-10-31 Thread Uros Bizjak
On Thu, Oct 30, 2014 at 3:55 PM, Ilya Tocar tocarip.in...@gmail.com wrote: Hi, I've run gcc.dg/torture/* tests with -mavx512bw -mavx512vl -mavx512dq flags, and got a bunch of fails (mostly in permutes autogen). Patch below fixes them. Ok for trunk? 2014-10-30 Ilya Tocar

[PATCH, i386]: Workaround PR 63620 by allowing only direct immediate values in FP push patterns

2014-10-31 Thread Uros Bizjak
-materialization issues. OTOH, the patch will generate non-optimal push sequences when general registers are available. 2014-10-31 Uros Bizjak ubiz...@gmail.com PR target/63620 * config/i386/i386.md (*pushtf): Allow only CONST_DOUBLES that won't be reloaded through memory. (*pushxf

<    1   2   3   4   5   6   7   8   9   10   >