From 98e040ea71a2285ea57d956114b0dc5f373e4409 Mon Sep 17 00:00:00 2001
From: Evan Cheng <evan.cheng@apple.com>
Date: Wed, 23 Feb 2011 02:35:33 +0000
Subject: [PATCH] Change VFPNeonA8 definition to make the code easier to read.

llvm-svn: 126298
---
 lib/Target/ARM/ARMInstrFormats.td | 2 +-
 lib/Target/ARM/NEONMoveFix.cpp    | 9 ++-------
 2 files changed, 3 insertions(+), 8 deletions(-)

diff --git a/lib/Target/ARM/ARMInstrFormats.td b/lib/Target/ARM/ARMInstrFormats.td
index b0cec4b1c37..359ac45cee1 100644
--- a/lib/Target/ARM/ARMInstrFormats.td
+++ b/lib/Target/ARM/ARMInstrFormats.td
@@ -134,7 +134,7 @@ def GenericDomain : Domain<0>;
 def VFPDomain     : Domain<1>; // Instructions in VFP domain only
 def NeonDomain    : Domain<2>; // Instructions in Neon domain only
 def VFPNeonDomain : Domain<3>; // Instructions in both VFP & Neon domains
-def VFPNeonA8Domain : Domain<7>; // Instructions in VFP & Neon under A8
+def VFPNeonA8Domain : Domain<5>; // Instructions in VFP & Neon under A8
 
 //===----------------------------------------------------------------------===//
 // ARM special operands.
diff --git a/lib/Target/ARM/NEONMoveFix.cpp b/lib/Target/ARM/NEONMoveFix.cpp
index e8393c53dd8..965665c2821 100644
--- a/lib/Target/ARM/NEONMoveFix.cpp
+++ b/lib/Target/ARM/NEONMoveFix.cpp
@@ -45,13 +45,8 @@ namespace {
 }
 
 static bool inNEONDomain(unsigned Domain, bool isA8) {
-  if (Domain & ARMII::DomainNEON) {
-    // Some instructions only go down NEON pipeline when executed on CortexA8.
-    if (Domain & ARMII::DomainNEONA8)
-      return isA8;
-    return true;
-  }
-  return false;
+  return (Domain & ARMII::DomainNEON) ||
+    (isA8 && (Domain & ARMII::DomainNEONA8));
 }
 
 bool NEONMoveFixPass::InsertMoves(MachineBasicBlock &MBB) {