Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(686)

Unified Diff: src/platform-linux.cc

Issue 11428137: ARM: Make use of d16-d31 when available. (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Address more comments Created 8 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: src/platform-linux.cc
diff --git a/src/platform-linux.cc b/src/platform-linux.cc
index ce45babdf042265b5448bcd345ae9cb35c85810b..538fa86a388e55812c491ce1b8679e17dd4d303c 100644
--- a/src/platform-linux.cc
+++ b/src/platform-linux.cc
@@ -156,10 +156,17 @@ bool OS::ArmCpuHasFeature(CpuFeature feature) {
case SUDIV:
search_string = "idiva";
break;
+ case VFP32DREGS:
+ search_string = "vfpv3";
Rodolph Perfetta 2012/12/17 14:38:30 The search_string variable is not used with VFP32D
hans 2012/12/17 15:32:46 Done.
+ break;
default:
UNREACHABLE();
}
+ if (feature == VFP32DREGS) {
+ return CPUInfoContainsString("vfpv3") && !CPUInfoContainsString("d16");
Rodolph Perfetta 2012/12/17 14:38:30 I believe some early devices such as nexus one do
hans 2012/12/17 15:32:46 Done.
+ }
+
if (CPUInfoContainsString(search_string)) {
return true;
}
« src/arm/assembler-arm-inl.h ('K') | « src/mips/assembler-mips.h ('k') | src/serialize.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698