Skip to content

Commit 7f8779e

Browse files
committed
[llvm][AArch64] Actually check expected FPU for CPUs
We were passing this as an argument but never using it. ARM has always checked this. Note that the FPU list is shared between ARM and AArch64 so there is no AArch64::getFPUName, just ARM::getFPUName. Reviewed By: dmgreen Differential Revision: https://reviews.llvm.org/D93387
1 parent eba09a2 commit 7f8779e

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed

llvm/unittests/Support/TargetParserTest.cpp

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -805,6 +805,9 @@ bool testAArch64CPU(StringRef CPUName, StringRef ExpectedArch,
805805
else
806806
pass &= (ExtKind == ExpectedFlags);
807807

808+
unsigned FPUKind = AArch64::getDefaultFPU(CPUName, AK);
809+
pass &= ARM::getFPUName(FPUKind).equals(ExpectedFPU);
810+
808811
pass &= AArch64::getCPUAttr(AK).equals(CPUAttr);
809812

810813
return pass;

0 commit comments

Comments
 (0)