[llvm] [llvm-readobj][COFF] Dump .modmeta section (PR #201695)

[James Henderson via llvm-commits]

================
@@ -22,99 +21,95 @@ bool COFFCxxModuleMetadataReader::hasModuleData() const {
 }
 
 Expected<uint32_t> COFFCxxModuleMetadataReader::readModuleID() {
+  if (ModuleIndexWidth != 1 && ModuleIndexWidth != 2 && ModuleIndexWidth != 4)
+    return createStringError("unsupported index width: %d", ModuleIndexWidth);
----------------
jh7370 wrote:

If I'm not mistaken, it would be fine to keep this as a `default` case. The only difference would be that the size check wouldn't fail, but in that case, it's fine, because we'd just get the invalid width error.

https://github.com/llvm/llvm-project/pull/201695