Revision: 19503
Author: [email protected]
Date: Thu Feb 20 10:08:04 2014 UTC
Log: A64: Fix int32 use in Lithium string functions
Assert register sizes in StringCharLoadGenerator, and fix char_code
comparison
in DoStringCharFromCode.
BUG=
[email protected]
Review URL: https://codereview.chromium.org/172483002
http://code.google.com/p/v8/source/detail?r=19503
Modified:
/branches/bleeding_edge/src/a64/code-stubs-a64.cc
/branches/bleeding_edge/src/a64/codegen-a64.cc
/branches/bleeding_edge/src/a64/codegen-a64.h
/branches/bleeding_edge/src/a64/lithium-codegen-a64.cc
=======================================
--- /branches/bleeding_edge/src/a64/code-stubs-a64.cc Wed Feb 19 13:55:25
2014 UTC
+++ /branches/bleeding_edge/src/a64/code-stubs-a64.cc Thu Feb 20 10:08:04
2014 UTC
@@ -3540,7 +3540,7 @@
StringCharLoadGenerator::Generate(masm,
object_,
- index_,
+ index_.W(),
result_,
&call_runtime_);
__ SmiTag(result_);
=======================================
--- /branches/bleeding_edge/src/a64/codegen-a64.cc Mon Feb 17 15:20:54 2014
UTC
+++ /branches/bleeding_edge/src/a64/codegen-a64.cc Thu Feb 20 10:08:04 2014
UTC
@@ -410,6 +410,7 @@
Register index,
Register result,
Label* call_runtime) {
+ ASSERT(string.Is64Bits() && index.Is32Bits() && result.Is64Bits());
// Fetch the instance type of the receiver into result register.
__ Ldr(result, FieldMemOperand(string, HeapObject::kMapOffset));
__ Ldrb(result, FieldMemOperand(result, Map::kInstanceTypeOffset));
@@ -424,10 +425,10 @@
// Handle slices.
Label indirect_string_loaded;
- __ Ldrsw(result,
- UntagSmiFieldMemOperand(string, SlicedString::kOffsetOffset));
+ __ Ldr(result.W(),
+ UntagSmiFieldMemOperand(string, SlicedString::kOffsetOffset));
__ Ldr(string, FieldMemOperand(string, SlicedString::kParentOffset));
- __ Add(index, index, result);
+ __ Add(index, index, result.W());
__ B(&indirect_string_loaded);
// Handle cons strings.
@@ -479,11 +480,11 @@
STATIC_ASSERT(kTwoByteStringTag == 0);
__ TestAndBranchIfAnySet(result, kStringEncodingMask, &ascii);
// Two-byte string.
- __ Ldrh(result, MemOperand(string, index, LSL, 1));
+ __ Ldrh(result, MemOperand(string, index, SXTW, 1));
__ B(&done);
__ Bind(&ascii);
// Ascii string.
- __ Ldrb(result, MemOperand(string, index));
+ __ Ldrb(result, MemOperand(string, index, SXTW));
__ Bind(&done);
}
=======================================
--- /branches/bleeding_edge/src/a64/codegen-a64.h Wed Feb 12 09:19:30 2014
UTC
+++ /branches/bleeding_edge/src/a64/codegen-a64.h Thu Feb 20 10:08:04 2014
UTC
@@ -38,7 +38,8 @@
public:
// Generates the code for handling different string types and loading the
// indexed character into |result|. We expect |index| as untagged input
and
- // |result| as untagged output.
+ // |result| as untagged output. Register index is asserted to be a
32-bit W
+ // register.
static void Generate(MacroAssembler* masm,
Register string,
Register index,
=======================================
--- /branches/bleeding_edge/src/a64/lithium-codegen-a64.cc Wed Feb 19
14:03:48 2014 UTC
+++ /branches/bleeding_edge/src/a64/lithium-codegen-a64.cc Thu Feb 20
10:08:04 2014 UTC
@@ -5161,7 +5161,7 @@
StringCharLoadGenerator::Generate(masm(),
ToRegister(instr->string()),
- ToRegister(instr->index()),
+ ToRegister32(instr->index()),
ToRegister(instr->result()),
deferred->entry());
__ Bind(deferred->exit());
@@ -5208,13 +5208,13 @@
new(zone()) DeferredStringCharFromCode(this, instr);
ASSERT(instr->hydrogen()->value()->representation().IsInteger32());
- Register char_code = ToRegister(instr->char_code());
+ Register char_code = ToRegister32(instr->char_code());
Register result = ToRegister(instr->result());
- __ Cmp(char_code, Operand(String::kMaxOneByteCharCode));
+ __ Cmp(char_code, String::kMaxOneByteCharCode);
__ B(hi, deferred->entry());
__ LoadRoot(result, Heap::kSingleCharacterStringCacheRootIndex);
- __ Add(result, result, Operand(char_code, LSL, kPointerSizeLog2));
+ __ Add(result, result, Operand(char_code, SXTW, kPointerSizeLog2));
__ Ldr(result, FieldMemOperand(result, FixedArray::kHeaderSize));
__ CompareRoot(result, Heap::kUndefinedValueRootIndex);
__ B(eq, deferred->entry());
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.
