Bug 138243 - Remove IncrementalSweeper::create()
Summary: Remove IncrementalSweeper::create()
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Gyuyoung Kim
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-10-31 00:30 PDT by Gyuyoung Kim
Modified: 2014-11-11 19:13 PST (History)
4 users (show)

See Also:


Attachments
Patch (7.68 KB, patch)
2014-10-31 00:32 PDT, Gyuyoung Kim
no flags Details | Formatted Diff | Diff
Patch (7.80 KB, patch)
2014-10-31 19:11 PDT, Gyuyoung Kim
no flags Details | Formatted Diff | Diff
Rebased (7.80 KB, patch)
2014-11-09 22:09 PST, Gyuyoung Kim
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Gyuyoung Kim 2014-10-31 00:30:00 PDT
SSIA
Comment 1 Gyuyoung Kim 2014-10-31 00:32:47 PDT
Created attachment 240728 [details]
Patch
Comment 2 Geoffrey Garen 2014-10-31 10:00:34 PDT
Comment on attachment 240728 [details]
Patch

/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/clang -x c++ -arch x86_64 -fmessage-length=0 -fdiagnostics-show-note-include-stack -fmacro-backtrace-limit=0 -std=gnu++11 -stdlib=libc++ -Wno-trigraphs -fno-exceptions -fno-rtti -fpascal-strings -O3 -Werror -Wno-missing-field-initializers -Wmissing-prototypes -Wnon-virtual-dtor -Wno-overloaded-virtual -Wno-exit-time-destructors -Wno-missing-braces -Wparentheses -Wswitch -Wunused-function -Wno-unused-label -Wno-unused-parameter -Wunused-variable -Wunused-value -Wempty-body -Wuninitialized -Wno-unknown-pragmas -Wno-shadow -Wno-four-char-constants -Wno-conversion -Wconstant-conversion -Wint-conversion -Wbool-conversion -Wenum-conversion -Wsign-compare -Wno-shorten-64-to-32 -Wnewline-eof -Wno-c++11-extensions -DNDEBUG -DHAVE_DTRACE=1 -DWEBKIT_VERSION_MIN_REQUIRED=WEBKIT_VERSION_LATEST -DHAVE_HEADER_DETECTION_H -DJSC_OBJC_API_AVAILABLE_MAC_OS_X_1080 -DENABLE_3D_RENDERING -DENABLE_CACHE_PARTITIONING -DENABLE_CANVAS_PATH -DENABLE_CHANNEL_MESSAGING -DENABLE_CONTENT_FILTERING -DENABLE_CSS_BOX_DECORATION_BREAK -DENABLE_CSS_COMPOSITING -DENABLE_CSS_GRID_LAYOUT -DENABLE_CSS_REGIONS -DENABLE_CSS_SELECTORS_LEVEL4 -DENABLE_CSS_SHAPES -DENABLE_CURSOR_VISIBILITY -DENABLE_DASHBOARD_SUPPORT -DENABLE_DETAILS_ELEMENT -DENABLE_DOM4_EVENTS_CONSTRUCTOR -DENABLE_FONT_LOAD_EVENTS -DENABLE_FULLSCREEN_API -DENABLE_GAMEPAD -DENABLE_GEOLOCATION -DENABLE_HIDDEN_PAGE_DOM_TIMER_THROTTLING -DENABLE_ICONDATABASE -DENABLE_INDEXED_DATABASE -DENABLE_INDIE_UI -DENABLE_INPUT_TYPE_COLOR -DENABLE_INPUT_TYPE_COLOR_POPOVER -DENABLE_INSPECTOR -DENABLE_LEGACY_CSS_VENDOR_PREFIXES -DENABLE_LEGACY_NOTIFICATIONS -DENABLE_LEGACY_VENDOR_PREFIXES -DENABLE_LEGACY_WEB_AUDIO -DENABLE_MATHML -DENABLE_MEDIA_CONTROLS_SCRIPT -DENABLE_METER_ELEMENT -DENABLE_MOUSE_CURSOR_SCALE -DENABLE_NAVIGATOR_CONTENT_UTILS -DENABLE_NAVIGATOR_HWCONCURRENCY -DENABLE_NOTIFICATIONS -DENABLE_PDFKIT_PLUGIN -DENABLE_PROMISES -DENABLE_PUBLIC_SUFFIX_LIST -DENABLE_REQUEST_ANIMATION_FRAME -DENABLE_RUBBER_BANDING -DENABLE_SHARED_WORKERS -DENABLE_CSS_SCROLL_SNAP -DENABLE_SPEECH_SYNTHESIS -DENABLE_SQL_DATABASE -DENABLE_SVG_FONTS -DENABLE_TEMPLATE_ELEMENT -DENABLE_USERSELECT_ALL -DENABLE_VIDEO -DENABLE_VIDEO_TRACK -DENABLE_DATACUE_VALUE -DENABLE_WEBGL -DENABLE_WEB_AUDIO -DENABLE_WEB_REPLAY -DENABLE_WEB_SOCKETS -DENABLE_PICTURE_SIZES -DENABLE_WEBVTT_REGIONS -DENABLE_XHR_TIMEOUT -DENABLE_XSLT -DENABLE_FTL_JIT -DENABLE_SATURATED_LAYOUT_ARITHMETIC -isysroot /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.8.sdk -fasm-blocks -fstrict-aliasing -Wdeprecated-declarations -Winvalid-offsetof -mmacosx-version-min=10.8 -g -fvisibility=hidden -fvisibility-inlines-hidden -fno-threadsafe-statics -Wno-sign-conversion -I/Volumes/Data/EWS/WebKit/WebKitBuild/JavaScriptCore.build/Release/JavaScriptCore.build/JavaScriptCore.hmap -I/Volumes/Data/EWS/WebKit/WebKitBuild/Release/include -I/Volumes/Data/EWS/WebKit/WebKitBuild/Release/DerivedSources/JavaScriptCore -I. -Iicu -I/Volumes/Data/EWS/WebKit/WebKitBuild/Release/usr/local/LLVMForJavaScriptCore/include -I/Volumes/Data/EWS/WebKit/WebKitBuild/Release/ExtraIncludesForLocalLLVMBuild -I/usr/local/LLVMForJavaScriptCore/include -I/Volumes/Data/EWS/WebKit/WebKitBuild/Release/usr/local/include -I/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/include -I/Volumes/Data/EWS/WebKit/WebKitBuild/JavaScriptCore.build/Release/JavaScriptCore.build/DerivedSources/x86_64 -I/Volumes/Data/EWS/WebKit/WebKitBuild/JavaScriptCore.build/Release/JavaScriptCore.build/DerivedSources -Wall -Wextra -Wcast-qual -Wchar-subscripts -Wextra-tokens -Wformat=2 -Winit-self -Wmissing-format-attribute -Wmissing-noreturn -Wpacked -Wpointer-arith -Wredundant-decls -Wundef -Wwrite-strings -Wexit-time-destructors -Wglobal-constructors -Wtautological-compare -Wimplicit-fallthrough -F/Volumes/Data/EWS/WebKit/WebKitBuild/Release -include /Volumes/Data/EWS/WebKit/WebKitBuild/PrecompiledHeaders/JavaScriptCorePrefix-gwdztaoleeiortczefcjhhyjwybx/JavaScriptCorePrefix.h -MMD -MT dependencies -MF /Volumes/Data/EWS/WebKit/WebKitBuild/JavaScriptCore.build/Release/JavaScriptCore.build/Objects-normal/x86_64/Heap.d --serialize-diagnostics /Volumes/Data/EWS/WebKit/WebKitBuild/JavaScriptCore.build/Release/JavaScriptCore.build/Objects-normal/x86_64/Heap.dia -c /Volumes/Data/EWS/WebKit/Source/JavaScriptCore/heap/Heap.cpp -o /Volumes/Data/EWS/WebKit/WebKitBuild/JavaScriptCore.build/Release/JavaScriptCore.build/Objects-normal/x86_64/Heap.o
In file included from /Volumes/Data/EWS/WebKit/Source/JavaScriptCore/heap/Heap.cpp:21:
In file included from /Volumes/Data/EWS/WebKit/Source/JavaScriptCore/config.h:56:
In file included from /Volumes/Data/EWS/WebKit/WebKitBuild/Release/usr/local/include/wtf/FastMalloc.h:27:
/Volumes/Data/EWS/WebKit/WebKitBuild/Release/usr/local/include/wtf/StdLibExtras.h:337:30: error: no matching constructor for initialization of 'JSC::IncrementalSweeper'
    return unique_ptr<T>(new T(std::forward<Args>(args)...));
                             ^ ~~~~~~~~~~~~~~~~~~~~~~~~
