Bug 196162 – Refactor clz/ctz and fix getLSBSet.

• Status: RESOLVED
• Resolution: FIXED
• Component: New Bugs

[Keith Miller]

Refactor clz/ctz and fix getLSBSet.

[Keith Miller]

Created attachment 365770 [details]
Patch

[Saam Barati]

Comment on attachment 365770 [details]
Patch

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

> Source/WTF/ChangeLog:12
> +        numeric type. Since these methods work on any type we don't need
> +        to have an, incorrect, implementation of getLSBSet. This patch
> +        also adds getMSBSet as there may be users who want that in the
> +        future.

I'd explain a bit more here.
- The old implementation of getLSBSet was really getMSBSet

> Tools/TestWebKitAPI/Tests/WTF/MathExtras.cpp:435
>  

nit: I think writing out these numbers in binary would make it easier to follow.

[Keith Miller]

Created attachment 365774 [details]
Patch

[Keith Miller]

Created attachment 365775 [details]
Patch

[Keith Miller]

Created attachment 365822 [details]
Patch for landing

[WebKit Commit Bot]

Comment on attachment 365822 [details]
Patch for landing

Clearing flags on attachment: 365822

Committed r243418: <https://trac.webkit.org/changeset/243418>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Radar WebKit Bug Importer]

<rdar://problem/49191697>

[Fujii Hironori]

It breaks x64 Windows builds.

https://build.webkit.org/builders/WinCairo%2064-bit%20WKL%20Debug%20%28Build%29/builds/7601/steps/compile-webkit/logs/stdio

> [21/1544] Building CXX object Tools\TestWebKitAPI\CMakeFiles\TestWTFLib.dir\Tests\WTF\MathExtras.cpp.obj
> FAILED: Tools/TestWebKitAPI/CMakeFiles/TestWTFLib.dir/Tests/WTF/MathExtras.cpp.obj 
> "C:\Program Files (x86)\Microsoft Visual Studio\2017\BuildTools\VC\Tools\MSVC\14.14.26428\bin\Hostx64\x64\cl.exe"  (...) /FdTools\TestWebKitAPI\CMakeFiles\TestWTFLib.dir\ /FS -c ..\..\Tools\TestWebKitAPI\Tests\WTF\MathExtras.cpp
> C:\WebKit-BuildWorker\wincairo-wkl-debug\build\WebKitBuild\Debug\DerivedSources\ForwardingHeaders\wtf/MathExtras.h(636): error C2065: 'number': undeclared identifier

[WebKit Commit Bot]

Re-opened since this is blocked by bug 196191

[Keith Miller]

(In reply to Fujii Hironori from comment #9)
> It breaks x64 Windows builds.
> 
> https://build.webkit.org/builders/WinCairo%2064-
> bit%20WKL%20Debug%20%28Build%29/builds/7601/steps/compile-webkit/logs/stdio
> 
> > [21/1544] Building CXX object Tools\TestWebKitAPI\CMakeFiles\TestWTFLib.dir\Tests\WTF\MathExtras.cpp.obj
> > FAILED: Tools/TestWebKitAPI/CMakeFiles/TestWTFLib.dir/Tests/WTF/MathExtras.cpp.obj 
> > "C:\Program Files (x86)\Microsoft Visual Studio\2017\BuildTools\VC\Tools\MSVC\14.14.26428\bin\Hostx64\x64\cl.exe"  (...) /FdTools\TestWebKitAPI\CMakeFiles\TestWTFLib.dir\ /FS -c ..\..\Tools\TestWebKitAPI\Tests\WTF\MathExtras.cpp
> > C:\WebKit-BuildWorker\wincairo-wkl-debug\build\WebKitBuild\Debug\DerivedSources\ForwardingHeaders\wtf/MathExtras.h(636): error C2065: 'number': undeclared identifier

Whoops, I think it's a simple fix. I forgot to change the variable name for windows...

[Keith Miller]

Should hopefully be fixed with: https://trac.webkit.org/changeset/243429