[PATCH] D66724: [AIX]Emit function descriptor csect in assembly

[Xing Xue via Phabricator via llvm-commits]

xingxue added inline comments.


================
Comment at: llvm/lib/MC/MCSectionXCOFF.cpp:27
+  default:
+    llvm_unreachable("Unsupported storage-mapping class");
+  }
----------------
jasonliu wrote:
> sfertile wrote:
> > A `report_fatal_error` is probably more appropriate until we have added all the storage mapping classes.
> This marked done, but haven't really addressed. 
In my committed patch https://reviews.llvm.org/D66154, the llvm_unreachable() here was replaced with assert() under Hubert's suggestion, which makes sense to me.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D66724