The new patch set is ready.
https://codereview.chromium.org/164793003/diff/250001/src/a64/macro-assembler-a64.cc
File src/a64/macro-assembler-a64.cc (right):
https://codereview.chromium.org/164793003/diff/250001/src/a64/macro-assembler-a64.cc#newcode4828
src/a64/macro-assembler-a64.cc:4828:
ASSERT(!TmpList()->IncludesAliasOf(arg0));
On 2014/02/28 13:44:30, rmcilroy wrote:
nit -
ASSERT(!(TmpList()->IncludesAliasOf(arg0) ||
FPTmpList->IncludesAliasOf(arg0)))
etc.
Fixed, though they won't fit on one line so I just made separate
assertions.
https://codereview.chromium.org/164793003/diff/250001/src/a64/macro-assembler-a64.cc#newcode4984
src/a64/macro-assembler-a64.cc:4984: RegList old_fptmp_list =
FPTmpList()->list();
On 2014/02/28 13:44:30, rmcilroy wrote:
nit - old_fp_tmp_list (not old_fptmp_list)
Done.
https://codereview.chromium.org/164793003/diff/250001/src/a64/macro-assembler-a64.cc#newcode4996
src/a64/macro-assembler-a64.cc:4996: CPURegList fptmp_list =
kCallerSavedFP;
On 2014/02/28 13:44:30, rmcilroy wrote:
nit - fp_tmp_list (not fptmp_list)
Done.
https://codereview.chromium.org/164793003/diff/250001/src/a64/macro-assembler-a64.h
File src/a64/macro-assembler-a64.h (right):
https://codereview.chromium.org/164793003/diff/250001/src/a64/macro-assembler-a64.h#newcode2232
src/a64/macro-assembler-a64.h:2232: static void
ReleaseByCode(CPURegList* available, int code);
On 2014/02/28 13:44:30, rmcilroy wrote:
I don't think you need ReleaseByCode, ReleaseByRegList or
IncludeByRegList any
more do you? Could you delete these please.
Ah yes, I missed those, thanks for the reminder!
https://codereview.chromium.org/164793003/
--
--
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.
