kpdev42 created this revision.
kpdev42 added reviewers: clayborg, davide, k8stone, DavidSpickett,
granata.enrico.
kpdev42 added a project: LLDB.
Herald added subscribers: omjavaid, JDevlieghere, kristof.beyls.
Herald added a project: All.
kpdev42 requested review of this revision.
Herald added a subscriber: lldb-commits.
Patch sets ARM cpu, before compiling JIT code. This enables FastISel for armv6
and higher CPUs and allows using hardware FPU
~~~
OS Laboratory. Huawei RRI. Saint-Petersburg
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D131783
Files:
lldb/source/Utility/ArchSpec.cpp
lldb/test/API/lang/c/fpeval/TestFPEval.py
Index: lldb/test/API/lang/c/fpeval/TestFPEval.py
===================================================================
--- lldb/test/API/lang/c/fpeval/TestFPEval.py
+++ lldb/test/API/lang/c/fpeval/TestFPEval.py
@@ -19,7 +19,6 @@
# Find the line number to break inside main().
self.line = line_number('main.c', '// Set break point at this line.')
- @skipIf(archs=no_match(['amd64', 'x86_64', 'arm64'])) # lldb jitter
incorrectly evals function with FP args on 32 bit arm
def test(self):
"""Test floating point expressions while jitter is disabled."""
self.build()
Index: lldb/source/Utility/ArchSpec.cpp
===================================================================
--- lldb/source/Utility/ArchSpec.cpp
+++ lldb/source/Utility/ArchSpec.cpp
@@ -583,7 +583,6 @@
std::string ArchSpec::GetClangTargetCPU() const {
std::string cpu;
-
if (IsMIPS()) {
switch (m_core) {
case ArchSpec::eCore_mips32:
@@ -630,6 +629,9 @@
break;
}
}
+
+ if (GetTriple().isARM())
+ cpu = GetTriple().getARMCPUForArch("").str();
return cpu;
}
Index: lldb/test/API/lang/c/fpeval/TestFPEval.py
===================================================================
--- lldb/test/API/lang/c/fpeval/TestFPEval.py
+++ lldb/test/API/lang/c/fpeval/TestFPEval.py
@@ -19,7 +19,6 @@
# Find the line number to break inside main().
self.line = line_number('main.c', '// Set break point at this line.')
- @skipIf(archs=no_match(['amd64', 'x86_64', 'arm64'])) # lldb jitter incorrectly evals function with FP args on 32 bit arm
def test(self):
"""Test floating point expressions while jitter is disabled."""
self.build()
Index: lldb/source/Utility/ArchSpec.cpp
===================================================================
--- lldb/source/Utility/ArchSpec.cpp
+++ lldb/source/Utility/ArchSpec.cpp
@@ -583,7 +583,6 @@
std::string ArchSpec::GetClangTargetCPU() const {
std::string cpu;
-
if (IsMIPS()) {
switch (m_core) {
case ArchSpec::eCore_mips32:
@@ -630,6 +629,9 @@
break;
}
}
+
+ if (GetTriple().isARM())
+ cpu = GetTriple().getARMCPUForArch("").str();
return cpu;
}
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
