[PATCH] D37953: [ARM] Relax `cpsie`/`cpsid` flag parsing.

Peter Smith via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 18 03:00:09 PDT 2017


peter.smith added a comment.

This looks like the right change to make. There isn't anything written down about case-sensitivity in unified assembly language syntax that I could find in the Arm ARM, the best I could find was in the Armasm documentation http://infocenter.arm.com/help/topic/com.arm.doc.dui0473m/dom1359731141352.html "Considerations when writing assembly language source code", which implies that all lower case or all upper case is acceptable. I think the only potential problem would be if the same letter meant two different things in upper and lower case and I can't think of any case where that happens. Both armasm and gcc seem to be fine with AIF as well as aif.


https://reviews.llvm.org/D37953





More information about the llvm-commits mailing list