[PATCH] D137924: [TargetParser] Split AArch64TargetParser from ARMTargetParser

Tomas Matheson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Nov 13 23:11:45 PST 2022


tmatheson created this revision.
tmatheson added reviewers: lenary, pratlucas, dmgreen, DavidSpickett.
Herald added subscribers: hiraditya, kristof.beyls.
Herald added a project: All.
tmatheson requested review of this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

AArch64TargetParser reuses data structures and some data from ARMTargetParser,
which causes more problems than it solves. This change separates them.

Code which is common to ARM and AArch64 is moved to ARMTargetParserCommon
which both ARMTargetParser and AArch64TargetParser use.

Some of the information in AArch64TargetParser.def was unused or nonsensical
(CPU_ATTR, ARCH_ATTR, ARCH_FPU) because it reused data strutures from
ARMTargetParser where some of these make sense. These are removed.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D137924

Files:
  llvm/include/llvm/Support/AArch64TargetParser.def
  llvm/include/llvm/Support/AArch64TargetParser.h
  llvm/include/llvm/Support/ARMTargetParser.h
  llvm/include/llvm/Support/ARMTargetParserCommon.h
  llvm/lib/Support/AArch64TargetParser.cpp
  llvm/lib/Support/ARMTargetParser.cpp
  llvm/lib/Support/ARMTargetParserCommon.cpp
  llvm/lib/Support/CMakeLists.txt
  llvm/unittests/Support/TargetParserTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D137924.475058.patch
Type: text/x-patch
Size: 25646 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20221114/9f19efd9/attachment-0001.bin>


More information about the llvm-commits mailing list