• Andreas Haas's avatar
    Revert "[refactoring] Remove the isolate from signatures of ExternalReferences" · 7bfed2ad
    Andreas Haas authored
    This reverts commit 44ea425a.
    
    Reason for revert: https://ci.chromium.org/buildbot/client.v8.ports/V8%20Arm%20-%20debug%20builder/13575
    
    Original change's description:
    > [refactoring] Remove the isolate from signatures of ExternalReferences
    > 
    > In this CL I remove the isolate from signatures of ExternalReference
    > accessor functions where the isolate is not used. The uses of the
    > isolate were already removed in previous CLs.
    > 
    > Changes:
    > * I split the ExternalReference list in external-reference.h into
    > those which need the isolate for initialization and those which do not.
    > 
    > * I removed the public constructors and replaced them by
    >   ExternalReference::Create(). The reason is to separate external
    >   creation more clearly from internal creation, because externally
    >   created ExternalReferences sometimes need redirection, whereas
    >   internally created ExternalReferences are just stored as they are.
    >   In addition, by removing the isolate from the signature of the
    >   public constructors, they suddenly exactly matched the interal
    >   constructor.
    > 
    > * Replace all uses of the public constructors with
    >   ExternalReference::Create().
    > 
    > * Remove the isolate from all call sites where necessary.
    > 
    > 
    > This is a step towards making WebAssembly compilation independent of
    > the isolate.
    > 
    > Bug: v8:7570
    > R=​mstarzinger@chromium.org
    > 
    > Cq-Include-Trybots: luci.v8.try:v8_linux_noi18n_rel_ng
    > Change-Id: I14f511fc6acc50ab2d6a6641299f5ddbeabef0da
    > Reviewed-on: https://chromium-review.googlesource.com/1018982
    > Commit-Queue: Andreas Haas <ahaas@chromium.org>
    > Reviewed-by: Michael Starzinger <mstarzinger@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#52768}
    
    TBR=mstarzinger@chromium.org,ahaas@chromium.org
    
    Change-Id: I7c0d8d420f815cede23d550dee8942ac4d7791cc
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    Bug: v8:7570
    Cq-Include-Trybots: luci.v8.try:v8_linux_noi18n_rel_ng
    Reviewed-on: https://chromium-review.googlesource.com/1026570Reviewed-by: 's avatarAndreas Haas <ahaas@chromium.org>
    Commit-Queue: Andreas Haas <ahaas@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#52769}
    7bfed2ad
Name
Last commit
Last update
..
arm Loading commit data...
arm64 Loading commit data...
ia32 Loading commit data...
mips Loading commit data...
mips64 Loading commit data...
ppc Loading commit data...
s390 Loading commit data...
x64 Loading commit data...
DEPS Loading commit data...
liftoff-assembler-defs.h Loading commit data...
liftoff-assembler.cc Loading commit data...
liftoff-assembler.h Loading commit data...
liftoff-compiler.cc Loading commit data...
liftoff-compiler.h Loading commit data...
liftoff-register.h Loading commit data...