[libcxx-commits] [PATCH] D145225: [libunwind][RISC-V] Rewrite testcase with C as possible.

[Fangrui Song via Phabricator via libcxx-commits]

MaskRay accepted this revision.
MaskRay added inline comments.
This revision is now accepted and ready to land.


================
Comment at: libunwind/test/unwind_scalable_vectors.pass.cpp:29
+// Check correct unwinding of frame with VLENB-sized objects (vector registers).
+void foo() {
+  __rvv_int32m1_t v;
----------------
Add `noinline` and make it `static`


================
Comment at: libunwind/test/unwind_scalable_vectors.pass.cpp:31
+  __rvv_int32m1_t v;
+  asm volatile("#" : "=vr"(v)); // Dummy inline asm to def v.
+  stepper();                    // def-use of v has cross the function, so that
----------------
`"#"` can be `""`


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D145225