Bug 219158 – [JSC] Build failed due to unknown values in LLIntDesiredOffsets.h

• Status: RESOLVED
• Resolution: FIXED
• Component: Tools / Tests

[Adrian Perez]

After updating from the repository the following happens:


In file included from ../Source/JavaScriptCore/llint/LLIntOffsetsExtractor.cpp:103:
DerivedSources/JavaScriptCore/LLIntDesiredOffsets.h:610:56: error: use of undeclared identifier 'WasmI32Extend16S_m_dst_index'; did you mean 'WasmI32Load16S_m_dst_index'?
constexpr int64_t constValue607 = static_cast<int64_t>(WasmI32Extend16S_m_dst_index);
                                                       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~
                                                       WasmI32Load16S_m_dst_index
DerivedSources/JavaScriptCore/BytecodeIndices.h:1053:16: note: 'WasmI32Load16S_m_dst_index' declared here
const unsigned WasmI32Load16S_m_dst_index = 0;
               ^
In file included from ../Source/JavaScriptCore/llint/LLIntOffsetsExtractor.cpp:103:
DerivedSources/JavaScriptCore/LLIntDesiredOffsets.h:611:56: error: use of undeclared identifier 'WasmI32Extend16S_m_operand_index'
constexpr int64_t constValue608 = static_cast<int64_t>(WasmI32Extend16S_m_operand_index);
                                                       ^
DerivedSources/JavaScriptCore/LLIntDesiredOffsets.h:612:56: error: use of undeclared identifier 'WasmI32Extend8S_m_dst_index'; did you mean 'WasmI32Load8S_m_dst_index'?
constexpr int64_t constValue609 = static_cast<int64_t>(WasmI32Extend8S_m_dst_index);
                                                       ^~~~~~~~~~~~~~~~~~~~~~~~~~~
                                                       WasmI32Load8S_m_dst_index
DerivedSources/JavaScriptCore/BytecodeIndices.h:1047:16: note: 'WasmI32Load8S_m_dst_index' declared here
const unsigned WasmI32Load8S_m_dst_index = 0;
               ^
In file included from ../Source/JavaScriptCore/llint/LLIntOffsetsExtractor.cpp:103:
DerivedSources/JavaScriptCore/LLIntDesiredOffsets.h:613:56: error: use of undeclared identifier 'WasmI32Extend8S_m_operand_index'
constexpr int64_t constValue610 = static_cast<int64_t>(WasmI32Extend8S_m_operand_index);
                                                       ^
DerivedSources/JavaScriptCore/LLIntDesiredOffsets.h:1103:57: error: use of undeclared identifier 'wasm_i32_extend16_s_length'; did you mean 'wasm_i32_load16_s_length'?
constexpr int64_t constValue1100 = static_cast<int64_t>(wasm_i32_extend16_s_length);
                                                        ^~~~~~~~~~~~~~~~~~~~~~~~~~
                                                        wasm_i32_load16_s_length
../Source/JavaScriptCore/bytecode/Opcode.h:88:22: note: 'wasm_i32_load16_s_length' declared here
    FOR_EACH_WASM_ID(OPCODE_ID_LENGTHS);
                     ^
In file included from ../Source/JavaScriptCore/llint/LLIntOffsetsExtractor.cpp:103:
DerivedSources/JavaScriptCore/LLIntDesiredOffsets.h:1104:57: error: use of undeclared identifier 'wasm_i32_extend8_s_length'; did you mean 'wasm_i32_load8_s_length'?
constexpr int64_t constValue1101 = static_cast<int64_t>(wasm_i32_extend8_s_length);
                                                        ^~~~~~~~~~~~~~~~~~~~~~~~~
                                                        wasm_i32_load8_s_length
../Source/JavaScriptCore/bytecode/Opcode.h:88:22: note: 'wasm_i32_load8_s_length' declared here
    FOR_EACH_WASM_ID(OPCODE_ID_LENGTHS);
                     ^
6 errors generated.

[Adrian Perez]

Doing a clean build works, so I suspect this is caused by some
dependency that should trigger the rebuild of LLIntDesiredOffsets.h
not being listed in the corresponding CMake custom_command().

[Adrian Perez]

Created attachment 414579 [details]
Patch

[Michael Catanzaro]

Comment on attachment 414579 [details]
Patch

Grrr, now this is good proof that I am not smart. I found one file missing in bug #219043 but managed to miss four others. Nice. :P

[EWS]

Committed r270036: <https://trac.webkit.org/changeset/270036>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 414579 [details].

[Radar WebKit Bug Importer]

<rdar://problem/71589994>