[PATCH] D21785: [RFC]Add unittests to AArch64TargetParser

jojo.ma via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 27 23:07:54 PDT 2016


jojo created this revision.
jojo added reviewers: rengolin, echristo, abdulras, compnerd, jmolloy, t.p.northover.
jojo added a subscriber: llvm-commits.
Herald added subscribers: mehdi_amini, rengolin, aemerson.

    - Add unittest to AArch64TargetParser
    - Correct a incorrect indexing problem in AArch64TargetParser
      The architecture enumeration is shared across ARM and AArch64 in 
      original implementation.But In the code,I just used the index which
       was offset by the ARM, and this  would index into the array incorrectly.
      To make AArch64 has its own arch enum,or we will do a lot of slowly 
       iterating.
  -  Correct a spelling error by the way.
     parameter of llvm::AArch64::getArchExtName









http://reviews.llvm.org/D21785

Files:
  include/llvm/Support/AArch64TargetParser.def
  include/llvm/Support/TargetParser.h
  lib/Support/TargetParser.cpp
  lib/Target/AArch64/AsmParser/AArch64AsmParser.cpp
  unittests/Support/TargetParserTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D21785.62057.patch
Type: text/x-patch
Size: 19491 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160628/664a32d3/attachment.bin>


More information about the llvm-commits mailing list