Use Type* in crankshaft rather than HeapType.

Use Type* in crankshaft rather than HeapType.

R=rossberg@chromium.org

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

[Toon Verwaest, 2014-02-03 15:52:04]

PTAL

[rossberg, 2014-02-03 16:54:05]

https://codereview.chromium.org/152863002/diff/30001/src/accessors.h
File src/accessors.h (right):

https://codereview.chromium.org/152863002/diff/30001/src/accessors.h#newcode103
src/accessors.h:103: template <class T>
You don't necessarily need to move this to the header file. I'd keep the
implementation in the .cc and specialise the template to the only two types we
need.

https://codereview.chromium.org/152863002/diff/30001/src/hydrogen.h
File src/hydrogen.h (right):

https://codereview.chromium.org/152863002/diff/30001/src/hydrogen.h#newcode2313
src/hydrogen.h:2313: return Accessors::IsJSObjectFieldAccessor<Type>(type_,
name_, &offset);
Was it necessary to specify the template argument?

https://codereview.chromium.org/152863002/diff/30001/src/ic.h
File src/ic.h (right):

https://codereview.chromium.org/152863002/diff/30001/src/ic.h#newcode142
src/ic.h:142: template<class T>
Same here.

[Toon Verwaest, 2014-02-04 09:24:19]

Addressed comments, ptal again

[rossberg, 2014-02-04 12:23:00]

lgtm

[Toon Verwaest, 2014-02-04 12:44:24]

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