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

Unified Diff: unittest/AssemblerX8632/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
Index: unittest/AssemblerX8632/TestUtil.h
diff --git a/unittest/AssemblerX8632/TestUtil.h b/unittest/AssemblerX8632/TestUtil.h
index a22c637d357913e69b07a1e724da8d4e26a1da06..5104fc49a41b04ec0f91546e14df428804633370 100644
--- a/unittest/AssemblerX8632/TestUtil.h
+++ b/unittest/AssemblerX8632/TestUtil.h
@@ -729,8 +729,12 @@ protected:
if (NeedsEpilogue) {
addEpilogue();
}
-
NeedsEpilogue = false;
+
+ for (auto *Fixup : assembler()->fixups()) {
Jim Stichnoth 2016/02/03 23:18:27 const auto *Fixup, right?
John 2016/02/04 18:28:50 Done.
+ Fixup->emitOffset(assembler());
+ }
+
return AssembledTest(codeBytes(), codeBytesSize(), NumAllocatedDwords);
}

Powered by Google App Engine
This is Rietveld 408576698