[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