[PATCH] More robust deployment target parsing on darwin
Tim Northover
t.p.northover at gmail.com
Tue Jan 13 10:35:56 PST 2015
Hi Steven,
This mostly looks fine; just one comment:
On 12 January 2015 at 15:00, Steven Wu <stevenwu at apple.com> wrote:
> + if (BeginSDK != std::string::npos && EndSDK != std::string::npos) {
These should probably be using StringRef::npos, since under a fairly
bizarre set of circumstances it may differ from std::string::npos.
Tim.
More information about the cfe-commits
mailing list