[PATCH] D157811: [DAGCombiner] Don't merge stores to an illegal integer type

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Aug 13 18:19:25 PDT 2023


arsenm added inline comments.


================
Comment at: llvm/test/CodeGen/PowerPC/merge_stores_post_legalization_crash.ll:16
+%arrayidx = getelementptr inbounds i64, i64* %arr1, i64 2
+%0 = load i64, i64* %arrayidx, align 8
+store i64 %0, i64* %arr1, align 8
----------------
Test should use opaque pointers 


================
Comment at: llvm/test/CodeGen/PowerPC/merge_stores_post_legalization_crash.ll:25
+
+define dso_local void @test_vec(i64* nocapture %arr1) local_unnamed_addr {
+; CHECK-LABEL: test_vec:
----------------
Can drop attributes and local_unnamed_addr


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D157811



More information about the llvm-commits mailing list