[PATCH] D62532: [AIX] Implement function descriptor on SDAG

[Hubert Tong via Phabricator via llvm-commits]

hubert.reinterpretcast added a comment.

I think everyone's comments have been addressed. Thanks again, Xiangling.



================
Comment at: llvm/lib/CodeGen/LLVMTargetMachine.cpp:205
+    if (this->getTargetTriple().isOSAIX()) {
+      // On AIX , we replace GlobalAddressSDNode with MCSymbolSDNode in SDAG for
+      // the callee of a direct function call. To create the MCSymbolXCOFF
----------------
Sorry for missing this earlier. Please remove the space before the comma.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D62532