PATCH: R600/SI: Change all instruction names to lowercase

Matt Arsenault Matthew.Arsenault at amd.com
Tue Nov 4 17:08:47 PST 2014


On 11/04/2014 02:56 PM, Tom Stellard wrote:
> Hi,
>
> This branch: http://cgit.freedesktop.org/~tstellar/llvm/log/?h=si-lowercase
> contains patches for converting all of the assembly strings for SI instructions
> to lower case.
>
> The rationale behind this change is to make the assembly output of the R600
> backend match the AMD proprietary compiler, so it is easier to compare code.
> This will also allow us to implement an assembler which uses a syntax more
> people are familiar with.
>
> I've pasted some shell scripts into the commit message that you can use for
> converting old branches to the new naming scheme in order to make merging
> patches easier.  If you run into major merge problems from this change,
> let me know and I can help.
>
> -Tom
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
LGTM. My main concerns with doing this seem to have been converted OK



More information about the llvm-commits mailing list