[llvm-commits] Patch: First one.

Relph, Richard Richard.Relph at amd.com
Wed Dec 12 16:36:38 PST 2012


On Dec 12, 2012, at 4:28 PM, Eli Bendersky <eliben at google.com> wrote:

> On Wed, Dec 12, 2012 at 4:20 PM, Relph, Richard <Richard.Relph at amd.com> wrote:
>> Or, since I'm guessing full branch-relative paths would be preferred, this
>> equivalent patch…
>> Again, relative to the AMDILBackend branch.
>> 
>> 
>> 
> 
> While the patch itself looks fine (after fixing it to use full
> repo-relative path)
Thanks.

> , a couple of notes:
> 
> 1. Always provide descriptive titles to your PATCH requests to
> llvm-commits. "First One" doesn't say much.
Understood.

> 2. I'm not sure what you mean by "branch" in your emails, but it's
> unlikely the thing that's usually meant by "branch" in LLVM and
> software in general.
Maybe so… I am just using the terminology Xcode applies to it and that it appears "under" in llvm.org.
What term should be used to describe this?
http://llvm.org/svn/llvm-project/llvm/branches/AMDILBackend

Thanks,
Richard
> 
> Eli
> 






More information about the llvm-commits mailing list