[PATCH] D16269: [llvm-readobj][ELF] Teach llvm-readobj to show arch specific ELF section's flags
Simon Atanasyan via llvm-commits
llvm-commits at lists.llvm.org
Sun Jan 17 14:05:27 PST 2016
atanasyan added inline comments.
================
Comment at: tools/llvm-readobj/ELFDumper.cpp:1161
@@ +1160,3 @@
+ std::end(ElfX86_64SectionFlags));
+ break;
+ }
----------------
davide wrote:
> Missing default case I guess?
By default we do not need to add any additional flags to the `SectionFlags` container. Is it okay to put the following a bit redundant code here?
```
default:
break;
```
Repository:
rL LLVM
http://reviews.llvm.org/D16269
More information about the llvm-commits
mailing list