[PATCH] D79284: [llvm-objdump][ARM] Print inline relocations when dumping ARM data

[Diana Picus via Phabricator via llvm-commits]

rovka added a comment.

Hi, I like that we're looping in only one place now, but I have 2 nitpicks:

1. The formatting changes are pretty distracting, I think you should commit them separately.
2. The commit message should explain what the problem is and why this change fixes it. Just referencing a bug number, where there haven't even been any discussions, is not very helpful.

Thanks!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79284