Re: [PATCH] Fix PR53703

On Sun, 17 Jun 2012, William J. Schmidt wrote: > The test case exposes a bug that occurs only when a diamond control flow > pattern has the arguments of the joining phi in a different order from > the successor arcs of the entry block. My logic for setting > bb_for_def[12] was just brain-dead. T

PR 53698: missing address conversion in expand_expr_addr_expr_1

The result of expand_expr_addr_expr_1 is eventually converted to tmode, and any variable offset is likewise applied in tmode, so tmode also seems like the natural mode for applying constant offsets. This patch makes sure that the operand is converted to tmode first, just like we do for variable of

Re: [Patch 4.6] In system.h, wrap include of C++ header in 'extern C++'

On Sun, Jun 17, 2012 at 2:56 PM, Gabriel Dos Reis wrote: > On Sun, Jun 17, 2012 at 6:55 AM, Richard Guenther > wrote: >> On Fri, Jun 15, 2012 at 5:18 PM, Gabriel Dos Reis >> wrote: >>> On Fri, Jun 15, 2012 at 10:13 AM, Duncan Sands wrote: Hi Gabriel, >>> Richard just reminded me t

Re: PR 53698: missing address conversion in expand_expr_addr_expr_1

On Mon, Jun 18, 2012 at 11:18 AM, Richard Sandiford wrote: > The result of expand_expr_addr_expr_1 is eventually converted to tmode, > and any variable offset is likewise applied in tmode, so tmode also seems > like the natural mode for applying constant offsets.  This patch makes sure > that the

Re: [Patch 4.6] In system.h, wrap include of C++ header in 'extern C++'

On Mon, Jun 18, 2012 at 4:18 AM, Richard Guenther wrote: > On Sun, Jun 17, 2012 at 2:56 PM, Gabriel Dos Reis > wrote: >> On Sun, Jun 17, 2012 at 6:55 AM, Richard Guenther >> wrote: >>> On Fri, Jun 15, 2012 at 5:18 PM, Gabriel Dos Reis >>> wrote: On Fri, Jun 15, 2012 at 10:13 AM, Duncan San

[PING ARM Patches] PR53447: optimizations of 64bit ALU operation with constant

Hi Could ARM maintainers review following patches? http://gcc.gnu.org/ml/gcc-patches/2012-06/msg00497.html 64bit add/sub constants. http://gcc.gnu.org/ml/gcc-patches/2012-05/msg01834.html 64bit and with constants. http://gcc.gnu.org/ml/gcc-patches/2012-05/msg01974.html 64bit xor with constants.

[patch] Remove NO_IMPLICIT_EXTERN_C target macro

Hello, For targets that do not define NO_IMPLICIT_EXTERN_C, GCC assumes that included system headers are implicitly wrapped in 'extern "C" {...}'. I could not find where this target macro was introduced, but it is very old (it is already mentioned in FSFChangeLog.10 for a change in 1995). AFAICT,

Re: [PATCH] GCC Ada/GNAT configuration for GNU/Hurd

> Here is a patch that adds a basic GCC Ada/GNAT configuration for x86 > GNU/Hurd. This has originally been created by Svante Signell for use > with Debian's GCC 4.6 package, and now I have ported it to trunk, and > also applied some further changes. > > 2012-06-16 Svante Signell > T

Re: [PATCH] GCC Ada/GNAT configuration for GNU/Hurd

Hi! On Mon, 18 Jun 2012 12:23:47 +0200, Arnaud Charlet wrote: > > Here is a patch that adds a basic GCC Ada/GNAT configuration for x86 > > GNU/Hurd. This has originally been created by Svante Signell for use > > with Debian's GCC 4.6 package, and now I have ported it to trunk, and > > also appli

Re: [PATCH] GCC Ada/GNAT configuration for GNU/Hurd

> > Another comment: > > > > > --- a/gcc/ada/s-oscons-tmplt.c > > > +++ b/gcc/ada/s-oscons-tmplt.c > > > @@ -80,7 +80,7 @@ pragma Style_Checks ("M32766"); > > > > > > /* Feature macro definitions */ > > > > > > -#if defined (__linux__) && !defined (_XOPEN_SOURCE) > > > +#if (defined (__linux_

[PATCH resend] Add implicit C linkage for win32-specific entry points

