[PATCH] D144472: [InstCombine][debuginfo] Update valueCoversEntireFragment for fixed size fragment and scalable value.

Wed Mar 1 16:00:52 PST 2023

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

================
Comment at: llvm/test/Transforms/InstCombine/dbg-scalable-store-fixed-frag.ll:25
+; CHECK-NEXT:    [[ARR:%.*]] = alloca [4 x i32], align 8
+; CHECK-NEXT:    call void @llvm.dbg.declare(metadata ptr [[ARR]], metadata [[META8]], metadata !DIExpression()), !dbg [[DBG15:![0-9]+]]
+; CHECK-NEXT:    store <vscale x 2 x i32> [[X:%.*]], ptr [[ARR]], align 8
----------------
aprantl wrote:
> This IR isn't well-formed and a future verifier might reject it. Can you created two distinct DISubprograms and local variables for each of the functions?
Thank your good point. I have fixed it.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D144472