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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to