Comment #5 on issue 3952 by [email protected]: Support for inlining
builtins in TurboFan
https://code.google.com/p/v8/issues/detail?id=3952#c5
The following revision refers to this bug:
https://chromium.googlesource.com/v8/v8.git/+/eacd6d8b7d175032b65aa65fa11c9419abdaa236
commit eacd6d8b7d175032b65aa65fa11c9419abdaa236
Author: Benedikt Meurer <[email protected]>
Date: Mon Mar 09 13:01:30 2015
[turbofan] Make inlining independent of typing.
The JSInliner used to load the context from the JSFunction node at
runtime, which introduced a HeapConstant (because we had to materialize
the JSFunction after context specialization) and a LoadField operation,
independent whether the inlinee actually uses the context. This is
rather cumbersome currently, and therefore this is now changed to just
embed the context constant instead. Once we do inlining based on
SharedFunctionInfo rather than JSFunction, we should reconsider this
decision and come up with a proper heuristic.
BUG=v8:3952
LOG=n
[email protected]
Review URL: https://codereview.chromium.org/994523002
Cr-Commit-Position: refs/heads/master@{#27069}
[modify]
http://crrev.com/eacd6d8b7d175032b65aa65fa11c9419abdaa236/src/compiler/js-inlining.cc
[modify]
http://crrev.com/eacd6d8b7d175032b65aa65fa11c9419abdaa236/src/flag-definitions.h
--
You received this message because this project is configured to send all
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings
--
--
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/d/optout.
