Page MenuHomePhabricator

[analyzer] ccc-analyzer: handle --sysroot=/path in addition to --sysroot /path
ClosedPublic

Authored by chris.laplante on Wed, Aug 21, 5:08 PM.

Diff Detail

Repository
rL LLVM

Event Timeline

chris.laplante created this revision.Wed, Aug 21, 5:08 PM
Herald added a project: Restricted Project. · View Herald TranscriptWed, Aug 21, 5:08 PM

I encountered this issue when trying to integrate scan-build with CMake in a cross-compiling environment.

NoQ accepted this revision.Wed, Sep 4, 2:58 PM

Thanks!!

I wish i had time to fix D55683 so that i could have you write tests for it :)

This revision is now accepted and ready to land.Wed, Sep 4, 2:58 PM
In D66569#1658413, @NoQ wrote:

I wish i had time to fix D55683 so that i could have you write tests for it :)

I just subscribed to it, so if/when you do, I'll add some tests :)

NoQ added a comment.Wed, Sep 4, 3:05 PM

You don't have commit access, right? Do you want me to commit this patch for you?

Correct, no access so please commit for me. This is my first contribution.

This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptWed, Sep 4, 5:43 PM