<div class="gmail_quote">On Wed, Sep 28, 2011 at 12:25 PM, Eli Friedman <span dir="ltr"><<a href="mailto:eli.friedman@gmail.com">eli.friedman@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
On Wed, Sep 28, 2011 at 4:58 AM, Justin Holewinski<br>
<div><div></div><div class="h5"><<a href="mailto:justin.holewinski@gmail.com">justin.holewinski@gmail.com</a>> wrote:<br>
> On Tue, Sep 27, 2011 at 4:01 PM, Eli Friedman <<a href="mailto:eli.friedman@gmail.com">eli.friedman@gmail.com</a>><br>
> wrote:<br>
>><br>
>> On Tue, Sep 27, 2011 at 12:25 PM, Justin Holewinski<br>
>> <<a href="mailto:justin.holewinski@gmail.com">justin.holewinski@gmail.com</a>> wrote:<br>
>> > Author: jholewinski<br>
>> > Date: Tue Sep 27 14:25:49 2011<br>
>> > New Revision: 140624<br>
>> ><br>
>> > URL: <a href="http://llvm.org/viewvc/llvm-project?rev=140624&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=140624&view=rev</a><br>
>> > Log:<br>
>> > PTX: Fix case where printed alignment could be 0<br>
>> ><br>
>> > Modified:<br>
>> >    llvm/trunk/lib/Target/PTX/PTXAsmPrinter.cpp<br>
>> ><br>
>> > Modified: llvm/trunk/lib/Target/PTX/PTXAsmPrinter.cpp<br>
>> > URL:<br>
>> > <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/PTX/PTXAsmPrinter.cpp?rev=140624&r1=140623&r2=140624&view=diff" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/PTX/PTXAsmPrinter.cpp?rev=140624&r1=140623&r2=140624&view=diff</a><br>

>> ><br>
>> > ==============================================================================<br>
>> > --- llvm/trunk/lib/Target/PTX/PTXAsmPrinter.cpp (original)<br>
>> > +++ llvm/trunk/lib/Target/PTX/PTXAsmPrinter.cpp Tue Sep 27 14:25:49 2011<br>
>> > @@ -480,7 +480,7 @@<br>
>> >   unsigned alignment = gv->getAlignment();<br>
>> >   if (alignment != 0) {<br>
>> >     decl += ".align ";<br>
>> > -    decl += utostr(Log2_32(gv->getAlignment()));<br>
>> > +    decl += utostr(std::max(1U, Log2_32(gv->getAlignment())));<br>
>> >     decl += " ";<br>
>> >   }<br>
>><br>
>> This looks wrong: you're making sure the alignment is at least 2 bytes?!<br>
><br>
> No, this is making sure it is at least 1 byte. There were some cases where<br>
> the Log2 function was returning 0.<br>
<br>
</div></div>log2(1) == 0.  log2(2) == 1.  Hence, your patch enforces a minimum<br>
alignment of 2 bytes.<br></blockquote><div><br></div><div>Ah, I see the source of the confusion.  I was actually fixing the wrong bug. :)</div><div><br></div><div>The log shouldn't even be there, not sure when it made it in.  Anyway, its fixed in r140709 now.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<font color="#888888"><br>
-Eli<br>
</font></blockquote></div><br><br clear="all"><div><br></div>-- <br><br><div>Thanks,</div><div><br></div><div>Justin Holewinski</div><br>