gcc/ChangeLog: * config/i386/mingw-w64.h: Specify entry points with implicit C linkage gcc/cp/ChangeLog: * decl.c: Allow custom target implicit C linkage gcc/testsuite/ChangeLog: * g++.dg/ext/main.C: Added implicit C linkage tests --- gcc/config/i386/mingw-w64.h |

[patch] Remove VMS_TARGET

Hello, There are exactly 2 references to the target macro VMS_TARGET: cp/decl.c:#if VMS_TARGET config/vax/vax.h:#define VMS_TARGET 0 I'm not sure what the macro is for. GCC seems to support a couple of VMS targets, but none of them define VMS_TARGET. This patch remove the macro. I can't test th

Re: [PATCH resend] Add implicit C linkage for win32-specific entry points

On Mon, Jun 18, 2012 at 12:54 PM, Jacek Caban wrote: > > gcc/ChangeLog: >        * config/i386/mingw-w64.h: Specify entry points with implicit C linkage > > gcc/cp/ChangeLog: >        * decl.c: Allow custom target implicit C linkage > > gcc/testsuite/ChangeLog: >        * g++.dg/ext/main.C: Added

Re: [PATCH][4/n] VRP and anti-range handling

On Fri, 15 Jun 2012, Richard Guenther wrote: > > This tries to completely implement the intersect primitive for > VRP (what extract_range_from_assert does at its end when merging > new and old knowledge). > > Bootstrap and regtest pending on x86_64-unknown-linux-gnu. > > I plan to re-organize v

Re: [patch] Remove NO_IMPLICIT_EXTERN_C target macro

