[PATCH] D97236: [MC][ARM] add .w suffixes for BL (T1) and DBG

Tue Feb 23 10:30:04 PST 2021

nickdesaulniers marked 2 inline comments as done.
nickdesaulniers added a comment.

In D97236#2581308 <https://reviews.llvm.org/D97236#2581308>, @DavidSpickett wrote:

> I couldn't find an official rule for allowing ".w" when the instructions only have 32 bit encodings.

My interpretation of "F1 <https://reviews.llvm.org/F1>.2 Standard assembler syntax fields":

>> .W Meaning wide, specifies that the assembler must select a 32-bit encoding for the
>> instruction. If this is not possible, an assembler error is produced.

(and says the same for `.N`/16b, so an error diagnostic should be produced whenever there is not a corresponding encoding for that suffix).

See also this thread starting from: https://github.com/ClangBuiltLinux/linux/issues/1296#issuecomment-778598573.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D97236/new/

https://reviews.llvm.org/D97236