[PATCH] D90020: [AArch64][SVE] Emit DWARF location expression for SVE stack objects.

[Sander de Smalen via Phabricator via llvm-commits]

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


================
Comment at: llvm/test/CodeGen/AArch64/debug-info-sve-dbg-value.mir:15
+#
+# CHECK2: : DW_OP_breg31 WSP+16, DW_OP_lit16, DW_OP_bregx VG+0, DW_OP_mul, DW_OP_plus)
+# CHECK2: DW_AT_type {{.*}}svint32_t
----------------
david-arm wrote:
> I'm not very familiar with the DWARF dump so I apologise if I misunderstood something, but is this only testing either pure scalable or pure fixed offsets? Looking at fillOffsetExpression I thought there might be offsets made up of byte-sized and vg-sized offsets, i.e. with two DW_OP_plus commands.
This is testing a combination of both, i.e.

  (DW_OP_breg31 WSP+16), 
  ((DW_OP_lit16, DW_OP_bregx VG+0), DW_OP_mul),
  DW_OP_plus

  <=> (SP + 16) + (16 * VG)


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

https://reviews.llvm.org/D90020