Use memcpy in List::AddAll for fundamental types.

Use memcpy in List::AddAll for fundamental types.

R=svenpanne@chromium.org

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

[Sven Panne, 2014-10-24 08:04:01]

NOT LGTM.

https://codereview.chromium.org/663893004/diff/1/src/list-inl.h
File src/list-inl.h (right):

https://codereview.chromium.org/663893004/diff/1/src/list-inl.h#newcode36
src/list-inl.h:36: if (std::is_fundamental<T>::value) {
std:is_fundamental is a C++11 header feature, so we can't use that for now.

[Yang, 2014-10-24 08:34:35]

On 2014/10/24 08:04:01, Sven Panne wrote:
> NOT LGTM.
> 
> https://codereview.chromium.org/663893004/diff/1/src/list-inl.h
> File src/list-inl.h (right):
> 
> https://codereview.chromium.org/663893004/diff/1/src/list-inl.h#newcode36
> src/list-inl.h:36: if (std::is_fundamental<T>::value) {
> std:is_fundamental is a C++11 header feature, so we can't use that for now.

Implemented a replacement. Please review again.

[Yang, 2014-10-24 09:55:44]

On 2014/10/24 08:34:35, Yang wrote:
> On 2014/10/24 08:04:01, Sven Panne wrote:
> > NOT LGTM.
> > 
> > https://codereview.chromium.org/663893004/diff/1/src/list-inl.h
> > File src/list-inl.h (right):
> > 
> > https://codereview.chromium.org/663893004/diff/1/src/list-inl.h#newcode36
> > src/list-inl.h:36: if (std::is_fundamental<T>::value) {
> > std:is_fundamental is a C++11 header feature, so we can't use that for now.
> 
> Implemented a replacement. Please review again.

moved it to src/base/macros.h as discussed.

[Sven Panne, 2014-10-24 10:31:00]

lgtm

[Yang, 2014-10-24 12:34:06]

Committed patchset #4 (id:60001) manually as 24868 (presubmit successful).