[PATCH] D124071: [NewGVN][LoadCoercion][1/3] Add support for load coercion between store and load instructions

[Konstantina Mitropoulou via Phabricator via llvm-commits]

kmitropoulou added inline comments.


================
Comment at: llvm/test/Transforms/NewGVN/load_coercion_between_store_and_load.ll:377
 ; OLDGVN-NEXT:    store i32 [[V1:%.*]], i32* [[P1:%.*]], align 4
 ; OLDGVN-NEXT:    [[P2:%.*]] = bitcast i32* [[P1]] to i8*
 ; OLDGVN-NEXT:    [[TMP1:%.*]] = trunc i32 [[V1]] to i8
----------------
xbolva00 wrote:
> GVN fails to remove dead instructions?
Yes, these instructions are removed by dce. This sometimes happens with NewGVN as well.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D124071