...
Created attachment 361615 [details] patch
Attachment 361615 [details] did not pass style-queue: ERROR: Source/JavaScriptCore/wasm/WasmAirIRGenerator.cpp:36: Alphabetical sorting problem. [build/include_order] [4] Total errors found: 1 in 2 files If any of these errors are false positives, please file a bug against check-webkit-style.
Created attachment 361616 [details] patch
Comment on attachment 361616 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=361616&action=review r=me > Source/JavaScriptCore/wasm/WasmAirIRGenerator.cpp:1528 > unifyValuesWithBlock(expressionStack, targets[i]->resultForBranch()); Use `for (const ControlData& target : targets)` instead. > Source/JavaScriptCore/wasm/WasmAirIRGenerator.cpp:1539 > + for (size_t i = 0; i < targets.size(); ++i) Ditto. > Source/JavaScriptCore/wasm/WasmAirIRGenerator.cpp:1574 > + params.addLatePath([=] (CCallHelpers& jit) { Is WTFMove(caseJumps) & WTFMove(successorLabels) better?
Thanks for the review. I addressed your comments. landed in: https://trac.webkit.org/changeset/241254/webkit
<rdar://problem/47953543>