/Volumes/Data/EWS/WebKit/Source/JavaScriptCore/heap/Heap.cpp:334:17: note: in instantiation of function template specialization 'std::make_unique<JSC::IncrementalSweeper, JSC::VM *>' requested here
    , m_sweeper(std::make_unique<IncrementalSweeper>(this->vm()))
                ^
In file included from /Volumes/Data/EWS/WebKit/Source/JavaScriptCore/heap/Heap.cpp:38:
/Volumes/Data/EWS/WebKit/Source/JavaScriptCore/heap/IncrementalSweeper.h:38:7: note: candidate constructor (the implicit copy constructor) not viable: no known conversion from 'JSC::VM *' to 'const JSC::IncrementalSweeper' for 1st argument
class IncrementalSweeper : public HeapTimer {
      ^
/Volumes/Data/EWS/WebKit/Source/JavaScriptCore/heap/IncrementalSweeper.h:41:23: note: candidate constructor not viable: requires 2 arguments, but 1 was provided
    JS_EXPORT_PRIVATE IncrementalSweeper(Heap*, CFRunLoopRef);
                      ^
1 error generated.
Comment 3 Gyuyoung Kim 2014-10-31 19:11:14 PDT
Created attachment 240778 [details]
Patch
Comment 4 Gyuyoung Kim 2014-11-03 21:44:12 PST
Geoffrey, could you take a look this again ?
Comment 5 Gyuyoung Kim 2014-11-09 22:09:35 PST
Created attachment 241277 [details]
Rebased
Comment 6 Filip Pizlo 2014-11-11 18:33:42 PST
Comment on attachment 241277 [details]
Rebased

r=me.  I like this.
Comment 7 WebKit Commit Bot 2014-11-11 19:13:48 PST
Comment on attachment 241277 [details]
Rebased

Clearing flags on attachment: 241277

Committed r176000: <http://trac.webkit.org/changeset/176000>
Comment 8 WebKit Commit Bot 2014-11-11 19:13:52 PST
All reviewed patches have been landed.  Closing bug.