This is an archive of the discontinued LLVM Phabricator instance.

clang-cl: Accept MSVC 2015's `/execution-charset:utf-8` flag.
ClosedPublic

Authored by thakis on Aug 26 2016, 11:14 AM.

Details

Reviewers
hans
Summary

Like https://reviews.llvm.org/D23807, but for execution-charset.

Diff Detail

Event Timeline

thakis updated this revision to Diff 69408.Aug 26 2016, 11:14 AM
thakis retitled this revision from to clang-cl: Accept MSVC 2015's `/execution-charset:utf-8` flag. .
thakis updated this object.
thakis added a reviewer: hans.
thakis added a subscriber: cfe-commits.
majnemer added inline comments.
lib/Driver/Tools.cpp
5838–5846

I'd use compare_lower because lower introduces a std::string.

hans accepted this revision.Aug 26 2016, 1:35 PM
hans edited edge metadata.

lgtm

majnemer's suggestion sounds good though, and for inputCharset too.

This revision is now accepted and ready to land.Aug 26 2016, 1:35 PM
thakis closed this revision.Aug 26 2016, 2:20 PM
thakis marked an inline comment as done.

r279866, thanks!