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

Unified Diff: unittest/AssemblerX8664/TestUtil.h

Issue 1669443002: Subzero. Uses fixups to calculate addend to relocations. (Closed) Base URL: https://chromium.googlesource.com/native_client/pnacl-subzero.git@master
Patch Set: Addresses comments. 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
« unittest/AssemblerX8632/TestUtil.h ('K') | « unittest/AssemblerX8664/GPRArith.cpp ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: unittest/AssemblerX8664/TestUtil.h
diff --git a/unittest/AssemblerX8664/TestUtil.h b/unittest/AssemblerX8664/TestUtil.h
index 1c681312b15ea77635bf9020cfde0361b8e31db1..4600a9a0653814a4f3ca330f298c55bc018fb32b 100644
--- a/unittest/AssemblerX8664/TestUtil.h
+++ b/unittest/AssemblerX8664/TestUtil.h
@@ -913,8 +913,12 @@ protected:
if (NeedsEpilogue) {
addEpilogue();
}
-
NeedsEpilogue = false;
+
+ for (auto *Fixup : assembler()->fixups()) {
Jim Stichnoth 2016/02/03 23:18:27 const auto * ?
John 2016/02/04 18:28:50 Done.
+ Fixup->emitOffset(assembler());
+ }
+
return AssembledTest(codeBytes(), codeBytesSize(), NumAllocatedDwords);
}
« unittest/AssemblerX8632/TestUtil.h ('K') | « unittest/AssemblerX8664/GPRArith.cpp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698