[PATCH] D60459: SILoadStoreOptimizer pass mischedules s_add,s_addc with interfering s_lshl

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 9 06:44:26 PDT 2019


arsenm requested changes to this revision.
arsenm added a comment.
This revision now requires changes to proceed.

You should use computeRegisterLiveness instead of adding a more naive search for a def, which may not exist.

The testcase can also be a lot simpler (and should probable be MIR)



================
Comment at: test/CodeGen/AMDGPU/scc-add-lshl-addc.ll:79
+
+!0 = !{!"clang version 8.0.0 "}
+!1 = !{!2, !2, i64 0}
----------------
I doubt you need any of this metadata


Repository:
  rL LLVM

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

https://reviews.llvm.org/D60459





More information about the llvm-commits mailing list