[llvm-dev] [PATCH] *_OPTIONS="help=1" should print to stdout and fit 80 chars width

[ChrisBieneman via llvm-dev]

I actually think this may deserve discussion on LLVM-dev. I commented on the review (http://reviews.llvm.org/D17566) summarizing my opinion, but I'll recap here for people who may not have noticed it on LLVM-commits.

This patch is a move to make sanitizer help spew conform to the GNU project conventions. These are not conventions we recognize anywhere in any of the LLVM projects. Changing the sanitizer libraries to conform would result in an inconstant convention across LLVM projects, which I don't really think is something we should have.

If we're going to make compiler-rt, or any other LLVM project conform to a convention, we should probably discuss that more widely than a patch review.

-Chris

> On Feb 23, 2016, at 11:54 PM, Maxim Ostapenko via llvm-dev <llvm-dev at lists.llvm.org> wrote:
> 
> Hi, Martin!
> 
> You can find pretty nice instructions how to do it via Phabricator here: http://llvm.org/docs/Phabricator.html.
> 
> -Maxim
> 
>> On 23/02/16 21:30, Xinliang David Li via llvm-dev wrote:
>> Can you send this patch to llvm-commits ?
>> 
>> David
>> 
>> On Tue, Feb 23, 2016 at 1:15 AM, Martin Liška <llvm-dev at lists.llvm.org <mailto:llvm-dev at lists.llvm.org>> wrote:
>> 
>>    Hello.
>> 
>>    I would like to modify printing of help options in order to fix:
>>    https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69840
>> 
>>    I basically split all *_flags.inc options with new lines.
>>    Eventually, these
>>    string are printed to limit the output to 80 characters:
>>    https://www.gnu.org/prep/standards/html_node/Command_002dLine-Interfaces.html
>> 
>>    That's my first patch request in the project, please advise if
>>    anything is missing.
>> 
>>    Thanks,
>>    Martin
>> 
>>    _______________________________________________
>>    LLVM Developers mailing list
>>    llvm-dev at lists.llvm.org <mailto:llvm-dev at lists.llvm.org>
>>    http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev
>> 
>> 
>> 
>> 
>> _______________________________________________
>> LLVM Developers mailing list
>> llvm-dev at lists.llvm.org
>> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev
> 
> _______________________________________________
> LLVM Developers mailing list
> llvm-dev at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev