[PATCH] D98114: [Loads] Forward constant vector store to load of first element

[Sanjay Patel via Phabricator via llvm-commits]

spatel accepted this revision.
spatel added a comment.
This revision is now accepted and ready to land.

LGTM - might want to add a test with constant expression, so we know we're not going to die on something like:

  define i32 @vec_store_load_first(i32* %p) {
    %p2 = bitcast i32* %p to <2 x i32>*
    store <2 x i32> bitcast (i64 ptrtoint (i32 (i32*)* @vec_store_load_first to i64) to <2 x i32>), <2 x i32>* %p2, align 8
    %load = load i32, i32* %p, align 4
    ret i32 %load
  }


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D98114