[PATCH] D128302: [AArch64][CostModel] Detects that ExtractElement at index is not free in AArch64 when result is used as integer.

Thu Jun 30 14:21:09 PDT 2022

mingmingl marked an inline comment as done.
mingmingl added inline comments.

================
Comment at: llvm/test/Analysis/CostModel/AArch64/kryo.ll:43
+    ret i64 %val
+}
----------------
mingmingl wrote:
> RKSimon wrote:
> > pre-commit the test with trunk codegen and rebase the patch to show the change in cost reporting
> will do in another patch (got to reply first for compile-time question around scanning `uses()`)
created https://reviews.llvm.org/D128945, with this test case, and another test case in `LICM/AArch64/` subdirectory.

Both test cases are going to have diff, and the diff for `LICM/AArch64/extract-element.ll` would be like https://www.diffchecker.com/RyiE4Zn5 (code of trunk on the left, code of this patch on the right)

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D128302/new/

https://reviews.llvm.org/D128302