[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