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

[Sean Fertile via Phabricator via llvm-commits]

sfertile added inline comments.


================
Comment at: llvm/lib/MC/MCSectionXCOFF.cpp:27
+  default:
+    llvm_unreachable("Unsupported storage-mapping class");
+  }
----------------
xingxue wrote:
> 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.
I'm OK with `assert` as well.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D66724