[PATCH] D139637: [AArch64][SVE][ISel] Combine dup of load to replicating load
Allen zhong via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jan 19 00:49:13 PST 2023
Allen added inline comments.
================
Comment at: llvm/test/tools/llvm-mca/AArch64/Neoverse/N2-sve-instructions.s:4475
# CHECK-NEXT: 1 6 0.50 * ld1h { z5.h }, p3/z, [x17, x16, lsl #1]
-# CHECK-NEXT: 1 6 0.33 * U ld1rb { z0.b }, p0/z, [x0]
-# CHECK-NEXT: 1 6 0.33 * U ld1rb { z0.d }, p0/z, [x0]
-# CHECK-NEXT: 1 6 0.33 * U ld1rb { z0.h }, p0/z, [x0]
-# CHECK-NEXT: 1 6 0.33 * U ld1rb { z0.s }, p0/z, [x0]
-# CHECK-NEXT: 1 6 0.33 * U ld1rb { z31.b }, p7/z, [sp, #63]
-# CHECK-NEXT: 1 6 0.33 * U ld1rb { z31.d }, p7/z, [sp, #63]
-# CHECK-NEXT: 1 6 0.33 * U ld1rb { z31.h }, p7/z, [sp, #63]
-# CHECK-NEXT: 1 6 0.33 * U ld1rb { z31.s }, p7/z, [sp, #63]
-# CHECK-NEXT: 1 6 0.33 * U ld1rd { z0.d }, p0/z, [x0]
-# CHECK-NEXT: 1 6 0.33 * U ld1rd { z31.d }, p7/z, [sp, #504]
-# CHECK-NEXT: 1 6 0.33 * U ld1rh { z0.d }, p0/z, [x0]
-# CHECK-NEXT: 1 6 0.33 * U ld1rh { z0.h }, p0/z, [x0]
-# CHECK-NEXT: 1 6 0.33 * U ld1rh { z0.s }, p0/z, [x0]
-# CHECK-NEXT: 1 6 0.33 * U ld1rh { z31.d }, p7/z, [sp, #126]
-# CHECK-NEXT: 1 6 0.33 * U ld1rh { z31.h }, p7/z, [sp, #126]
-# CHECK-NEXT: 1 6 0.33 * U ld1rh { z31.s }, p7/z, [sp, #126]
+# CHECK-NEXT: 1 6 0.33 * ld1rb { z0.b }, p0/z, [x0]
+# CHECK-NEXT: 1 6 0.33 * ld1rb { z0.d }, p0/z, [x0]
----------------
Excuse me, I'm curious why the above changes affect the **SideEffects attribute** here. Would you give me some guidance? Thank you.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D139637/new/
https://reviews.llvm.org/D139637
More information about the llvm-commits
mailing list