DavidSpickett added a comment.

I realised my mistake, I thought this was adding a new core file but in fact 
it's using the one you added for the register tests. So now the outfile is 
there the test passes.



================
Comment at: 
lldb/test/API/functionalities/unwind/aarch64_unwind_pac/TestAArch64UnwindPAC.py:41
+            self.assertEqual(frame.GetFunctionName(), backtrace[frame_idx])
+                       # Check line number for functions in main.c
+            if (frame_idx < 4):
----------------
Strange indent here


================
Comment at: lldb/test/API/functionalities/unwind/aarch64_unwind_pac/main.c:5
+// To enable PAC return address signing compile with following clang arguments:
+// -march=armv8.5-a -mbranch-protection=pac-ret+leaf
+
----------------
I'd rather use `armv8.3-a` here just for correctness.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99944

_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to