Merge lp:~ams-codesourcery/gcc-linaro/neon-not64-4.7 into lp:gcc-linaro/4.7

Proposed by Andrew Stubbs
Status: Merged
Approved by: Andrew Stubbs
Approved revision: not available
Merge reported by: Andrew Stubbs
Merged at revision: not available
Proposed branch: lp:~ams-codesourcery/gcc-linaro/neon-not64-4.7
Merge into: lp:gcc-linaro/4.7
To merge this branch: bzr merge lp:~ams-codesourcery/gcc-linaro/neon-not64-4.7
Reviewer Review Type Date Requested Status
Andrew Stubbs (community) Approve
Michael Hope Pending
Review via email: mp+100019@code.launchpad.net

This proposal supersedes a proposal from 2012-03-28.

Description of the change

Implement NEON DImode not operator.

UPDATE 20120328: Patch now reworked with new attributes.

New patch posted upstream here:
http://gcc.gnu.org/ml/gcc-patches/2012-03/msg01751.html

UPDATE 20120329: Mark VMVN as not predicable.

To post a comment you must log in.
Revision history for this message
Michael Hope (michaelh1) wrote : Posted in a previous version of this proposal

cbuild has taken a snapshot of this branch at r114970 and queued it for build.

The diff against the ancestor r114969 is available at:
 http://builds.linaro.org/toolchain/snapshots/gcc-linaro-4.7+bzr114970~ams-codesourcery~neon-not64-4.7.diff

and will be built on the following builders:
 a9-builder armv5-builder i686 x86_64

You can track the build queue at:
 http://ex.seabright.co.nz/helpers/scheduler

cbuild-snapshot: gcc-linaro-4.7+bzr114970~ams-codesourcery~neon-not64-4.7
cbuild-ancestor: lp:gcc-linaro/4.7+bzr114969
cbuild-state: check

Revision history for this message
Michael Hope (michaelh1) wrote : Posted in a previous version of this proposal
Revision history for this message
Michael Hope (michaelh1) wrote : Posted in a previous version of this proposal
Revision history for this message
Michael Hope (michaelh1) wrote : Posted in a previous version of this proposal
Revision history for this message
Michael Hope (michaelh1) wrote : Posted in a previous version of this proposal
Revision history for this message
Ulrich Weigand (uweigand) wrote : Posted in a previous version of this proposal

What's the upstream status? It looks like Richard requested some changes ...

Revision history for this message
Andrew Stubbs (ams-codesourcery) wrote : Posted in a previous version of this proposal

Yes, this patch tests out ok, but it's not been approved upstream.

I need to either change it or show why it's already correct ....

review: Needs Fixing
Revision history for this message
Michael Hope (michaelh1) wrote : Posted in a previous version of this proposal

cbuild has taken a snapshot of this branch at r114971 and queued it for build.

The diff against the ancestor r114969 is available at:
 http://builds.linaro.org/toolchain/snapshots/gcc-linaro-4.7+bzr114971~ams-codesourcery~neon-not64-4.7.diff

and will be built on the following builders:
 a9-builder armv5-builder i686 x86_64

You can track the build queue at:
 http://ex.seabright.co.nz/helpers/scheduler

cbuild-snapshot: gcc-linaro-4.7+bzr114971~ams-codesourcery~neon-not64-4.7
cbuild-ancestor: lp:gcc-linaro/4.7+bzr114969
cbuild-state: check

Revision history for this message
Michael Hope (michaelh1) wrote : Posted in a previous version of this proposal
Revision history for this message
Michael Hope (michaelh1) wrote : Posted in a previous version of this proposal
Revision history for this message
Michael Hope (michaelh1) wrote : Posted in a previous version of this proposal

cbuild had trouble building this on armv7l-natty-cbuild280-tcpanda06-cortexa9r1.

See the *failed.txt logs under the build results at:
 http://ex.seabright.co.nz/build/gcc-linaro-4.7+bzr114971~ams-codesourcery~neon-not64-4.7/logs/armv7l-natty-cbuild280-tcpanda06-cortexa9r1

The test suite results were not checked.

cbuild-checked: armv7l-natty-cbuild280-tcpanda06-cortexa9r1

review: Needs Fixing
Revision history for this message
Michael Hope (michaelh1) wrote :

cbuild has taken a snapshot of this branch at r114972 and queued it for build.

The diff against the ancestor r114969 is available at:
 http://builds.linaro.org/toolchain/snapshots/gcc-linaro-4.7+bzr114972~ams-codesourcery~neon-not64-4.7.diff

and will be built on the following builders:
 a9-builder armv5-builder i686 x86_64

You can track the build queue at:
 http://ex.seabright.co.nz/helpers/scheduler

cbuild-snapshot: gcc-linaro-4.7+bzr114972~ams-codesourcery~neon-not64-4.7
cbuild-ancestor: lp:gcc-linaro/4.7+bzr114969
cbuild-state: check

Revision history for this message
Michael Hope (michaelh1) wrote :
Revision history for this message
Michael Hope (michaelh1) wrote :
Revision history for this message
Michael Hope (michaelh1) wrote :
Revision history for this message
Michael Hope (michaelh1) wrote :
Revision history for this message
Andrew Stubbs (ams-codesourcery) wrote :

This is now approved upstream.

review: Approve
Revision history for this message
Ramana Radhakrishnan (ramana) wrote :

+1 test results look ok to me.

ramana

Subscribers

People subscribed via source and target branches