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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: unittest/AssemblerX8664/Locked.cpp

Issue 1616103002: Subzero. X8664. Enables RIP-based addressing mode. (Closed) Base URL: https://chromium.googlesource.com/native_client/pnacl-subzero.git@master
Patch Set: make presubmit happy. Created 4 years, 11 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
« src/IceTargetLoweringX8664.cpp ('K') | « unittest/AssemblerX8664/GPRArith.cpp ('k') | unittest/AssemblerX8664/LowLevel.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: unittest/AssemblerX8664/Locked.cpp
diff --git a/unittest/AssemblerX8664/Locked.cpp b/unittest/AssemblerX8664/Locked.cpp
index 9a0904de2898ec59c79401d89268f6fbd0092895..150a09c8c0240726e42017636721631b9cf8075b 100644
--- a/unittest/AssemblerX8664/Locked.cpp
+++ b/unittest/AssemblerX8664/Locked.cpp
@@ -204,62 +204,60 @@ TEST_F(AssemblerX8664LowLevelTest, Xadd) {
// Ensures that xadd emits a lock prefix accordingly.
{
- __ xadd(IceType_i8, Address(0x1FF00, AssemblerFixup::NoFixup),
+ __ xadd(IceType_i8, Address::RipRelative(0x1FF00, AssemblerFixup::NoFixup),
Encoded_GPR_r14(), NotLocked);
- static constexpr uint8_t ByteCountNotLocked8 = 10;
+ static constexpr uint8_t ByteCountNotLocked8 = 9;
ASSERT_EQ(ByteCountNotLocked8, codeBytesSize());
- ASSERT_TRUE(verifyBytes<ByteCountNotLocked8>(codeBytes(), 0x67, 0x44, 0x0F,
- 0xC0, 0x34, 0x25, 0x00, 0xFF,
- 0x01, 0x00));
+ ASSERT_TRUE(verifyBytes<ByteCountNotLocked8>(
+ codeBytes(), 0x67, 0x44, 0x0F, 0xC0, 0x35, 0x00, 0xFF, 0x01, 0x00));
reset();
- __ xadd(IceType_i8, Address(0x1FF00, AssemblerFixup::NoFixup),
+ __ xadd(IceType_i8, Address::RipRelative(0x1FF00, AssemblerFixup::NoFixup),
Encoded_GPR_r14(), Locked);
static constexpr uint8_t ByteCountLocked8 = 1 + ByteCountNotLocked8;
ASSERT_EQ(ByteCountLocked8, codeBytesSize());
ASSERT_TRUE(verifyBytes<ByteCountLocked8>(codeBytes(), 0xF0, 0x67, 0x44,
- 0x0F, 0xC0, 0x34, 0x25, 0x00,
- 0xFF, 0x01, 0x00));
+ 0x0F, 0xC0, 0x35, 0x00, 0xFF,
+ 0x01, 0x00));
reset();
}
{
- __ xadd(IceType_i16, Address(0x1FF00, AssemblerFixup::NoFixup),
+ __ xadd(IceType_i16, Address::RipRelative(0x1FF00, AssemblerFixup::NoFixup),
Encoded_GPR_r14(), NotLocked);
- static constexpr uint8_t ByteCountNotLocked16 = 11;
+ static constexpr uint8_t ByteCountNotLocked16 = 10;
ASSERT_EQ(ByteCountNotLocked16, codeBytesSize());
ASSERT_TRUE(verifyBytes<ByteCountNotLocked16>(codeBytes(), 0x66, 0x67, 0x44,
- 0x0F, 0xC1, 0x34, 0x25, 0x00,
- 0xFF, 0x01, 0x00));
+ 0x0F, 0xC1, 0x35, 0x00, 0xFF,
+ 0x01, 0x00));
reset();
- __ xadd(IceType_i16, Address(0x1FF00, AssemblerFixup::NoFixup),
+ __ xadd(IceType_i16, Address::RipRelative(0x1FF00, AssemblerFixup::NoFixup),
Encoded_GPR_r14(), Locked);
static constexpr uint8_t ByteCountLocked16 = 1 + ByteCountNotLocked16;
ASSERT_EQ(ByteCountLocked16, codeBytesSize());
ASSERT_TRUE(verifyBytes<ByteCountLocked16>(codeBytes(), 0x66, 0xF0, 0x67,
- 0x44, 0x0F, 0xC1, 0x34, 0x25,
- 0x00, 0xFF, 0x01, 0x00));
+ 0x44, 0x0F, 0xC1, 0x35, 0x00,
+ 0xFF, 0x01, 0x00));
reset();
}
{
- __ xadd(IceType_i32, Address(0x1FF00, AssemblerFixup::NoFixup),
+ __ xadd(IceType_i32, Address::RipRelative(0x1FF00, AssemblerFixup::NoFixup),
Encoded_GPR_r14(), NotLocked);
- static constexpr uint8_t ByteCountNotLocked32 = 10;
+ static constexpr uint8_t ByteCountNotLocked32 = 9;
ASSERT_EQ(ByteCountNotLocked32, codeBytesSize());
- ASSERT_TRUE(verifyBytes<ByteCountNotLocked32>(codeBytes(), 0x67, 0x44, 0x0F,
- 0xC1, 0x34, 0x25, 0x00, 0xFF,
- 0x01, 0x00));
+ ASSERT_TRUE(verifyBytes<ByteCountNotLocked32>(
+ codeBytes(), 0x67, 0x44, 0x0F, 0xC1, 0x35, 0x00, 0xFF, 0x01, 0x00));
reset();
- __ xadd(IceType_i32, Address(0x1FF00, AssemblerFixup::NoFixup),
+ __ xadd(IceType_i32, Address::RipRelative(0x1FF00, AssemblerFixup::NoFixup),
Encoded_GPR_r14(), Locked);
static constexpr uint8_t ByteCountLocked32 = 1 + ByteCountNotLocked32;
ASSERT_EQ(ByteCountLocked32, codeBytesSize());
ASSERT_TRUE(verifyBytes<ByteCountLocked32>(codeBytes(), 0xF0, 0x67, 0x44,
- 0x0F, 0xC1, 0x34, 0x25, 0x00,
- 0xFF, 0x01, 0x00));
+ 0x0F, 0xC1, 0x35, 0x00, 0xFF,
+ 0x01, 0x00));
reset();
}
}
@@ -337,18 +335,19 @@ TEST_F(AssemblerX8664LowLevelTest, Cmpxchg8b) {
static constexpr bool Locked = true;
// Ensures that cmpxchg8b emits a lock prefix accordingly.
- __ cmpxchg8b(Address(0x1FF00, AssemblerFixup::NoFixup), NotLocked);
- static constexpr uint8_t ByteCountNotLocked = 9;
+ __ cmpxchg8b(Address::RipRelative(0x1FF00, AssemblerFixup::NoFixup),
+ NotLocked);
+ static constexpr uint8_t ByteCountNotLocked = 8;
ASSERT_EQ(ByteCountNotLocked, codeBytesSize());
- ASSERT_TRUE(verifyBytes<ByteCountNotLocked>(
- codeBytes(), 0x67, 0x0F, 0xC7, 0x0C, 0x25, 0x00, 0xFF, 0x01, 0x00));
+ ASSERT_TRUE(verifyBytes<ByteCountNotLocked>(codeBytes(), 0x67, 0x0F, 0xC7,
+ 0x0D, 0x00, 0xFF, 0x01, 0x00));
reset();
- __ cmpxchg8b(Address(0x1FF00, AssemblerFixup::NoFixup), Locked);
+ __ cmpxchg8b(Address::RipRelative(0x1FF00, AssemblerFixup::NoFixup), Locked);
static constexpr uint8_t ByteCountLocked = 1 + ByteCountNotLocked;
ASSERT_EQ(ByteCountLocked, codeBytesSize());
ASSERT_TRUE(verifyBytes<ByteCountLocked>(codeBytes(), 0xF0, 0x67, 0x0F, 0xC7,
- 0x0C, 0x25, 0x00, 0xFF, 0x01, 0x00));
+ 0x0D, 0x00, 0xFF, 0x01, 0x00));
reset();
}
@@ -444,62 +443,66 @@ TEST_F(AssemblerX8664LowLevelTest, Cmpxchg) {
// Ensures that cmpxchg emits a lock prefix accordingly.
{
- __ cmpxchg(IceType_i8, Address(0x1FF00, AssemblerFixup::NoFixup),
+ __ cmpxchg(IceType_i8,
+ Address::RipRelative(0x1FF00, AssemblerFixup::NoFixup),
Encoded_GPR_r14(), NotLocked);
- static constexpr uint8_t ByteCountNotLocked8 = 10;
+ static constexpr uint8_t ByteCountNotLocked8 = 9;
ASSERT_EQ(ByteCountNotLocked8, codeBytesSize());
- ASSERT_TRUE(verifyBytes<ByteCountNotLocked8>(codeBytes(), 0x67, 0x44, 0x0F,
- 0xB0, 0x34, 0x25, 0x00, 0xFF,
- 0x01, 0x00));
+ ASSERT_TRUE(verifyBytes<ByteCountNotLocked8>(
+ codeBytes(), 0x67, 0x44, 0x0F, 0xB0, 0x35, 0x00, 0xFF, 0x01, 0x00));
reset();
- __ cmpxchg(IceType_i8, Address(0x1FF00, AssemblerFixup::NoFixup),
+ __ cmpxchg(IceType_i8,
+ Address::RipRelative(0x1FF00, AssemblerFixup::NoFixup),
Encoded_GPR_r14(), Locked);
static constexpr uint8_t ByteCountLocked8 = 1 + ByteCountNotLocked8;
ASSERT_EQ(ByteCountLocked8, codeBytesSize());
ASSERT_TRUE(verifyBytes<ByteCountLocked8>(codeBytes(), 0xF0, 0x67, 0x44,
- 0x0F, 0xB0, 0x34, 0x25, 0x00,
- 0xFF, 0x01, 0x00));
+ 0x0F, 0xB0, 0x35, 0x00, 0xFF,
+ 0x01, 0x00));
reset();
}
{
- __ cmpxchg(IceType_i16, Address(0x1FF00, AssemblerFixup::NoFixup),
+ __ cmpxchg(IceType_i16,
+ Address::RipRelative(0x1FF00, AssemblerFixup::NoFixup),
Encoded_GPR_r14(), NotLocked);
- static constexpr uint8_t ByteCountNotLocked16 = 11;
+ static constexpr uint8_t ByteCountNotLocked16 = 10;
ASSERT_EQ(ByteCountNotLocked16, codeBytesSize());
ASSERT_TRUE(verifyBytes<ByteCountNotLocked16>(codeBytes(), 0x66, 0x67, 0x44,
- 0x0F, 0xB1, 0x34, 0x25, 0x00,
- 0xFF, 0x01, 0x00));
+ 0x0F, 0xB1, 0x35, 0x00, 0xFF,
+ 0x01, 0x00));
reset();
- __ cmpxchg(IceType_i16, Address(0x1FF00, AssemblerFixup::NoFixup),
+ __ cmpxchg(IceType_i16,
+ Address::RipRelative(0x1FF00, AssemblerFixup::NoFixup),
Encoded_GPR_r14(), Locked);
static constexpr uint8_t ByteCountLocked16 = 1 + ByteCountNotLocked16;
ASSERT_EQ(ByteCountLocked16, codeBytesSize());
ASSERT_TRUE(verifyBytes<ByteCountLocked16>(codeBytes(), 0x66, 0xF0, 0x67,
- 0x44, 0x0F, 0xB1, 0x34, 0x25,
- 0x00, 0xFF, 0x01, 0x00));
+ 0x44, 0x0F, 0xB1, 0x35, 0x00,
+ 0xFF, 0x01, 0x00));
reset();
}
{
- __ cmpxchg(IceType_i32, Address(0x1FF00, AssemblerFixup::NoFixup),
+ __ cmpxchg(IceType_i32,
+ Address::RipRelative(0x1FF00, AssemblerFixup::NoFixup),
Encoded_GPR_r14(), NotLocked);
- static constexpr uint8_t ByteCountNotLocked32 = 10;
+ static constexpr uint8_t ByteCountNotLocked32 = 9;
ASSERT_EQ(ByteCountNotLocked32, codeBytesSize());
- ASSERT_TRUE(verifyBytes<ByteCountNotLocked32>(codeBytes(), 0x67, 0x44, 0x0F,
- 0xB1, 0x34, 0x25, 0x00, 0xFF,
- 0x01, 0x00));
+ ASSERT_TRUE(verifyBytes<ByteCountNotLocked32>(
+ codeBytes(), 0x67, 0x44, 0x0F, 0xB1, 0x35, 0x00, 0xFF, 0x01, 0x00));
reset();
- __ cmpxchg(IceType_i32, Address(0x1FF00, AssemblerFixup::NoFixup),
+ __ cmpxchg(IceType_i32,
+ Address::RipRelative(0x1FF00, AssemblerFixup::NoFixup),
Encoded_GPR_r14(), Locked);
static constexpr uint8_t ByteCountLocked32 = 1 + ByteCountNotLocked32;
ASSERT_EQ(ByteCountLocked32, codeBytesSize());
ASSERT_TRUE(verifyBytes<ByteCountLocked32>(codeBytes(), 0xF0, 0x67, 0x44,
- 0x0F, 0xB1, 0x34, 0x25, 0x00,
- 0xFF, 0x01, 0x00));
+ 0x0F, 0xB1, 0x35, 0x00, 0xFF,
+ 0x01, 0x00));
reset();
}
}
« src/IceTargetLoweringX8664.cpp ('K') | « unittest/AssemblerX8664/GPRArith.cpp ('k') | unittest/AssemblerX8664/LowLevel.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698