[Lldb-commits] [PATCH] D25832: Minidump plugin: Adding x86_32 register context converter

[Pavel Labath via lldb-commits]

labath added inline comments.


================
Comment at: unittests/Process/minidump/MinidumpParserTest.cpp:340
   std::map<uint64_t, uint64_t> reg_values;
 
+  reg_values[lldb_rax_x86_64] = 0x0000000000000000;
----------------
amccarth wrote:
> It seems a shame to remove the `// clang-format off` comment for this section.  The aligned version is easier to read.
I asked Dimitar to remove that. My feeling is that the directive should be reserved for just the extreme cases, where we would end up with a completely unreadable mess without it (like the current register contexts... ugh...). It's not a particularly strong feeling but e.g., there are zero occurrences of this in llvm.


https://reviews.llvm.org/D25832