This is an archive of the discontinued LLVM Phabricator instance.

Patch to make AArch64 AsmParser be case insensitive about accepting vector register names
ClosedPublic

Authored by rs on Jun 8 2015, 1:36 PM.

Details

Summary

The AArch64 AsmParser doesn't accept vector register names that start with an upper-case 'V'. GAS is case insensitive about these register names and so is armasm, this patch will make armclang behave the same way.

Diff Detail

Event Timeline

rs updated this revision to Diff 27330.Jun 8 2015, 1:36 PM
rs retitled this revision from to Patch to make AArch64 AsmParser be case insensitive about accepting vector register names.
rs updated this object.
rs edited the test plan for this revision. (Show Details)
rs added a subscriber: Unknown Object (MLST).
This revision was automatically updated to reflect the committed changes.
rs added a comment.Jun 8 2015, 2:40 PM

Hi James,

Thanks for the review.

Thanks,
Ranjeet

From: James Molloy [mailto:james@jamesmolloy.co.uk]
Sent: 08 June 2015 22:25
To: reviews+D10320+public+67e11b9caa35016c@reviews.llvm.org; Ranjeet Singh
Cc: kanheim@a-bix.com; llvm-commits@cs.uiuc.edu
Subject: Re: [PATCH] Patch to make AArch64 AsmParser be case insensitive about accepting vector register names

Hi Ranjeet,

This looks good to me.

Cheers,

James