This makes the warnings reported to be unique and adds test cases.
Details
Details
Diff Detail
Diff Detail
Event Timeline
Comment Actions
LGTM. The inline comment could be deferred if there are a number of places that would need updating.
llvm/tools/llvm-readobj/ELFDumper.cpp | ||
---|---|---|
2092 | Whilst you're changing this line, I recommend fixing the message case to match that from the coding standard. |
llvm/tools/llvm-readobj/ELFDumper.cpp | ||
---|---|---|
2092 | There are 4 test files with multiple places where this warning is reported. I'll fix this in a follow-up. |
llvm/tools/llvm-readobj/ELFDumper.cpp | ||
---|---|---|
2092 | Fixed in rG1daace3fbb30 |
Whilst you're changing this line, I recommend fixing the message case to match that from the coding standard.