A64: Minor cleaning in StoreStubCompiler::GenerateStoreField.

A64: Minor cleaning in StoreStubCompiler::GenerateStoreField.

Abstract a register to simplify code generation.

R=ulan@chromium.org

Committed: https://code.google.com/p/v8/source/detail?r=20116

[ulan, 2014-03-20 10:39:36]

https://codereview.chromium.org/206183005/diff/1/src/a64/stub-cache-a64.cc
File src/a64/stub-cache-a64.cc (right):

https://codereview.chromium.org/206183005/diff/1/src/a64/stub-cache-a64.cc#ne...
src/a64/stub-cache-a64.cc:465: (FLAG_track_double_fields &&
representation.IsDouble()) ? storage_reg
why FLAG_track_double_fields?

[Alexandre Rames, 2014-03-20 10:44:28]

On 2014/03/20 10:39:36, ulan wrote:
> https://codereview.chromium.org/206183005/diff/1/src/a64/stub-cache-a64.cc
> File src/a64/stub-cache-a64.cc (right):
> 
>
https://codereview.chromium.org/206183005/diff/1/src/a64/stub-cache-a64.cc#ne...
> src/a64/stub-cache-a64.cc:465: (FLAG_track_double_fields &&
> representation.IsDouble()) ? storage_reg
> why FLAG_track_double_fields?

Oh sorry, this was part of the test, but was changed very recently and I didn't
see it while rebasing. Updating the patch.

[Alexandre Rames, 2014-03-20 10:47:33]

Updated...

[ulan, 2014-03-20 10:49:26]

lgtm

https://codereview.chromium.org/206183005/diff/2/src/a64/stub-cache-a64.cc
File src/a64/stub-cache-a64.cc (right):

https://codereview.chromium.org/206183005/diff/2/src/a64/stub-cache-a64.cc#ne...
src/a64/stub-cache-a64.cc:464: Register prop_reg = (representation.IsDouble()) ?
storage_reg : value_reg;
Nit: no need to put representation.IsDouble() in ().

[Alexandre Rames, 2014-03-20 11:35:13]

Committed patchset #2 manually as r20116 (presubmit successful).