Revision: 19316
Author: [email protected]
Date: Wed Feb 12 11:55:52 2014 UTC
Log: A64: Remove two TODOs that will not be done.
- For the LDP/STP TODO: The arguments need dereferencing, so using LDP/STP
would
require some additional checks.
- For the constant pools TODO: Unless we need to, there is no benefit from
trying to generate constant pools more often, as it would likely only
pollute
the I-cache.
[email protected]
Review URL: https://codereview.chromium.org/151363004
http://code.google.com/p/v8/source/detail?r=19316
Modified:
/branches/bleeding_edge/src/a64/assembler-a64.cc
/branches/bleeding_edge/src/a64/builtins-a64.cc
=======================================
--- /branches/bleeding_edge/src/a64/assembler-a64.cc Wed Feb 12 09:19:30
2014 UTC
+++ /branches/bleeding_edge/src/a64/assembler-a64.cc Wed Feb 12 11:55:52
2014 UTC
@@ -2354,9 +2354,6 @@
}
-// TODO(all): We are never trying to emit constant pools after
unconditional
-// branches, because we only call it from Assembler::Emit() (or manually).
-// We should try to enable that.
void Assembler::CheckConstPool(bool force_emit, bool require_jump) {
// Some short sequence of instruction mustn't be broken up by constant
pool
// emission, such sequences are protected by calls to BlockConstPoolFor
and
=======================================
--- /branches/bleeding_edge/src/a64/builtins-a64.cc Wed Feb 12 09:19:30
2014 UTC
+++ /branches/bleeding_edge/src/a64/builtins-a64.cc Wed Feb 12 11:55:52
2014 UTC
@@ -684,8 +684,6 @@
// Compute the copy end address.
__ Add(x10, argv, Operand(argc, LSL, kPointerSizeLog2));
- // TODO(all): This can potentially be optimized with ldp/stp to speed
up
- // arguments passing from C++ to JS.
__ B(&entry);
__ Bind(&loop);
__ Ldr(x11, MemOperand(argv, kPointerSize, PostIndex));
--
--
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.
