<div dir="ltr">There is a file in the repo called git-clang-format.  Make sure that file is on your PATH somewhere, then just run `git clang-format`.  It will only touch lines that are part of your diff, and leave surrounding lines alone.  When making a diff, we only want to clang-format the lines we touched, not the entire files.</div><br><div class="gmail_quote"><div dir="ltr">On Thu, Apr 27, 2017 at 8:56 AM Kamil Rytarowski via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">krytarowski added inline comments.<br>
<br>
<br>
================<br>
Comment at: source/Plugins/ObjectFile/ELF/ObjectFileELF.cpp:302<br>
+  // default 32 or 64 bit arch (without any architecture revision) based on<br>
+  // object file's class.<br>
   if (header.e_type == ET_CORE) {<br>
----------------<br>
joerg wrote:<br>
> Unrelated cosmetic change.<br>
I let clang-format to go and alter minor things. I can run clang-format over original files - commit, and add my diff again.<br>
<br>
<br>
================<br>
Comment at: source/Plugins/Process/elf-core/ProcessElfCore.cpp:463<br>
<br>
-enum { NT_PROCINFO = 1, NT_AUXV, NT_AMD64_REGS = 33, NT_AMD64_FPREGS = 35 };<br>
+enum { NT_PROCINFO = 1, NT_PROCINFO_SIZE = 160, NT_AUXV = 2 };<br>
+<br>
----------------<br>
joerg wrote:<br>
> Either sort them by value or by name, but not randomly<br>
I will split this enum{} into two enums.<br>
<br>
<br>
================<br>
Comment at: source/Plugins/Process/elf-core/ProcessElfCore.cpp:523<br>
+<br>
+  offset += 108;<br>
+  cpi_nlwps = data.GetU32(&offset); /* number of LWPs */<br>
----------------<br>
joerg wrote:<br>
> Can you define a constant for the offset here below instead of a magic number?<br>
I will try to get something to define aliases for these magic numbers.<br>
<br>
<br>
================<br>
Comment at: source/Plugins/Process/elf-core/ProcessElfCore.cpp:844<br>
+  if (m_thread_data.size() != nlwps)<br>
+    return Error("rror parsing NetBSD core(5) notes: Mismatch between the "<br>
+                 "number of LWPs in netbsd_elfcore_procinfo and the number of "<br>
----------------<br>
joerg wrote:<br>
> Typo<br>
OK<br>
<br>
<br>
================<br>
Comment at: source/Plugins/Process/elf-core/ProcessElfCore.cpp:854<br>
+  /* Signal destinated for a particular LWP */<br>
+  else {<br>
+    bool passed = false;<br>
----------------<br>
joerg wrote:<br>
> Move the else to the } and the comment after the {<br>
OK<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="https://reviews.llvm.org/D32149" rel="noreferrer" target="_blank">https://reviews.llvm.org/D32149</a><br>
<br>
<br>
<br>
</blockquote></div>