[llvm-commits] Patch: First one.

Eli Bendersky eliben at google.com
Wed Dec 12 16:28:46 PST 2012


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), a couple of notes:

1. Always provide descriptive titles to your PATCH requests to
llvm-commits. "First One" doesn't say much.
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.

Eli




More information about the llvm-commits mailing list