[PATCH] D75866: [AIX] supporting the visibility attribute for aix assembly

[Digger via Phabricator via llvm-commits]

DiggerLin marked 10 inline comments as done.
DiggerLin added inline comments.


================
Comment at: llvm/lib/MC/MCAsmStreamer.cpp:833
+  default:
+    break;
+  case MCSA_Hidden:
----------------
hubert.reinterpretcast wrote:
> This should be the case for `MCSA_Invalid`, but for other values, we should call `llvm_unreachable` indicating that we got an unexpected value for `Visibility`.
I do not think the MCSA_Invalid will come to here. if come to the function. it should be  llvm_unreachable too.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75866