[cfe-commits] [PATCH] StringRef'ize clang::driver::Option::getName()

Chris Lattner clattner at apple.com
Thu Jun 30 13:04:47 PDT 2011


On Jun 30, 2011, at 11:47 AM, Zach Wheeler wrote:

> This patch changes the return type of Option::getName() from const char* to StringRef, doing away with the need for several calls to strlen.
> Clang builds fine with these changes.

THis patch looks reasonable, except that it completely breaks the testsuite :).  Please track down the bug and resubmit a patch.  Also, why not just embed a 'Name' StringRef in Option.h instead of storing the start+size?

-Chris




More information about the cfe-commits mailing list