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

Unified Diff: unittest/AssemblerX8632/X87.cpp

Issue 1419903002: Subzero: Refactor x86 register definitions to use the alias mechanism. (Closed) Base URL: https://chromium.googlesource.com/native_client/pnacl-subzero.git@master
Patch Set: Fix assembler unit tests. Fix register names. Code review changes. Rebase Created 5 years, 2 months 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
« no previous file with comments | « unittest/AssemblerX8632/TestUtil.h ('k') | unittest/AssemblerX8664/GPRArith.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: unittest/AssemblerX8632/X87.cpp
diff --git a/unittest/AssemblerX8632/X87.cpp b/unittest/AssemblerX8632/X87.cpp
index ff36f7e43b550cb7971abff42217c7f918467668..79a2d457e75d44753e0efb3f0ff8d77eac6f29a0 100644
--- a/unittest/AssemblerX8632/X87.cpp
+++ b/unittest/AssemblerX8632/X87.cpp
@@ -14,8 +14,8 @@ namespace Test {
namespace {
TEST_F(AssemblerX8632LowLevelTest, Fld) {
- __ fld(IceType_f32, Address(GPRRegister::Encoded_Reg_ebp, 1,
- AssemblerFixup::NoFixup));
+ __ fld(IceType_f32,
+ Address(GPRRegister::Encoded_Reg_ebp, 1, AssemblerFixup::NoFixup));
__ fld(IceType_f64, Address(GPRRegister::Encoded_Reg_ebp, 0x10000,
AssemblerFixup::NoFixup));
@@ -33,8 +33,8 @@ TEST_F(AssemblerX8632LowLevelTest, Fld) {
}
TEST_F(AssemblerX8632LowLevelTest, FstpAddr) {
- __ fstp(IceType_f32, Address(GPRRegister::Encoded_Reg_ebp, 1,
- AssemblerFixup::NoFixup));
+ __ fstp(IceType_f32,
+ Address(GPRRegister::Encoded_Reg_ebp, 1, AssemblerFixup::NoFixup));
__ fstp(IceType_f64, Address(GPRRegister::Encoded_Reg_ebp, 0x10000,
AssemblerFixup::NoFixup));
@@ -61,8 +61,8 @@ TEST_F(AssemblerX8632LowLevelTest, Fincstp) {
}
TEST_F(AssemblerX8632LowLevelTest, FnstcwAddr) {
- __ fnstcw(Address(GPRRegister::Encoded_Reg_ebp, 0x12345,
- AssemblerFixup::NoFixup));
+ __ fnstcw(
+ Address(GPRRegister::Encoded_Reg_ebp, 0x12345, AssemblerFixup::NoFixup));
constexpr size_t ByteCount = 6;
ASSERT_EQ(ByteCount, codeBytesSize());
@@ -74,8 +74,8 @@ TEST_F(AssemblerX8632LowLevelTest, FnstcwAddr) {
}
TEST_F(AssemblerX8632LowLevelTest, FldcwAddr) {
- __ fldcw(Address(GPRRegister::Encoded_Reg_ebp, 0x12345,
- AssemblerFixup::NoFixup));
+ __ fldcw(
+ Address(GPRRegister::Encoded_Reg_ebp, 0x12345, AssemblerFixup::NoFixup));
constexpr size_t ByteCount = 6;
ASSERT_EQ(ByteCount, codeBytesSize());
« no previous file with comments | « unittest/AssemblerX8632/TestUtil.h ('k') | unittest/AssemblerX8664/GPRArith.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698