RESOLVED FIXED Bug 149171
REGRESSION(r189774): CLoop doesn't build after r189774 
https://bugs.webkit.org/show_bug.cgi?id=149171
Summary REGRESSION(r189774): CLoop doesn't build after r189774
Michael Saboff
Reported 2015-09-15 10:58:12 PDT
After r189774, the CLoop doesn't build. CompileC /Volumes/Data/slave/yosemite-cloop-debug/build/WebKitBuild/JavaScriptCore.build/Debug/JavaScriptCore.build/Objects-normal/x86_64/LowLevelInterpreter.o llint/LowLevelInterpreter.cpp normal x86_64 c++ com.apple.compilers.llvm.clang.1_0.compiler cd /Volumes/Data/slave/yosemite-cloop-debug/build/Source/JavaScriptCore export LANG=en_US.US-ASCII /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 -O0 -fno-common -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 -DHAVE_HEADER_DETECTION_H -DENABLE_3D_TRANSFORMS -DENABLE_AVF_CAPTIONS -DENABLE_ATTACHMENT_ELEMENT -DENABLE_CACHE_PARTITIONING -DENABLE_CANVAS_PATH -DENABLE_CHANNEL_MESSAGING -DENABLE_ES6_ARROWFUNCTION_SYNTAX -DENABLE_ES6_CLASS_SYNTAX -DENABLE_ES6_TEMPLATE_LITERAL_SYNTAX -DENABLE_CONTENT_FILTERING -DENABLE_CSS_ANIMATIONS_LEVEL_2 -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_ENCRYPTED_MEDIA -DENABLE_ENCRYPTED_MEDIA_V2 -DENABLE_FILTERS_LEVEL_2 -DENABLE_FULLSCREEN_API -DENABLE_GAMEPAD -DENABLE_GEOLOCATION -DENABLE_HIDDEN_PAGE_DOM_TIMER_THROTTLING -DENABLE_ICONDATABASE -DENABLE_SERVICE_CONTROLS -DENABLE_INDEXED_DATABASE -DENABLE_INDIE_UI -DENABLE_INPUT_TYPE_COLOR -DENABLE_INPUT_TYPE_COLOR_POPOVER -DENABLE_INTL -DENABLE_LEGACY_CSS_VENDOR_PREFIXES -DENABLE_LEGACY_NOTIFICATIONS -DENABLE_LEGACY_VENDOR_PREFIXES -DENABLE_LEGACY_WEB_AUDIO -DENABLE_MATHML -DENABLE_MEDIA_CONTROLS_SCRIPT -DENABLE_MEDIA_SOURCE -DENABLE_MEDIA_STREAM -DENABLE_METER_ELEMENT -DENABLE_MOUSE_CURSOR_SCALE -DENABLE_NAVIGATOR_HWCONCURRENCY -DENABLE_NOTIFICATIONS -DENABLE_PDFKIT_PLUGIN -DENABLE_PUBLIC_SUFFIX_LIST -DENABLE_REQUEST_ANIMATION_FRAME -DENABLE_REMOTE_INSPECTOR -DENABLE_RUBBER_BANDING -DENABLE_CSS_SCROLL_SNAP -DENABLE_SPEECH_SYNTHESIS -DENABLE_STREAMS_API -DENABLE_SUBTLE_CRYPTO -DENABLE_SVG_FONTS -DENABLE_SVG_OTF_CONVERTER -DENABLE_TELEPHONE_NUMBER_DETECTION -DENABLE_TEMPLATE_ELEMENT -DENABLE_CSS_TRAILING_WORD -DENABLE_USERSELECT_ALL -DENABLE_VIDEO -DENABLE_VIDEO_TRACK -DENABLE_DATACUE_VALUE -DENABLE_WEBGL -DENABLE_WEB_AUDIO -DENABLE_WEB_REPLAY -DENABLE_WEB_SOCKETS -DENABLE_WEB_TIMING -DENABLE_XHR_TIMEOUT -DENABLE_XSLT -DENABLE_FTL_JIT=0 -DENABLE_JIT=0 -DENABLE_SATURATED_LAYOUT_ARITHMETIC -isysroot /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.10.sdk -fasm-blocks -fstrict-aliasing -Wdeprecated-declarations -Winvalid-offsetof -mmacosx-version-min=10.10 -g -fvisibility=hidden -fvisibility-inlines-hidden -fno-threadsafe-statics -Wno-sign-conversion -I/Volumes/Data/slave/yosemite-cloop-debug/build/WebKitBuild/JavaScriptCore.build/Debug/JavaScriptCore.build/JavaScriptCore.hmap -I/Volumes/Data/slave/yosemite-cloop-debug/build/WebKitBuild/Debug/include -I/Volumes/Data/slave/yosemite-cloop-debug/build/WebKitBuild/Debug/DerivedSources/JavaScriptCore -I. -Iicu -I/Volumes/Data/slave/yosemite-cloop-debug/build/WebKitBuild/Debug/usr/local/LLVMForJavaScriptCore/include -I/Volumes/Data/slave/yosemite-cloop-debug/build/WebKitBuild/Debug/ExtraIncludesForLocalLLVMBuild -I/usr/local/LLVMForJavaScriptCore/include -I/Volumes/Data/slave/yosemite-cloop-debug/build/WebKitBuild/Debug/usr/local/include -I/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/include -I/Volumes/Data/slave/yosemite-cloop-debug/build/WebKitBuild/JavaScriptCore.build/Debug/JavaScriptCore.build/DerivedSources/x86_64 -I/Volumes/Data/slave/yosemite-cloop-debug/build/WebKitBuild/JavaScriptCore.build/Debug/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/slave/yosemite-cloop-debug/build/WebKitBuild/Debug -include /Volumes/Data/slave/yosemite-cloop-debug/build/WebKitBuild/PrecompiledHeaders/JavaScriptCorePrefix-eymtnrvesgguerevudxwiwbiqvlc/JavaScriptCorePrefix.h -MMD -MT dependencies -MF /Volumes/Data/slave/yosemite-cloop-debug/build/WebKitBuild/JavaScriptCore.build/Debug/JavaScriptCore.build/Objects-normal/x86_64/LowLevelInterpreter.d --serialize-diagnostics /Volumes/Data/slave/yosemite-cloop-debug/build/WebKitBuild/JavaScriptCore.build/Debug/JavaScriptCore.build/Objects-normal/x86_64/LowLevelInterpreter.dia -c /Volumes/Data/slave/yosemite-cloop-debug/build/Source/JavaScriptCore/llint/LowLevelInterpreter.cpp -o /Volumes/Data/slave/yosemite-cloop-debug/build/WebKitBuild/JavaScriptCore.build/Debug/JavaScriptCore.build/Objects-normal/x86_64/LowLevelInterpreter.o In file included from /Volumes/Data/slave/yosemite-cloop-debug/build/Source/JavaScriptCore/llint/LowLevelInterpreter.cpp:425: /Volumes/Data/slave/yosemite-cloop-debug/build/WebKitBuild/Debug/DerivedSources/JavaScriptCore/LLIntAssembly.h:7003:27: error: use of undeclared identifier 'llint_cloop_did_return_from_js_10'; did you mean 'llint_cloop_did_return_from_js_1'? lr.opcode = getOpcode(llint_cloop_did_return_from_js_10); // JavaScriptCore/llint/LowLevelInterpreter.asm:703 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ llint_cloop_did_return_from_js_1 In file included from /Volumes/Data/slave/yosemite-cloop-debug/build/Source/JavaScriptCore/llint/LowLevelInterpreter.cpp:27: In file included from /Volumes/Data/slave/yosemite-cloop-debug/build/Source/JavaScriptCore/llint/LowLevelInterpreter.h:29: /Volumes/Data/slave/yosemite-cloop-debug/build/Source/JavaScriptCore/bytecode/Opcode.h:58:20: note: 'llint_cloop_did_return_from_js_1' declared here typedef enum { FOR_EACH_OPCODE_ID(OPCODE_ID_ENUM) } OpcodeID; ^ /Volumes/Data/slave/yosemite-cloop-debug/build/Source/JavaScriptCore/bytecode/Opcode.h:53:9: note: expanded from macro 'FOR_EACH_OPCODE_ID' FOR_EACH_LLINT_OPCODE_EXTENSION(macro) \ ^ In file included from /Volumes/Data/slave/yosemite-cloop-debug/build/Source/JavaScriptCore/llint/LowLevelInterpreter.cpp:27: In file included from /Volumes/Data/slave/yosemite-cloop-debug/build/Source/JavaScriptCore/llint/LowLevelInterpreter.h:29: In file included from /Volumes/Data/slave/yosemite-cloop-debug/build/Source/JavaScriptCore/bytecode/Opcode.h:34: /Volumes/Data/slave/yosemite-cloop-debug/build/Source/JavaScriptCore/llint/LLIntOpcode.h:49:48: note: expanded from macro 'FOR_EACH_LLINT_OPCODE_EXTENSION' #define FOR_EACH_LLINT_OPCODE_EXTENSION(macro) FOR_EACH_LLINT_NATIVE_HELPER(macro) ^ /Volumes/Data/slave/yosemite-cloop-debug/build/Source/JavaScriptCore/llint/LLIntOpcode.h:43:5: note: expanded from macro 'FOR_EACH_LLINT_NATIVE_HELPER' FOR_EACH_LLINT_NOJIT_NATIVE_HELPER(macro) \ ^ /Volumes/Data/slave/yosemite-cloop-debug/build/Source/JavaScriptCore/llint/LLIntOpcode.h:32:5: note: expanded from macro 'FOR_EACH_LLINT_NOJIT_NATIVE_HELPER' FOR_EACH_CLOOP_BYTECODE_HELPER_ID(macro) ^ In file included from /Volumes/Data/slave/yosemite-cloop-debug/build/Source/JavaScriptCore/llint/LowLevelInterpreter.cpp:27: In file included from /Volumes/Data/slave/yosemite-cloop-debug/build/Source/JavaScriptCore/llint/LowLevelInterpreter.h:29: In file included from /Volumes/Data/slave/yosemite-cloop-debug/build/Source/JavaScriptCore/bytecode/Opcode.h:33: /Volumes/Data/slave/yosemite-cloop-debug/build/WebKitBuild/Debug/DerivedSources/JavaScriptCore/Bytecodes.h:167:11: note: expanded from macro 'FOR_EACH_CLOOP_BYTECODE_HELPER_ID' macro(llint_cloop_did_return_from_js_1, 1) \ ^ In file included from /Volumes/Data/slave/yosemite-cloop-debug/build/Source/JavaScriptCore/llint/LowLevelInterpreter.cpp:27: In file included from /Volumes/Data/slave/yosemite-cloop-debug/build/Source/JavaScriptCore/llint/LowLevelInterpreter.h:29: /Volumes/Data/slave/yosemite-cloop-debug/build/Source/JavaScriptCore/bytecode/Opcode.h:57:40: note: expanded from macro 'OPCODE_ID_ENUM' #define OPCODE_ID_ENUM(opcode, length) opcode, ^ /Volumes/Data/slave/yosemite-cloop-debug/build/Source/JavaScriptCore/bytecode/Opcode.h:45:5: note: expanded from macro 'FOR_EACH_CORE_OPCODE_ID_WITH_EXTENSION' extension__ ^ In file included from /Volumes/Data/slave/yosemite-cloop-debug/build/Source/JavaScriptCore/llint/LowLevelInterpreter.cpp:425: /Volumes/Data/slave/yosemite-cloop-debug/build/WebKitBuild/Debug/DerivedSources/JavaScriptCore/LLIntAssembly.h:7039:27: error: use of undeclared identifier 'llint_cloop_did_return_from_js_11'; did you mean 'llint_cloop_did_return_from_js_1'? lr.opcode = getOpcode(llint_cloop_did_return_from_js_11); // JavaScriptCore/llint/LowLevelInterpreter.asm:703 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ llint_cloop_did_return_from_js_1 In file included from /Volumes/Data/slave/yosemite-cloop-debug/build/Source/JavaScriptCore/llint/LowLevelInterpreter.cpp:27: In file included from /Volumes/Data/slave/yosemite-cloop-debug/build/Source/JavaScriptCore/llint/LowLevelInterpreter.h:29: /Volumes/Data/slave/yosemite-cloop-debug/build/Source/JavaScriptCore/bytecode/Opcode.h:58:20: note: 'llint_cloop_did_return_from_js_1' declared here typedef enum { FOR_EACH_OPCODE_ID(OPCODE_ID_ENUM) } OpcodeID; ^ /Volumes/Data/slave/yosemite-cloop-debug/build/Source/JavaScriptCore/bytecode/Opcode.h:53:9: note: expanded from macro 'FOR_EACH_OPCODE_ID' FOR_EACH_LLINT_OPCODE_EXTENSION(macro) \ ^ In file included from /Volumes/Data/slave/yosemite-cloop-debug/build/Source/JavaScriptCore/llint/LowLevelInterpreter.cpp:27: In file included from /Volumes/Data/slave/yosemite-cloop-debug/build/Source/JavaScriptCore/llint/LowLevelInterpreter.h:29: In file included from /Volumes/Data/slave/yosemite-cloop-debug/build/Source/JavaScriptCore/bytecode/Opcode.h:34: /Volumes/Data/slave/yosemite-cloop-debug/build/Source/JavaScriptCore/llint/LLIntOpcode.h:49:48: note: expanded from macro 'FOR_EACH_LLINT_OPCODE_EXTENSION' #define FOR_EACH_LLINT_OPCODE_EXTENSION(macro) FOR_EACH_LLINT_NATIVE_HELPER(macro) ^ /Volumes/Data/slave/yosemite-cloop-debug/build/Source/JavaScriptCore/llint/LLIntOpcode.h:43:5: note: expanded from macro 'FOR_EACH_LLINT_NATIVE_HELPER' FOR_EACH_LLINT_NOJIT_NATIVE_HELPER(macro) \ ^ /Volumes/Data/slave/yosemite-cloop-debug/build/Source/JavaScriptCore/llint/LLIntOpcode.h:32:5: note: expanded from macro 'FOR_EACH_LLINT_NOJIT_NATIVE_HELPER' FOR_EACH_CLOOP_BYTECODE_HELPER_ID(macro) ^ In file included from /Volumes/Data/slave/yosemite-cloop-debug/build/Source/JavaScriptCore/llint/LowLevelInterpreter.cpp:27: In file included from /Volumes/Data/slave/yosemite-cloop-debug/build/Source/JavaScriptCore/llint/LowLevelInterpreter.h:29: In file included from /Volumes/Data/slave/yosemite-cloop-debug/build/Source/JavaScriptCore/bytecode/Opcode.h:33: /Volumes/Data/slave/yosemite-cloop-debug/build/WebKitBuild/Debug/DerivedSources/JavaScriptCore/Bytecodes.h:167:11: note: expanded from macro 'FOR_EACH_CLOOP_BYTECODE_HELPER_ID' macro(llint_cloop_did_return_from_js_1, 1) \ ^ In file included from /Volumes/Data/slave/yosemite-cloop-debug/build/Source/JavaScriptCore/llint/LowLevelInterpreter.cpp:27: In file included from /Volumes/Data/slave/yosemite-cloop-debug/build/Source/JavaScriptCore/llint/LowLevelInterpreter.h:29: /Volumes/Data/slave/yosemite-cloop-debug/build/Source/JavaScriptCore/bytecode/Opcode.h:57:40: note: expanded from macro 'OPCODE_ID_ENUM' #define OPCODE_ID_ENUM(opcode, length) opcode, ^ /Volumes/Data/slave/yosemite-cloop-debug/build/Source/JavaScriptCore/bytecode/Opcode.h:45:5: note: expanded from macro 'FOR_EACH_CORE_OPCODE_ID_WITH_EXTENSION' extension__ ^ 2 errors generated.
Attachments
Add attachment proposed patch, testcase, etc.
Michael Saboff
Comment 1 2015-09-15 12:00:25 PDT
Committed r189818: <http://trac.webkit.org/changeset/189818>
Note You need to log in before you can comment on or make changes to this bug.