From 36fdde818ab12e094ef8799fae4ab355d673095e Mon Sep 17 00:00:00 2001 From: Gerrit User 1347058 <1347058@3ce6091f-6c88-37e8-8c75-72f92ae8dfba> Date: Wed, 29 Apr 2026 13:11:04 -0700 Subject: [PATCH] Create patch set 8 Change has been successfully rebased and submitted as f2ac6db694d1e5b0af1d7b05dc431e0e455fe228 6 is the latest approved patch-set. The change was submitted with unreviewed changes in the following files: ``` The name of the file: source/row_win.cc Insertions: 4, Deletions: 4. @@ -315,15 +315,15 @@ #endif #ifdef HAS_ARGBTOUVMATRIXROW_AVX2 -LIBYUV_TARGET_AVX2 +LIBYUV_TARGET_AVX2 __attribute__((no_sanitize("cfi-icall"))) void ARGBToUVMatrixRow_AVX2(const uint8_t* src_argb, int src_stride_argb, uint8_t* dst_u, uint8_t* dst_v, int width, const struct ArgbConstants* c) { - __m256i ymm_u = _mm256_broadcastsi128_si256(_mm_loadu_si128((const __m128i*)(c->kRGBToY + 32))); // kRGBToU is offset 32 - __m256i ymm_v = _mm256_broadcastsi128_si256(_mm_loadu_si128((const __m128i*)(c->kRGBToY + 64))); // kRGBToV is offset 64 + __m256i ymm_u = _mm256_broadcastsi128_si256(_mm_loadu_si128((const __m128i*)c->kRGBToU)); + __m256i ymm_v = _mm256_broadcastsi128_si256(_mm_loadu_si128((const __m128i*)c->kRGBToV)); __m256i ymm_0101 = _mm256_set1_epi16(0x0101); __m256i ymm_shuf = _mm256_setr_epi8(0, 4, 1, 5, 2, 6, 3, 7, 8, 12, 9, 13, 10, 14, 11, 15, 0, 4, 1, 5, 2, 6, 3, 7, 8, 12, 9, 13, 10, 14, 11, 15); @@ -362,7 +362,7 @@ ymm0 = _mm256_hadd_epi16(ymm0, ymm1); ymm0 = _mm256_permute4x64_epi64(ymm0, 0xd8); - ymm0 = _mm256_sub_epi16(ymm0, ymm_8000); + ymm0 = _mm256_sub_epi16(ymm_8000, ymm0); ymm0 = _mm256_srli_epi16(ymm0, 8); ymm0 = _mm256_packus_epi16(ymm0, ymm0); ``` Patch-set: 8 Patch-set-description: Rebase Subject: RAWToNV21 using SME, SVE, I8MM or Neon Status: merged Commit: f2ac6db694d1e5b0af1d7b05dc431e0e455fe228 Tag: autogenerated:gerrit:merged Groups: 9bf683329936a9013e328084bcfde4f3292093a6 Label: Code-Review=+1, a716f44eaa00d97e44a79490569b9d40867e9015 Gerrit User 1001250 <1001250@3ce6091f-6c88-37e8-8c75-72f92ae8dfba> Label: Code-Review=+1, 35c32ec816b0148f39c82fa69d424dc135d53448 Gerrit User 1002030 <1002030@3ce6091f-6c88-37e8-8c75-72f92ae8dfba> Label: Commit-Queue=+2, 320847c067e6c6da1c192f38c431fdcd325a1a20 Gerrit User 1115898 <1115898@3ce6091f-6c88-37e8-8c75-72f92ae8dfba> Label: SUBM=+1, ff533767075d2d82b86db9e164c71d3aad62727c Submission-id: 7802432 Submitted-with: OK Submitted-with: Rule-Name: gerrit~DefaultSubmitRule Submitted-with: MAY: SLSA-Policy-Verified Submitted-with: MAY: Auto-Submit Submitted-with: MAY: Code-Coverage Submitted-with: MAY: Code-Review: Gerrit User 1001250 <1001250@3ce6091f-6c88-37e8-8c75-72f92ae8dfba> Submitted-with: MAY: Commit-Queue: Gerrit User 1115898 <1115898@3ce6091f-6c88-37e8-8c75-72f92ae8dfba> Submitted-with: MAY: Feels Submitted-with: MAY: Owners-Override Submitted-with: MAY: Lint Submitted-with: MAY: Bot-Commit Submitted-with: OK Submitted-with: Rule-Name: Code-Owners Base: b438739c8b08eba2c562a62ea5961f6215d525ed Ours: 9bf683329936a9013e328084bcfde4f3292093a6 Theirs: b438739c8b08eba2c562a62ea5961f6215d525ed Merge-Strategy: simple-two-way-in-core Contains-Conflicts: false --- 9bf683329936a9013e328084bcfde4f3292093a6 | 209 +++++++++++++++++++++++ 1 file changed, 209 insertions(+) create mode 100644 9bf683329936a9013e328084bcfde4f3292093a6 diff --git a/9bf683329936a9013e328084bcfde4f3292093a6 b/9bf683329936a9013e328084bcfde4f3292093a6 new file mode 100644 index 000000000..9fe2facce --- /dev/null +++ b/9bf683329936a9013e328084bcfde4f3292093a6 @@ -0,0 +1,209 @@ +{ + "comments": [], + "submitRequirementResults": [ + { + "submitRequirement": { + "name": "Admin-Approval", + "description": { + "value": "Changes to refs/meta/config must be approved by a git admin." + }, + "applicabilityExpression": { + "value": { + "expressionString": "branch:refs/meta/config" + } + }, + "submittabilityExpression": { + "expressionString": "(uploaderin:mdb/chrome-git-admins OR label:Code-Review\u003dMAX,group\u003dmdb/chrome-git-admins) AND label:Code-Review\u003dMAX,user\u003dnon_uploader AND -label:Code-Review\u003dMIN" + }, + "overrideExpression": { + "value": { + "expressionString": "label:Bot-Commit\u003d+1 AND -label:Code-Review\u003dMIN" + } + }, + "allowOverrideInChildProjects": false + }, + "applicabilityExpressionResult": { + "value": {"expression":{"expressionString":"branch:refs/meta/config"},"status":"FAIL","errorMessage":{"value":null},"passingAtoms":[],"failingAtoms":["branch:refs/meta/config"],"atomExplanations":{"value":{"branch:refs/meta/config":""}}} + }, + "submittabilityExpressionResult": { + "value": {"expression":{"expressionString":"(uploaderin:mdb/chrome-git-admins OR label:Code-Review=MAX,group=mdb/chrome-git-admins) AND label:Code-Review=MAX,user=non_uploader AND -label:Code-Review=MIN"},"status":"NOT_EVALUATED","errorMessage":{"value":null},"passingAtoms":[],"failingAtoms":[],"atomExplanations":{"value":null}} + }, + "overrideExpressionResult": { + "value": {"expression":{"expressionString":"label:Bot-Commit=+1 AND -label:Code-Review=MIN"},"status":"NOT_EVALUATED","errorMessage":{"value":null},"passingAtoms":[],"failingAtoms":[],"atomExplanations":{"value":null}} + }, + "patchSetCommitId": "9bf683329936a9013e328084bcfde4f3292093a6", + "legacy": { + "value": false + }, + "forced": {}, + "hidden": {} + }, + { + "submitRequirement": { + "name": "Code-Review", + "description": { + "value": "If the author is a libyuv committer, code review is required from one other committer. Otherwise, code review is required from two committers." + }, + "applicabilityExpression": { + "value": { + "expressionString": "-branch:refs/meta/config" + } + }, + "submittabilityExpression": { + "expressionString": "((uploaderin:cria/project-libyuv-committers AND label:Code-Review\u003dMAX,user\u003dnon_uploader) OR label:Code-Review\u003dMAX,count\u003e\u003d2) AND -label:Code-Review\u003dMIN" + }, + "overrideExpression": { + "value": { + "expressionString": "label:Bot-Commit\u003d+1 AND -label:Code-Review\u003dMIN" + } + }, + "allowOverrideInChildProjects": true + }, + "applicabilityExpressionResult": { + "value": {"expression":{"expressionString":"-branch:refs/meta/config"},"status":"PASS","errorMessage":{"value":null},"passingAtoms":[],"failingAtoms":["branch:refs/meta/config"],"atomExplanations":{"value":{"branch:refs/meta/config":""}}} + }, + "submittabilityExpressionResult": { + "value": {"expression":{"expressionString":"((uploaderin:cria/project-libyuv-committers AND label:Code-Review=MAX,user=non_uploader) OR label:Code-Review=MAX,count>=2) AND -label:Code-Review=MIN"},"status":"PASS","errorMessage":{"value":null},"passingAtoms":["label:Code-Review=MAX,user=non_uploader","uploaderin:cria/project-libyuv-committers","label:Code-Review=MAX,count>=2"],"failingAtoms":["label:Code-Review=MIN"],"atomExplanations":{"value":{"label:Code-Review=MIN":"","label:Code-Review=MAX,user=non_uploader":"","uploaderin:cria/project-libyuv-committers":"","label:Code-Review=MAX,count>=2":""}}} + }, + "overrideExpressionResult": { + "value": {"expression":{"expressionString":"label:Bot-Commit=+1 AND -label:Code-Review=MIN"},"status":"FAIL","errorMessage":{"value":null},"passingAtoms":[],"failingAtoms":["label:Bot-Commit=+1","label:Code-Review=MIN"],"atomExplanations":{"value":{"label:Bot-Commit=+1":"","label:Code-Review=MIN":""}}} + }, + "patchSetCommitId": "9bf683329936a9013e328084bcfde4f3292093a6", + "legacy": { + "value": false + }, + "forced": {}, + "hidden": {} + }, + { + "submitRequirement": { + "name": "No-Unresolved-Comments", + "description": { + "value": "Changes that have unresolved comments are not submittable. Unless overridden with Unresolved-Comment-Reason: \u003creason\u003e line in change message." + }, + "applicabilityExpression": { + "value": { + "expressionString": "has:unresolved" + } + }, + "submittabilityExpression": { + "expressionString": "-has:unresolved" + }, + "overrideExpression": { + "value": { + "expressionString": "message:\"^.*Unresolved-Comment-Reason:.*\"" + } + }, + "allowOverrideInChildProjects": false + }, + "applicabilityExpressionResult": { + "value": {"expression":{"expressionString":"has:unresolved"},"status":"FAIL","errorMessage":{"value":null},"passingAtoms":[],"failingAtoms":["has:unresolved"],"atomExplanations":{"value":{"has:unresolved":""}}} + }, + "submittabilityExpressionResult": { + "value": {"expression":{"expressionString":"-has:unresolved"},"status":"NOT_EVALUATED","errorMessage":{"value":null},"passingAtoms":[],"failingAtoms":[],"atomExplanations":{"value":null}} + }, + "overrideExpressionResult": { + "value": {"expression":{"expressionString":"message:\"^.*Unresolved-Comment-Reason:.*\""},"status":"NOT_EVALUATED","errorMessage":{"value":null},"passingAtoms":[],"failingAtoms":[],"atomExplanations":{"value":null}} + }, + "patchSetCommitId": "9bf683329936a9013e328084bcfde4f3292093a6", + "legacy": { + "value": false + }, + "forced": {}, + "hidden": {} + }, + { + "submitRequirement": { + "name": "Review-Enforcement", + "description": { + "value": "Two trusted contributors must approve the change. Uploading the change or voting positively on Code-Review count as approval. For Google employees, using a trusted device is required (http://go/gerrit-onboarding#review-enforcement). For non-Google employees, ReAuth is required (https://chromium.googlesource.com/chromium/src/+/HEAD/docs/gerrit_reauth.md)." + }, + "applicabilityExpression": { + "value": { + "expressionString": "is:review-enforced_gerrit" + } + }, + "submittabilityExpression": { + "expressionString": "is:review-enforcement-satisfied_gerrit" + }, + "overrideExpression": {}, + "allowOverrideInChildProjects": false + }, + "applicabilityExpressionResult": { + "value": {"expression":{"expressionString":"is:review-enforced_gerrit"},"status":"PASS","errorMessage":{"value":null},"passingAtoms":["is:review-enforced_gerrit"],"failingAtoms":[],"atomExplanations":{"value":{"is:review-enforced_gerrit":""}}} + }, + "submittabilityExpressionResult": { + "value": {"expression":{"expressionString":"is:review-enforcement-satisfied_gerrit"},"status":"PASS","errorMessage":{"value":null},"passingAtoms":["is:review-enforcement-satisfied_gerrit"],"failingAtoms":[],"atomExplanations":{"value":{"is:review-enforcement-satisfied_gerrit":"Required approval count satisfied"}}} + }, + "overrideExpressionResult": {}, + "patchSetCommitId": "9bf683329936a9013e328084bcfde4f3292093a6", + "legacy": { + "value": false + }, + "forced": {}, + "hidden": {} + }, + { + "submitRequirement": { + "name": "SLSA-Policy-Verified", + "description": { + "value": "Verify that binaries are authorized and free from tampering at the configured level." + }, + "applicabilityExpression": { + "value": { + "expressionString": "is:bcid-exit-gate-applicable_gerrit" + } + }, + "submittabilityExpression": { + "expressionString": "label:SLSA-Policy-Verified\u003dMAX\u0026user\u003ddevtools-gerritcodereview-exitgate" + }, + "overrideExpression": {}, + "allowOverrideInChildProjects": false + }, + "applicabilityExpressionResult": { + "value": {"expression":{"expressionString":"is:bcid-exit-gate-applicable_gerrit"},"status":"FAIL","errorMessage":{"value":null},"passingAtoms":[],"failingAtoms":["is:bcid-exit-gate-applicable_gerrit"],"atomExplanations":{"value":{"is:bcid-exit-gate-applicable_gerrit":""}}} + }, + "submittabilityExpressionResult": { + "value": {"expression":{"expressionString":"label:SLSA-Policy-Verified=MAX&user=devtools-gerritcodereview-exitgate"},"status":"NOT_EVALUATED","errorMessage":{"value":null},"passingAtoms":[],"failingAtoms":[],"atomExplanations":{"value":null}} + }, + "overrideExpressionResult": {}, + "patchSetCommitId": "9bf683329936a9013e328084bcfde4f3292093a6", + "legacy": { + "value": false + }, + "forced": {}, + "hidden": {} + }, + { + "submitRequirement": { + "name": "Verified", + "description": { + "value": "Submit requirement for the \u0027Verified\u0027 label" + }, + "applicabilityExpression": { + "value": { + "expressionString": "is:false" + } + }, + "submittabilityExpression": { + "expressionString": "is:true" + }, + "overrideExpression": {}, + "allowOverrideInChildProjects": true + }, + "applicabilityExpressionResult": { + "value": {"expression":{"expressionString":"is:false"},"status":"FAIL","errorMessage":{"value":null},"passingAtoms":[],"failingAtoms":["is:false"],"atomExplanations":{"value":{"is:false":""}}} + }, + "submittabilityExpressionResult": { + "value": {"expression":{"expressionString":"is:true"},"status":"NOT_EVALUATED","errorMessage":{"value":null},"passingAtoms":[],"failingAtoms":[],"atomExplanations":{"value":null}} + }, + "overrideExpressionResult": {}, + "patchSetCommitId": "9bf683329936a9013e328084bcfde4f3292093a6", + "legacy": { + "value": false + }, + "forced": {}, + "hidden": {} + } + ] +} \ No newline at end of file