[PATCH] D58930: Add XCOFF triple object format type for AIX

Jason Liu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Mar 5 12:52:01 PST 2019


jasonliu marked an inline comment as done.
jasonliu added inline comments.


================
Comment at: lldb/source/Plugins/Process/gdb-remote/GDBRemoteCommunicationClient.cpp:2079
+          if (log)
+            log->Printf("sorry: unimplemented for XCOFF");
+          return false;
----------------
JDevlieghere wrote:
> jasonliu wrote:
> > apaprocki wrote:
> > > No need to be `sorry:` :) This should probably just say `error: XCOFF is unimplemented` to be more direct in case anything is expecting "error:" in the output.
> > Sure. Will address in next revision.
> Just bundle this with the WASM case, the error message is correct for both.
I think they are different. 
The error message for WASM seems to suggest that it will never ever get supported on WASM. 
But it is not the case for XCOFF, we want to indicate that it is not implemented yet.  


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D58930





More information about the cfe-commits mailing list