summaryrefslogtreecommitdiff
path: root/compiler/utils/jni_macro_assembler_test.h
diff options
context:
space:
mode:
authorAndreas Gampe <agampe@google.com>2016-11-03 17:24:15 -0700
committerAndreas Gampe <agampe@google.com>2016-11-04 09:11:35 -0700
commit2e965aca73bacf84123b5c53bb0904b13b48e428 (patch)
treeedad34a00b358ca613b7e18781b91c4473e916fb /compiler/utils/jni_macro_assembler_test.h
parent2b64cccce7e3697e4ccca44fb95a6cc12c6441aa (diff)
ART: Use references in assembler tests
Move parameters to const references. Bug: 32619234 Test: m Change-Id: Ib68bdc313b91fee1e9e4e1e794eeca630837b005
Diffstat (limited to 'compiler/utils/jni_macro_assembler_test.h')
-rw-r--r--compiler/utils/jni_macro_assembler_test.h6
1 files changed, 3 insertions, 3 deletions
diff --git a/compiler/utils/jni_macro_assembler_test.h b/compiler/utils/jni_macro_assembler_test.h
index 829f34b4b7..293f4cde9c 100644
--- a/compiler/utils/jni_macro_assembler_test.h
+++ b/compiler/utils/jni_macro_assembler_test.h
@@ -39,12 +39,12 @@ class JNIMacroAssemblerTest : public testing::Test {
typedef std::string (*TestFn)(JNIMacroAssemblerTest* assembler_test, Ass* assembler);
- void DriverFn(TestFn f, std::string test_name) {
+ void DriverFn(TestFn f, const std::string& test_name) {
DriverWrapper(f(this, assembler_.get()), test_name);
}
// This driver assumes the assembler has already been called.
- void DriverStr(std::string assembly_string, std::string test_name) {
+ void DriverStr(const std::string& assembly_string, const std::string& test_name) {
DriverWrapper(assembly_string, test_name);
}
@@ -128,7 +128,7 @@ class JNIMacroAssemblerTest : public testing::Test {
virtual void Pad(std::vector<uint8_t>& data ATTRIBUTE_UNUSED) {
}
- void DriverWrapper(std::string assembly_text, std::string test_name) {
+ void DriverWrapper(const std::string& assembly_text, const std::string& test_name) {
assembler_->FinalizeCode();
size_t cs = assembler_->CodeSize();
std::unique_ptr<std::vector<uint8_t>> data(new std::vector<uint8_t>(cs));