This is an archive of the discontinued LLVM Phabricator instance.

Donot use -std=libstdc++ option if $(CC)=cc.
AbandonedPublic

Authored by sivachandra on Nov 11 2014, 12:28 PM.

Details

Reviewers
emaste
Summary

After D6199 (http://reviews.llvm.org/D6199) landed, this test is
erroring out because of incompatible -std=libstdc++ when building on stock
Ubuntu.

Diff Detail

Event Timeline

sivachandra retitled this revision from to Donot use -std=libstdc++ option if $(CC)=cc..
sivachandra updated this object.
sivachandra edited the test plan for this revision. (Show Details)
sivachandra added a reviewer: emaste.
sivachandra added a subscriber: Unknown Object (MLST).
emaste edited edge metadata.Nov 20 2014, 4:50 PM

Let me see if I follow,

Previously we appended -stdlib=libstdc++ for icc and clang, not for gcc (and we never set CC=cc)
Now we append -stdlib=libstdc++ for icc and clang, and not for gcc or cc

I think this is fine as far as fixing the Ubuntu failure.

On FreeBSD if we find cc it will be clang though, so that will be a new issue.

How does -stdlib=libstdc++ fail on Ubuntu?

Sorry for the delay, I missed this somehow.

So, when c++ is g++ (as is the case on Ubuntu), it will not recognize the -stdlib=libstdc++ option. Now since you say that cc is actually clang on FreeBSD, I do not think my patch is correct.

Do we still want to do something here?

sivachandra abandoned this revision.May 27 2015, 1:22 PM