[Lldb-commits] [lldb] [lldb] Improved formatting of 'register read' command. (PR #188049)

[David Spickett via lldb-commits]

================
@@ -0,0 +1,36 @@
+# REQUIRES: x86
+
+# RUN: %clang_host -g -O0 %S/Inputs/main.c -o %t.out
+# RUN: %lldb -b -o "breakpoint set --name main" \
+# RUN:        -o run \
+# RUN:        -o "register read" \
+# RUN:        %t.out | FileCheck --strict-whitespace %s
+
+# CHECK: (lldb) register read
+# CHECK: General Purpose Registers:
+# CHECK: {{      rax = }}
+# CHECK: {{      rbx = }}
+# CHECK: {{      rcx = }}
+# CHECK: {{      rdx = }}
+# CHECK: {{      rdi = }}
+# CHECK: {{      rsi = }}
+# CHECK: {{      rbp = }}
+# CHECK: {{      rsp = }}
+# CHECK: {{       r8 = }}
+# CHECK: {{       r9 = }}
+# CHECK: {{      r10 = }}
+# CHECK: {{      r11 = }}
+# CHECK: {{      r12 = }}
+# CHECK: {{      r13 = }}
+# CHECK: {{      r14 = }}
+# CHECK: {{      r15 = }}
+# CHECK: {{      rip = }}
+# CHECK: {{   rflags = }}
+# CHECK: {{       cs = }}
+# CHECK: {{       fs = }}
+# CHECK: {{       gs = }}
+# CHECK: {{       ss = }}
+# CHECK: {{  fs_base = }}
+# CHECK: {{  gs_base = }}
+# CHECK: {{       ds = }}
+# CHECK: {{       es = }}
----------------
DavidSpickett wrote:

No need to implement alignment between sets, I just want it to be documented via. the tests that they only align to themselves. It makes the intent clear(er) in future.

If you wanted that yourself and implemented it, I wouldn't mind that either but don't spend extra effort on it. Sets are already visually separated so I think it's fine to align within them.

https://github.com/llvm/llvm-project/pull/188049