[PATCH] D91194: Fixes an issue where adding a relative path to the C include Directories via the C_INCLUDE_DIRS compile time option, there is a "/" ommitted to join the sysroot and the supplied relati

2020-11-10 Thread Lance Fredrickson via Phabricator via cfe-commits
lancethepants created this revision. Herald added subscribers: cfe-commits, jgravelle-google, sbc100, dschuff. Herald added a project: clang. lancethepants requested review of this revision. Herald added a subscriber: aheejin. ...the relative path of course does not start with "/", so the "/" is n

[PATCH] D93164: [AST] Add generator for source location introspection

2022-01-18 Thread Lance Fredrickson via Phabricator via cfe-commits
lancethepants added a comment. In D93164#3048130 , @lancethepants wrote: > In D93164#2653067 , @mgorny wrote: > >> This change breaks cross-compilation now, as it tries running an executable >> built for the targe

[PATCH] D93164: [AST] Add generator for source location introspection

2022-01-19 Thread Lance Fredrickson via Phabricator via cfe-commits
lancethepants added a comment. > @lancethepants @smeenai thanks for the pointers! Unfortunately, it still > doesn't work for me after passing -DCMAKE_SYSTEM_NAME="Linux". Passing that > option did change the CMake output, so it's definitely recognized at least > (the messages about which saniti

[PATCH] D93164: [AST] Add generator for source location introspection

2021-10-07 Thread Lance Fredrickson via Phabricator via cfe-commits
lancethepants added a comment. In D93164#2653067 , @mgorny wrote: > This change breaks cross-compilation now, as it tries running an executable > built for the target system: > > FAILED: tools/clang/lib/Tooling/ASTNodeAPI.json > cd /home/mgorny/llvm-