On Mon, 18 Jun 2012, Steven Bosscher wrote: > For targets that do not define NO_IMPLICIT_EXTERN_C, GCC assumes that > included system headers are implicitly wrapped in 'extern "C" {...}'. > I could not find where this target macro was introduced, but it is > very old (it is already mentioned in FS

Re: [patch] Remove VMS_TARGET

On Mon, 18 Jun 2012, Steven Bosscher wrote: > gcc/ > * config/vax/vax.h (VMS_TARGET): Remove. > > cp/ > * decl.c (finish_function): Remove code conditional on VMS_TARGET. OK. Both C and C++ specify an implicit return of 0 from main; it's for the C runtime to convert return valu

Re: [patch] Remove VMS_TARGET

On Jun 18, 2012, at 1:01 PM, Steven Bosscher wrote: > Hello, > > There are exactly 2 references to the target macro VMS_TARGET: > > cp/decl.c:#if VMS_TARGET > config/vax/vax.h:#define VMS_TARGET 0 > > I'm not sure what the macro is for. GCC seems to support a couple of > VMS targets, but none

[Patch] Adjustments for Windows x64 SEH

Hi, at AdaCore, we are now using native SEH exceptions for Windows x64 based on Richard's patch http://gcc.gnu.org/ml/gcc-patches/2010-10/msg01966.html I plan to submit a rebased version of this work soon, but we have found a few issues in the port: * Registers cannot be pushed once the fram

[PATCH] Fix PR53693

When the vectorizer detects a widening shift pattern it has to make sure the widening operation is used only once as it will make it part of the recognized pattern. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk. Richard. 2012-06-18 Richard Guenther PR tree-op

Re: [patch] Remove NO_IMPLICIT_EXTERN_C target macro

On Mon, Jun 18, 2012 at 1:12 PM, Joseph S. Myers wrote: > On Mon, 18 Jun 2012, Steven Bosscher wrote: > >> For targets that do not define NO_IMPLICIT_EXTERN_C, GCC assumes that >> included system headers are implicitly wrapped in 'extern "C" {...}'. >> I could not find where this target macro was

[RFC C++ / PR51033 ] Handle __builtin_shuffle in constexpr properly in the C++ frontend.

Hi, This patch following on from the fix for turning on __builtin_shuffle for c++ , enables folding of vec_perm_exprs in the front-end for constexpr and constructor style values. I was originally going to go with Marc's patch but then took a look at what it would take to support this properly whi

Re: [RFC C++] Turn on builtin_shuffle for C++.

On 15 June 2012 20:04, Marc Glisse wrote: > On Fri, 15 Jun 2012, Ramana Radhakrishnan wrote: > >> On 15 June 2012 18:18, Marc Glisse wrote: >>> >>> On Fri, 15 Jun 2012, Ramana Radhakrishnan wrote: >>> > I just noticed this part. Rereading my comment in > > http://gcc.gnu.org/bugzilla/

[Patch] Don't test for pr53425 on mingw

Hi, I am told that this ABI test does not apply to mingw targets. OK to apply? Index: gcc/testsuite/gcc.target/i386/pr53425-1.c === --- gcc/testsuite/gcc.target/i386/pr53425-1.c (revision 188384) +++ gcc/testsuite/gcc.target/i386/pr

Re: [PATCH][4/n] VRP and anti-range handling

Hi, Before revision 188728, the test gcc.dg/tree-ssa/vrp68.c failed because link_error appeared twice in the dump file. After r188728 it fails also at link time and link_error appears three time: FAIL: gcc.dg/tree-ssa/vrp68.c (test for excess errors) Excess errors: Undefined symbols: "_link_err

Re: [Patch] Don't test for pr53425 on mingw

2012/6/18 JonY : > Hi, > > I am told that this ABI test does not apply to mingw targets. OK to apply? Hi JonY, The test doesn't apply to x64 windows targets, as for it sse is part of its ABI. As test already checks for !ia32, we could simply check for x86_64/i?86-*-mingw* targets instead. We don

Re: [Patch] Adjustments for Windows x64 SEH

Hello Tristan, patch works for me, too. Just one nit about the patch. 2012/6/18 Tristan Gingold : > @@ -8558,6 +8558,11 @@ ix86_frame_pointer_required (void) >   if (TARGET_32BIT_MS_ABI && cfun->calls_setjmp) >     return true; > > +  /* Win64 SEH, very large frames need a frame-pointer as maximu

Re: [4.6][ARM] Backport fix PR48126

On 18/06/12 04:11, Joey Ye wrote: > OK for 4.6? > > 2012-06-18 Joey Ye > > Backport from mainline > 2011-10-14 David Alan Gilbert > > PR target/48126 > * config/arm/arm.c (arm_output_sync_loop): Move label before > barrier. > > > Index: gcc/config/arm/arm.

RFA: Fix PR53688

Hi, now that we regard MEM_EXPR as a conservative approximation for MEM_SIZE (and MEM_OFFSET) we must ensure that this is really the case. It isn't currently for the string expanders, as they use the MEM_REF (whose address was taken) directly as the one to use for MEM_EXPR on the MEM rtx. Tha

Re: [PATCH] ARM: exclude fixed_regs for stack-alignment save/restore

On 16/06/12 13:42, Richard Sandiford wrote: > Roland McGrath writes: >> On Thu, Jun 14, 2012 at 1:13 PM, Mike Stump wrote: >>> On Jun 14, 2012, at 10:16 AM, Roland McGrath wrote: But if e.g. I use -ffixed-r9 then I think it's a reasonable expectation that no code is generated that touch

[arm] Remove obsolete FPA support (6/n): fix warnings

[I composed this message on Friday, before I left for the weekend, but somehow forgot to press send. The good news is that as a result I can now confirm that this has fixed the outstanding bootstrap issue.] This patch cleans up some build warnings that break native bootstrap. One of the warnings

[Patch, Testsuite, ARM] Improve test gcc.target/arm/handler-align.c

This test checks that the stack pointer is handled correctly in prologue/epilogue of Cortex-M interrupt handlers. An interrupt handler may be called when stack is not double-word aligned. The prologue of the interrupt handler aligns the stack pointer and the epilogue restores the original stack poi

[patch, testsuite] new test for arm epilogue

This test relies on epilogue generated in RTL to provide register liveness information that enables peephole optimization. OK for trunk? Thanks, Greta 2012-06-18 Joey Ye Greta Yorsh * gcc.target/arm/epilog-1.c: New test. diff --git a/gcc/testsuite/gcc.target/arm/epilog

Re: [PATCH 2/3] Add XLP-specific atomic instructions and tweaks.

On 2012-06-17 13:01, Richard Sandiford wrote: >> > Not required if you use the proper predicate in the expander. >> > The middle-end will take care of this for you. > I might be misunderstanding, sorry, but this expander is shared with > the normal LL/SC path, which can accept plain memory_operands

[patch] Use IDENTIFIER_LENGTH instead of strlen(IDENTIFIER_POINTER) in a few places

Hello, Obvious enough that I'll commit this a week or so from now, unless someone objects. Ciao! Steven gcc/ * tree-emultls.c (prefix_name): Use IDENTIFIER_LENGTH instead of strlen(IDENTIFIER_POINTER). objc/ * objc-encoding.c (encode_aggregate_fields): Use IDENTIFIER_LEN

RE: [Patch, ARM][0/8] Epilogue in RTL: introduction (Sameera's patches, Part I)

Paul, I did additional testing of the patches, as you suggested. For iwmmxt, no regression on qemu (using -cpu pxa270) for arm-none-eabi taget configured --with-cpu iwmmxt --with-float soft --with-arch iwmmxt --with-abi iwmmxt --disable-multilib. There is already a test for mmx stack alignment

Re: [PATCH] ARM: exclude fixed_regs for stack-alignment save/restore

OK then. If you like the original patch, would you like to commit it for me? Thanks, Roland

[PATCH] backport darwin12 fixes to gcc-4_7-branch

The attached patch backports... http://gcc.gnu.org/ml/gcc-patches/2012-05/msg01710.html http://gcc.gnu.org/ml/gcc-patches/2012-05/msg01707.html which implement changes for darwin12 and later. Bootstrap and regression tested on current x86_64-apple-darwin12. http://gcc.gnu.org/ml/gcc-testresult

Re: [Patch, Testsuite, ARM] Improve test gcc.target/arm/handler-align.c

On 18/06/12 16:39, Greta Yorsh wrote: > This test checks that the stack pointer is handled correctly in > prologue/epilogue of Cortex-M interrupt handlers. An interrupt handler may > be called when stack is not double-word aligned. The prologue of the > interrupt handler aligns the stack pointer an

Re: [patch, testsuite] new test for arm epilogue

On 18/06/12 16:40, Greta Yorsh wrote: > This test relies on epilogue generated in RTL to provide register liveness > information that enables peephole optimization. > > OK for trunk? > > Thanks, > Greta > > > 2012-06-18 Joey Ye > Greta Yorsh > > * gcc.target/arm/epilog

Re: *PING**2 [Patch, Fortran] Reject coarrays in MOVE_ALLOC

Dear Tobias, Good catch on the non-polymorphic 'TO'! >> Updated version (cf. below). Build and regtested on x86-64-linux. >> OK for the trunk? Indeed - OK for trunk. >> >> I asked at J3 and John Reid kindly pointed me to the pending >> interpretation request F08/0040 at >> http://j3-fortran.or

[patch] Fix PR48109 using artificial top-level asm statements (darwin/objc)

Hello, This patch started as an attempt to remove #include "output.h" from objc/: Instead of writing references directly to asm_out_file, the references are output as top-level asm statements. It's a bit of a hack, but it works and it's a "better hack" than writing to asm_out_file from a front end

Re: **PING**2 [Patch, Fortran] PR53526 - Fix MOVE_ALLOC for coarrays

Dear Tobias, >> OK for the trunk? Yes indeed - thanks for the patch. Cheers Paul

Re: [Patch, Fortran] PR53692 - Fix passing an absent array to an elemental procedure

Dear Tobias, > OK for the trunk? Yes, particularly with the development relative to the draft patch in the PR. OK for trunk. Thanks Paul

Re: [Patch, Fortran] Implement RANK

Hi Tobias, > > Build and regtested on x86-64-gnu-linux. > OK for the trunk? Yes, of course it's OK for trunk. It verges on obvious! Thanks Paul

[PATCH, i386]: Fix PR 53712, Does not combine unaligned load with _mm_cmpistri

Hello! Attached patch teaches gcc how to merge unaligned load (UNSPEC_MOVU) with pcmpistr/pcmpestr instructions. 2012-06-18 Uros Bizjak PR target/53712 * config/i386/sse.md (*sse4_2_pcmpestr_unaligned): New. (*sse4_2_pcmpistr_unaligned): New. testsuite/ChangeLog: 201

Re: [patch] Remove NO_IMPLICIT_EXTERN_C target macro

On Mon, Jun 18, 2012 at 7:12 AM, Joseph S. Myers wrote: > On Mon, 18 Jun 2012, Steven Bosscher wrote: > >> For targets that do not define NO_IMPLICIT_EXTERN_C, GCC assumes that >> included system headers are implicitly wrapped in 'extern "C" {...}'. >> I could not find where this target macro was

gcc4.7 backport committed (PR53700)

The following patch was backported to gcc4.7. The patch solves PR53700. It was successfully bootstrapped on x86/x86-64. 2012-06-18 Vladimir Makarov PR rtl-optimization/53700 Backport from mainline 2012-03-29 Vladimir Makarov * ira-color.c (setup_left_confl

Re: [PATCH] boehm-gc configuration support for GNU/Hurd

> "Thomas" == Thomas Schwinge writes: Thomas> This is a backport of upstream boehm-gc configuration support for Thomas> GNU/Hurd. The equivalent thing has been used in Debian's GCC packages Thomas> since GCC 4.2 already, and I decided that *NOW* ought to be the time to Thomas> put it into th

Re: [PATCH] Add vector cost model density heuristic

On Mon, 2012-06-11 at 13:40 +0200, Richard Guenther wrote: > On Fri, 8 Jun 2012, William J. Schmidt wrote: > > > Hmm. I don't like this patch or its general idea too much. Instead > I'd like us to move more of the cost model detail to the target, giving > it a chance to look at the whole loop

Re: [PATCH] Add vector cost model density heuristic

On Mon, 2012-06-18 at 13:49 -0500, William J. Schmidt wrote: > On Mon, 2012-06-11 at 13:40 +0200, Richard Guenther wrote: > > On Fri, 8 Jun 2012, William J. Schmidt wrote: > > > > > > > Hmm. I don't like this patch or its general idea too much. Instead > > I'd like us to move more of the cost

Re: [PATCH] GCC Ada/GNAT configuration for GNU/Hurd

On Mon, 2012-06-18 at 12:23 +0200, Arnaud Charlet wrote: > > Here is a patch that adds a basic GCC Ada/GNAT configuration for x86 > > GNU/Hurd. This has originally been created by Svante Signell for use > > with Debian's GCC 4.6 package, and now I have ported it to trunk, and > > also applied some

Re: [PATCH] GCC Ada/GNAT configuration for GNU/Hurd

> Hi, I tried with s-taprop-posix.adb earlier (among many tries), but > GNU/Hurd does not seem to be fully POSIX pthread compliant yet, > unfortunately. At least to my findings. Well, would be good to understnad in more details what's missing. Some trivial missing wrappers are better put in s-osi

[PATCH] Bad code generation: incorrect folding of TARGET_MEM_REF into a constant

Hi all, I have tracked down a bug which results in invalid code being generated for indexed TARGET_MEM_REF expressions during dominator optimization. The conditions are: accessing objects adjacent in memory in a loop (in order to generate the TARGET_MEM_REF gimple) and optimizing this tree item d

Re: [PATCH, testsuite]: Fix scan-tree-dump-times argument order in gcc.dg/tree-ssa/vrp68.c.

On 06/17/2012 05:03 AM, Richard Guenther wrote: > On Sun, Jun 17, 2012 at 10:41 AM, Uros Bizjak wrote: >> Hello! >> >> The testcase still fails on x86_64-pc-linux-gnu with: >> >> FAIL: gcc.dg/tree-ssa/vrp68.c scan-tree-dump-times vrp1 "link_error" 1 >> >> since there are two calls to link_error. >

Re: [PATCH 3/3] Handle const_vector in mulv4si3 for pre-sse4.1.

On 2012-06-17 11:37, Uros Bizjak wrote: > Hello! > > Please note that you will probably hit PR33329, this is the reason > that we expand multiplications after reload. Please see [1] for > further explanation. There is gcc.target/i386/pr33329.c test to cover > this issue, but it is not effective an

[PATCH, libcpp]: Use x86 __builtin_ia32_pcmpestri128 instead of asm.

Hello! Following the patch that allows unaligned operands in pcmpestri [1], we can substitute x86 asm in lex.c with equivalent builtin functions. 2011-06-18 Uros Bizjak * lex.c (search_line_sse42): Use __builtin_ia32_loaddqu and __builtin_ia32_pcmpestri128 instead of asm. Boo

Re: [RFA] dwarf2.def (DW_OP): Add DW_OP_GNU_const_index.

Ping. On Thu, Jun 14, 2012 at 12:48 PM, Doug Evans wrote: > Hi. > > Ok to commit? > > 2012-06-14  Doug Evans   > >        * dwarf2.def (DW_OP): Add DW_OP_GNU_const_index. > > Index: dwarf2.def > === > --- dwarf2.def  (revision 188634

[PATCH 4/4] - Merging gdc (GNU D Compiler) into gcc

[PATCH 4/4]: Add D frontend and library to build - configure - configure.ac - Makefile.def - Makefile.in Add mention of D compiler and Phobos in documentation - gcc/doc/frontends.texi - gcc/doc/install.texi - gcc/doc/invoke.texi - gcc/doc/sourcebuild.texi - gcc/doc/stand

[PATCH, gdc] - Merging gdc (GNU D Compiler) into gcc

Hi, Seems that I can't upload the files individually due to a size limit, so I've put the files up on my site. Sorry for any initial noise. Long story short: So I have been working on this for quite some time, and now I feel it is approaching some state of readiness for inclusion. Any feedback

fix libstdc++/53678 - namespace pollution on NetBSD

This fixes a regression on NetBSD. 2012-06-18 Jörg Sonnenberger Jonathan Wakely PR libstdc++/53678 * config/os/bsd/netbsd/ctype_base.h: Check for _CTYPE_U. * testsuite/22_locale/ctype_base/53678.cc: New. Tested x86_64-unknown-netbsd5.1, committed to trunk,

Re: [RFA] dwarf2.def (DW_OP): Add DW_OP_GNU_const_index.

On 2012-06-14 12:48, Doug Evans wrote: > Hi. > > Ok to commit? > > 2012-06-14 Doug Evans > > * dwarf2.def (DW_OP): Add DW_OP_GNU_const_index. Ok. r~

Re: [PATCH, libcpp]: Use x86 __builtin_ia32_pcmpestri128 instead of asm.

On 2012-06-18 13:19, Uros Bizjak wrote: >/* ??? The builtin doesn't understand that the PCMPESTRI read from >memory need not be aligned. */ > - __asm ("%vpcmpestri $0, (%1), %2" > - : "=c"(index) : "r"(s), "x"(search), "a"(4), "d"(16)); > + sv = __builtin_ia32_lo

Re: [wwwdocs] Update coding conventions for C++

This patch updates the coding conventions to C++. The primary source file is codingconventions.html. The coding conventions now refer to a new file, codingrationale.html, providing the rationale for some of the C++ conventions. The two files in question are attached whole for your convenience.

Reorganized documentation for warnings -- attempt 2

Some friendly users in the gcc irc chat informed me that I may be more likely to get a review for my patch if it weren't a 100 KiB file. This is my attempt at making the patch a little easier to read. I'm not sure how important it is for documentation, but I have already sent in my copyright assig

Re: [PATCH] Improve pattern recognizer for division by constant (PR tree-optimization/51581)

On 2012-06-14 13:58, Jakub Jelinek wrote: > + if (!supportable_widening_operation (WIDEN_MULT_EXPR, last_stmt, > +vecwtype, vectype, > +&dummy, &dummy, &dummy_code, > +&dummy_code, &dummy_in

[testsuite] profopt.exp and friends: use expected list of options

There are tests in g++.tree-prof that have non-unique lines in test summaries for scan-*-dump checks. Investigation showed that these tests were being run multiple times, for a list of options that had leaked over from another set of profile-directed optimization tests. This patch doesn't fix dup

Re: Make timevar phases mutually exclusive. (issue6302064)

On 6/13/12, Diego Novillo wrote: > On 12-06-13 08:46 , Diego Novillo wrote: > > The LTO bits are fine. I would prefer if an FE maintainer takes > > a second look over the other bits. Jason, Joseph? > > Incidentally, could you please test it with an LTO-enabled bootstrap? > > $ ../src/configure --w

[4.6][ARM] Backport "MCR Not available in Thumb1"

Backporting trunk r179979 OK for 4.6? Backported from mainline 2011-10-14 David Alan Gilbert PR target/48126 * config/arm/arm.c (arm_output_sync_loop): Move label before barrier. Index: gcc/config/arm/arm.h =

Re: [PATCH, libcpp]: Use x86 __builtin_ia32_pcmpestri128 instead of asm.

On Tue, Jun 19, 2012 at 12:07 AM, Richard Henderson wrote: > On 2012-06-18 13:19, Uros Bizjak wrote: >>        /* ??? The builtin doesn't understand that the PCMPESTRI read from >>        memory need not be aligned.  */ >> -      __asm ("%vpcmpestri $0, (%1), %2" >> -          : "=c"(index) : "r"(

Re: [PATCH 2/3] Add XLP-specific atomic instructions and tweaks.

On 16/06/2012, at 7:45 PM, Richard Sandiford wrote: > Maxim Kuvyrkov writes: >> Updated patch attached. Any further comments? > > It's due to my bad explanation, sorry, but this isn't what I meant. > The two main changes I was looking for were: > > 1) Your pattern uses: > >[(mem:GPR (