Patchset 3 (id:??) landed as
https://crrev.com/218e101297dc21f674499731b7119e339ca157e1
Cr-Commit-Position: refs/heads/master@{#28510}
https://codereview.chromium.org/1140743004/
--
--
v8-dev mailing list
v8-dev@googlegroups.com
http://groups.google.com/group/v8-dev
---
You received this messag
Committed patchset #3 (id:40001)
https://codereview.chromium.org/1140743004/
--
--
v8-dev mailing list
v8-dev@googlegroups.com
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
Feedback addressed
https://codereview.chromium.org/1140743004/diff/1/src/compiler/js-inlining.cc
File src/compiler/js-inlining.cc (right):
https://codereview.chromium.org/1140743004/diff/1/src/compiler/js-inlining.cc#newcode298
src/compiler/js-inlining.cc:298: if (mode_ == kBuiltinsInlining &&
CQ is trying da patch. Follow status at
https://chromium-cq-status.appspot.com/patch-status/1140743004/40001
https://codereview.chromium.org/1140743004/
--
--
v8-dev mailing list
v8-dev@googlegroups.com
http://groups.google.com/group/v8-dev
---
You received this message because you are subscr
LGTM with nits.
https://codereview.chromium.org/1140743004/diff/1/src/compiler/js-inlining.cc
File src/compiler/js-inlining.cc (right):
https://codereview.chromium.org/1140743004/diff/1/src/compiler/js-inlining.cc#newcode298
src/compiler/js-inlining.cc:298: if (mode_ == kBuiltinsInlining &&
!fu