[PATCH] D72736: [AIX] Add improved interface for retrieving load module paths

Hubert Tong via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri May 29 13:06:57 PDT 2020


hubert.reinterpretcast added inline comments.


================
Comment at: llvm/lib/Support/Unix/Path.inc:243
+#elif defined(_AIX)
+  return getLoadModuleFilenameForFunction(MainAddr)->c_str();
 #elif defined(__linux__) || defined(__CYGWIN__) || defined(__gnu_hurd__)
----------------
hubert.reinterpretcast wrote:
> The "expected" here is not checked for doing the error handling (soft-fail in this case). It appears `LLVM_ENABLE_ABI_BREAKING_CHECKS` would help to catch these.
Ping @Jonathan.Crowther.


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

https://reviews.llvm.org/D72736





More information about the cfe-commits mailing list