Update patch set 4

Patch Set 4:

(1 comment)

Patch-set: 4
CC: Gerrit User 1115898 <1115898@3ce6091f-6c88-37e8-8c75-72f92ae8dfba>
Attention: {"person_ident":"Gerrit User 1234564 \u003c1234564@3ce6091f-6c88-37e8-8c75-72f92ae8dfba\u003e","operation":"ADD","reason":"\u003cGERRIT_ACCOUNT_1115898\u003e replied on the change"}
This commit is contained in:
Gerrit User 1115898 2026-09-22 13:46:54 -07:00 committed by Gerrit Code Review
parent b6e255fb93
commit d8eee7eb70

View File

@ -0,0 +1,27 @@
{
"comments": [
{
"unresolved": true,
"key": {
"uuid": "e31fbc79_3c0ffade",
"filename": "source/scale_gcc.cc",
"patchSetId": 4
},
"lineNbr": 1803,
"author": {
"id": 1115898
},
"writtenOn": "2026-09-22T20:46:54Z",
"side": 1,
"message": "Suggest the name ScaleUVAddCols3_AVX2\ncleanest match to third_party/libyuv/files/include/libyuv/scale_row.h:815 and third_party/libyuv/files/source/scale.cc:819) \n • AddCols already denotes the box-filter column accumulator pass (the counterpart to ScaleAddRow), and 3 replaces the generic 1/2 suffix to mean \"sum 3 columns\": \n • Kernel: ScaleUVAddCols3_AVX2 (macro: HAS_SCALEUVADDCOLS3_AVX2)\n • Tail wrapper: ScaleUVAddCols3_Any_AVX2 (instead of ScaleUVAddCols1_Box3_AVX2_C)\n • C reference (if added): ScaleUVAddCols3_C",
"range": {
"startLine": 1803,
"startChar": 5,
"endLine": 1803,
"endChar": 30
},
"revId": "084140ebb3bbd1c350293b895e85db2d0c797f2b",
"serverId": "3ce6091f-6c88-37e8-8c75-72f92ae8dfba"
}
]
}