[PATCH] D115862: [LoongArch][test] (5/5) Add encoding and mnemonics tests

Min-Yih Hsu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 20 03:18:20 PST 2021


myhsu added a comment.

Just want to double check: AsmParser is not implemented yet, is that correct?
Also, can you rename the test filename to be more expressive and straightforward.



================
Comment at: llvm/utils/extract-section.py:62
             help='Whether to print a \'.\' every 8 bits in bits printing mode')
+    arg_parser.add_argument('--little-endian', action='store_true',
+            help='Print the binary code in little-endian format')
----------------
nit: can we use `--endian=<little/big>` instead with default value being "big"?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D115862



More information about the llvm-commits mailing list