[PATCH] D75890: [libunwind] Remove __FILE__ and __LINE__ from error reporting

Petr Hosek via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Mar 10 14:47:11 PDT 2020


phosek added a comment.

In D75890#1915493 <https://reviews.llvm.org/D75890#1915493>, @thakis wrote:

> `__FILE__` only expands to an absolute path if you pass an absolute path to clang (ctrl-f "__FILE__" on http://blog.llvm.org/2019/11/deterministic-builds-with-clang-and-lld.html).


Yes, but unfortunately that's what CMake does by default (that is using absolute paths) so we have to deal with that somehow since CMake is still the official build system.

> It feels like this is working around a deficiency in the compiler. Maybe fixing whatever's going wrong in the compiler might be better? (There's also -ffile-file-prefix map, but using that makes your _commandline_ depend on the name of the build dir, and (in clang) `__FILE_NAME__ `.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75890





More information about the cfe-commits mailing list