Displaying 4 results from an estimated 4 matches for "d8197".
Did you mean:
8197
2015 Mar 12
2
[LLVMdev] [cfe-dev] Commit message policy?
Gentle ping?
On 11 March 2015 at 16:12, Renato Golin <renato.golin at linaro.org> wrote:
> If everyone's happy, I'll commit the change and see where we go from then, ok?
>
> cheers,
> --renato
2015 Mar 15
2
[LLVMdev] [cfe-dev] Bikeshedding commit message policy - Round 3 - Fight!
On 15 March 2015 at 20:22, Chris Lattner <clattner at apple.com> wrote:
> Can you post the entire revised diff that you want to include? Is it Diff 21913 on Phab? If so, LGTM.
Hi Chris,
Here's the final version:
http://reviews.llvm.org/D8197
cheers,
--renato
2015 Mar 14
2
[LLVMdev] Bikeshedding commit message policy - Round 3 - Fight!
Folks,
On review http://reviews.llvm.org/D8197, we're basically down to two
bikeshedding issues:
1. Title tags
Some people use "[CSE] Change blah", others use "CSE: Change blah". I
hadn't put anything regarding tags because not everyone use it and
when they do, it's slightly different. I personally don't th...
2015 Mar 15
2
[LLVMdev] [cfe-dev] Bikeshedding commit message policy - Round 3 - Fight!
On 15 March 2015 at 16:31, Hal Finkel <hfinkel at anl.gov> wrote:
> I don't want to code when to use them. But it makes sense to say, "If you want to include a title tag, do it like this...".
I'm ok with that.
So, do we have consensus?
1. Don't require, but recommend using [] for tags.
2. Don't specify attribution more than just "patch by Foo." and