[LNT] Add cflags string argument to lnt

Chris Matthews chris.matthews at apple.com
Tue Jun 9 16:10:42 PDT 2015


Thanks. I am all for enhancements that make LNT easier to use on the command line!

Can you modify you approach to correctly handle quoted and escaped spaces?

Can you make a test? Probably in the runtest section of the tests.

I just created a Phab account.

> On Jun 9, 2015, at 5:02 AM, Charlie Turner <charlie.turner at arm.com> wrote:
> 
> Hi,
>  
> The current behavior seems to only allow you to pass one cflag at a time to the lnt script,
>  
> --cflag cf1
> --cflag cf2
>> --cflag cfn
>  
> The attached patch adds an option that allows you to instead pass –cflags “cf1 cf2 … cfn” to the lnt script.
>  
> Regards,
> Charlie.
>  
> P.S. Chris, do you have Phabricator account? I was going to use the Phab reviewing tool, but I could find your name in the reviewers list.
> <cflag_lnt.patch>_______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150609/b812fc62/attachment.html>


More information about the llvm-commits mailing list