Opcode.h(159,27): warning: adding 'unsigned int' to a string does not append to the string [-Wstring-plus-int]

[Fujii Hironori]

Clang 8.0.0 reports the following compilation warning.

> [1229/6361] Building CXX object Source\JavaScriptCore\CMakeFiles\LLIntOffsetsExtractor.dir\llint\LLIntOffsetsExtractor.cpp.obj
> In file included from ..\..\Source\JavaScriptCore\llint\LLIntOffsetsExtractor.cpp:31:
> In file included from DerivedSources\JavaScriptCore\BytecodeStructs.h:32:
> In file included from ..\..\Source\JavaScriptCore\bytecode\BytecodeDumper.h:29:
> In file included from ..\..\Source\JavaScriptCore\bytecode/CallLinkInfo.h:31:
> In file included from ..\..\Source\JavaScriptCore\jit\PolymorphicCallStubRoutine.h:30:
> In file included from ..\..\Source\JavaScriptCore\bytecode\CallEdge.h:28:
> In file included from ..\..\Source\JavaScriptCore\bytecode/CallVariant.h:28:
> In file included from ..\..\Source\JavaScriptCore\runtime\ExecutableBaseInlines.h:28:
> In file included from ..\..\Source\JavaScriptCore\runtime/ExecutableBase.h:34:
> In file included from ..\..\Source\JavaScriptCore\bytecode\UnlinkedCodeBlock.h:34:
> In file included from ..\..\Source\JavaScriptCore\bytecode/InstructionStream.h:29:
> In file included from ..\..\Source\JavaScriptCore\bytecode/Instruction.h:28:
> ..\..\Source\JavaScriptCore\bytecode/Opcode.h(159,27): warning: adding 'unsigned int' to a string does not append to the string [-Wstring-plus-int]
>     return PADDING_STRING + PADDING_STRING_LENGTH - pad;
>            ~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~
> ..\..\Source\JavaScriptCore\bytecode/Opcode.h(159,27): note: use array indexing to silence this warning
>     return PADDING_STRING + PADDING_STRING_LENGTH - pad;
>                           ^
>            &              [                      ]
> 1 warning generated.


> clang version 8.0.0 (tags/RELEASE_800/final)
> Target: x86_64-pc-windows-msvc
> Thread model: posix

[Fujii Hironori]

Created attachment 366158 [details]
Patch

[Fujii Hironori]

I have questions.
* How to test this code?
* Is there a port enabling OPCODE_STATS?

[Fujii Hironori]

Created attachment 366159 [details]
Patch

[Fujii Hironori]

If define ENABLE_OPCODE_STATS 1, can't compile due to other issues.

> In file included from DerivedSources\JavaScriptCore\unified-sources\UnifiedSource-f0a787a9-8.cpp:6:
> ..\..\Source\JavaScriptCore\bytecode/Opcode.cpp(97,38): error: no template named 'pair'; did you mean 'std::pair'?
>     std::pair<int, int> leftPair = *(pair<int, int>*) left;
>                                      ^~~~
>                                      std::pair
> C:\Program Files (x86)\Microsoft Visual Studio\2017\Professional\VC\Tools\MSVC\14.14.26428\include\utility(93,9): note: 'std::pair' declared here
>         struct pair
>                ^
> In file included from DerivedSources\JavaScriptCore\unified-sources\UnifiedSource-f0a787a9-8.cpp:6:
> ..\..\Source\JavaScriptCore\bytecode/Opcode.cpp(99,39): error: no template named 'pair'; did you mean 'std::pair'?
>     std::pair<int, int> rightPair = *(pair<int, int>*) right;
>                                       ^~~~
>                                       std::pair
> C:\Program Files (x86)\Microsoft Visual Studio\2017\Professional\VC\Tools\MSVC\14.14.26428\include\utility(93,9): note: 'std::pair' declared here
>         struct pair
>                ^
> 2 errors generated.

[Fujii Hironori]

Comment on attachment 366159 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=366159&action=review

> Source/JavaScriptCore/bytecode/Opcode.cpp:63
> +    return &padding[stringLength - width];

out-of-bound access occurs if the opcode name is longer than width.

[Fujii Hironori]

Created attachment 366162 [details]
Patch

[Fujii Hironori]

(In reply to Fujii Hironori from comment #2)
> I have questions.
> * How to test this code?

Define ENABLE_OPCODE_STATS and execute jsc.exe and type "quit()".

> * Is there a port enabling OPCODE_STATS?

No.

[Fujii Hironori]

Committed r243633: <https://trac.webkit.org/changeset/243633>

[Radar WebKit Bug Importer]

<rdar://problem/49405319>