<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Dec 10, 2013 at 6:22 AM, Peter Bergner <span dir="ltr"><<a href="mailto:bergner@vnet.ibm.com" target="_blank">bergner@vnet.ibm.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div>> <span>Revert</span> three patches which were committed without explicit contribution<br>

> by their authors.<br>
><br>
> This may break builds where others added code relying on these patches,<br>
> but please *do not* <span>revert</span> this commit. Instead, we will prepare patches<br>
> which fix the failures.<br>
><br>
> Reverts the following commits:<br>
> r168306: "[asan] support x32 mode in the fast stack unwinder. Patch by H.J. Lu"<br>
> r168356: "[asan] more support for powerpc, patch by Peter Bergner"<br>
> r196489: "[sanitizer] fix the ppc32 build (patch by Jakub Jelinek)"<br>
<br>
<br>
</div>As one of the authors of these patches, what is the rational for reverting them?</blockquote></div><br>Sorry for the terrible delays in responding to this thread. Holiday vacation and travel plans by multiple different people conspired to hold up my response until this week.</div>

<div class="gmail_extra"><br></div><div class="gmail_extra">The crux of the problem is that we need have some process for folks to contribute the patch explicitly to the LLVM project (in addition to potentially contributing it to the branch of the code in use by GCC). We hadn't really clarified what that process was, and there wasn't really any process followed for these patches. That's not cool, as it isn't respectful to the authors.</div>

<div class="gmail_extra"><br></div><div class="gmail_extra">I've mailed out a patch to the LLVM developer's policy which should clarify this. The only thing that would help is for the authors of these three patches (and the fourth patch I found later) to just email a copy of the patch to llvm-commits, or if any of you have commit access to directly commit the patch. </div>

<div class="gmail_extra"><br></div><div class="gmail_extra">I don't want someone to look back and say "hey, how come this guy just submitted that other person's code!" Instead I want them to see an explicit contribution of the code by that person to the project. Typically not a big deal, but I felt like we didn't do a good job the first time here, so I want to make it extra clear here that we were respecting your wishes here. =] That's all.</div>

</div>