This is an archive of the discontinued LLVM Phabricator instance.

[libcxx] Work around for clang calling GAS after having already failed.
ClosedPublic

Authored by dsanders on Jan 26 2016, 8:06 AM.

Details

Summary

This is a workaround to a clang bug which causes libcxx tests to fail in the 3.8
release. The clang bug is currently being investigated. It seems that clang
does not stop after frontend errors when using -verify and -fno-integrated-as
(or when this is the default). This patch adds -fsyntax-only to prevent GAS
from being called, fixing the libcxx failures.

PR26277

Patch by Eric Fiselier

Diff Detail

Event Timeline

dsanders updated this revision to Diff 45994.Jan 26 2016, 8:06 AM
dsanders retitled this revision from to [libcxx] Work around for clang calling GAS after having already failed..
dsanders updated this object.
dsanders added reviewers: EricWF, mclow.lists, hans.
dsanders added a subscriber: cfe-commits.
EricWF accepted this revision.Jan 26 2016, 12:40 PM
EricWF edited edge metadata.

Obviously LGTM. @hans I'm OK with this going into 3.8.

This revision is now accepted and ready to land.Jan 26 2016, 12:40 PM

Thanks. Given that you're the patch author should I wait for someone else to LGTM too?

I'm OK signing off on this myself.

dsanders closed this revision.Jan 28 2016, 5:53 AM