[Bug 54867] bug in r300 compiler
bugzilla-daemon at freedesktop.org
bugzilla-daemon at freedesktop.org
Mon Jun 3 06:31:28 PDT 2013
https://bugs.freedesktop.org/show_bug.cgi?id=54867
--- Comment #2 from Tom Stellard <tstellar at gmail.com> ---
Comment on attachment 80074
--> https://bugs.freedesktop.org/attachment.cgi?id=80074
s/signed/int/
Review of attachment 80074:
-----------------------------------------------------------------
This patch looks good to me, but for the commit message, you need to wrap long
lines to 80 or fewer characters (I actually wrap to 75, but I'm not sure what
the standard convention is) and re-organize the commit message in the form of:
Code area: Brief description
Long description (if necessary)
Link to fixed bugs
For this patch, it should be something like:
r300g/compiler: Fix unsigned comparison with less than zero
rc_find_free_temporary_list() returns signed integer (in case of lack of free
temporary registersreturns -1), so new_index in radeon_rename_regs() should be
signed.
https://bugs.freedesktop.org/show_bug.cgi?id=54867
--
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20130603/2c777b23/attachment.html>
More information about the dri-devel
mailing list