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

Issue 24233003: MIPS: replace RegExpCEntryStub with DirectCEntryStub. (Closed)

Created:
7 years, 3 months ago by kilvadyb
Modified:
7 years, 2 months ago
CC:
v8-dev
Base URL:
https://github.com/v8/v8.git@gbl
Visibility:
Public.

Description

MIPS: replace RegExpCEntryStub with DirectCEntryStub. Port r16618 (0eebc593) Original commit message: RegExpCEntryStub is therefore removed. BUG=none TEST=none

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+30 lines, -88 lines) Patch
M src/mips/code-stubs-mips.h View 1 chunk +0 lines, -16 lines 0 comments Download
M src/mips/code-stubs-mips.cc View 2 chunks +9 lines, -28 lines 0 comments Download
M src/mips/regexp-macro-assembler-mips.h View 1 chunk +0 lines, -8 lines 0 comments Download
M src/mips/regexp-macro-assembler-mips.cc View 3 chunks +21 lines, -36 lines 2 comments Download

Messages

Total messages: 3 (0 generated)
kilvadyb
7 years, 3 months ago (2013-09-19 17:37:33 UTC) #1
Paul Lind
inline question on your stack adjustment.... https://codereview.chromium.org/24233003/diff/1/src/mips/regexp-macro-assembler-mips.cc File src/mips/regexp-macro-assembler-mips.cc (right): https://codereview.chromium.org/24233003/diff/1/src/mips/regexp-macro-assembler-mips.cc#newcode1089 src/mips/regexp-macro-assembler-mips.cc:1089: __ lw(sp, MemOperand(sp, ...
7 years, 3 months ago (2013-09-19 18:07:10 UTC) #2
kilvadyb
7 years, 3 months ago (2013-09-19 18:37:26 UTC) #3
https://codereview.chromium.org/24233003/diff/1/src/mips/regexp-macro-assembl...
File src/mips/regexp-macro-assembler-mips.cc (right):

https://codereview.chromium.org/24233003/diff/1/src/mips/regexp-macro-assembl...
src/mips/regexp-macro-assembler-mips.cc:1089: __ lw(sp, MemOperand(sp, 16 +
stack_alignment + kCArgsSlotsSize));
On 2013/09/19 18:07:10, Paul Lind wrote:
> Where does the magic number 16 come from? You're calling
PrepareCallCFunction()
> with 3 args, but those fit within the (4) on-stack ArgsSlots. You already
> account for that with kCArgsSlotsSize. The only un-accounted for thing is the
> return address. That would be 4 bytes, or maybe 8 to preserve alignment.
The 16 comes from the allocation done by PrepareCallCFunction (3 arguments and 1
for the original stake pointer). Previous
RegExpMacroAssemblerMIPS::CallCFunctionUsingStub used the magic 16 also. I
couldn't find a good name for that. The kCArgsSlotsSize value comes from
DirectCEntryStub::Generate's stack allocation. There is no stack release/pop in
DirectCEntryStub::Generate as that function can be called from other places
where LeaveExitFrame takes care of the sp.

Powered by Google App Engine
This is Rietveld 408576698