Re: [PATCH] Avoid invalid sharing of ADDR_EXPRs (PR fortran/55935)

2013-01-12 Thread Richard Biener
Jakub Jelinek wrote: >Hi! > >As discussed in the PR, the extra verification of location blocks >Richard >posted recently fails on some Fortran testcases. The problem is that >ADDR_EXPRs in static const var initializers contain locations (fixed by >the >trans-expr.c hunks), and that gimple-fold m

[Patch, fortran] PR54286 - [4.8 Regression] Accepts invalid proc-pointer assignments involving proc-ptr function result

2013-01-12 Thread Paul Richard Thomas
Dear All, It is something of an exaggeration to say that this PR is a regession, although it is true that gcc-4.7 gives error messages for the testcase in the correct places. In fact, these messages disappear if IMPLICIT INTEGER (a) at the start of the testcase. The fix ensures that the interfac

[Patch, fortran] PR55868 - [4.8 Regression] gfortran generates for CLASS(*) __m_MOD___vtab__$tar on NO_DOLLAR_IN_LABEL systems

2013-01-12 Thread Paul Richard Thomas
Fix discussed and okayed by Jakub in PR. Patch committed as 'obvious' in revision 195124. Cheers Paul 2013-01-08 Paul Thomas PR fortran/55868 * class.c (get_unique_type_string): Change $tar to STAR and replace sprintf by strcpy where there is no formatting. * decl.c (gfc_matc

[PATCH 1/2] Document HLE / RTM intrinsics

2013-01-12 Thread Andi Kleen
From: Andi Kleen The TSX HLE/RTM intrinsics were missing documentation. Add this to the manual. Ok for release / trunk? 2013-01-11 Andi Kleen * doc/extend.texi: Document __ATOMIC_HLE_ACQUIRE, __ATOMIC_HLE_RELEASE. Document __builtin_ia32 TSX intrincs. Document _x* TS

[PATCH 2/2] Support __ATOMIC_HLE_RELEASE for __atomic_clear/store_n

2013-01-12 Thread Andi Kleen
From: Andi Kleen __atomic_clear and __atomic_store_n didn't have code to generate the TSX HLE RELEASE prefix. Add this plus test cases. Right now it would need another target hook to check for someone passing __ATOMIC_HLE_ACQUIRE to store/clear. I just ignore this for now. Passes bootstrap/test

Re: [PATCH 1/2] Document HLE / RTM intrinsics

2013-01-12 Thread Bernhard Reutner-Fischer
On 12 January 2013 16:28:41 Andi Kleen wrote: From: Andi Kleen +Returns _XBEGIN_STARTED when the transaction +started successfully (not this is not 0, so the constant has to be not this is not 0? Or note? Thanks, Sent with AquaMail for Android http://www.aqua-mail.com

Re: [PATCH 1/2] Document HLE / RTM intrinsics

2013-01-12 Thread Andi Kleen
On Sat, Jan 12, 2013 at 06:04:19PM +0100, Bernhard Reutner-Fischer wrote: > On 12 January 2013 16:28:41 Andi Kleen wrote: > >From: Andi Kleen > > >+Returns _XBEGIN_STARTED when the transaction > >+started successfully (not this is not 0, so the constant has to be > > not this is not 0? Or note?

Re: [Patch, Fortran] PR 55072: [4.6/4.7/4.8 Regression] Missing internal_pack leads to wrong code with derived type

2013-01-12 Thread Janus Weil
> OK by me for trunk, followed by 4.6 and 4.7. Thanks, Paul. Committed to trunk as r195125. Will do the branches soon. Cheers, Janus > On 11 January 2013 21:51, Mikael Morin wrote: >> Le 11/01/2013 21:31, Janus Weil a écrit : >> >>> Thanks for your review (which I read as an OK for all branc

Re: [Patch, fortran] PR54286 - [4.8 Regression] Accepts invalid proc-pointer assignments involving proc-ptr function result

2013-01-12 Thread Janus Weil
Hi Paul, > It is something of an exaggeration to say that this PR is a regession, > although it is true that gcc-4.7 gives error messages for the testcase > in the correct places. In fact, these messages disappear if IMPLICIT > INTEGER (a) at the start of the testcase. > > The fix ensures that th

[PATCH] Add faster HTM fastpath for libitm TSX

2013-01-12 Thread Andi Kleen
From: Andi Kleen The libitm TSX hardware transaction fast path currently does quite a bit of unnecessary work (saving registers etc.) before even trying to start a hardware transaction. This patch moves the initial attempt at a transaction early into the assembler stub. Complicated work like retr