[Lldb-commits] [lldb] Fixing makefile issue to include riscv emulation test only when RISCV is present. (PR #159842)

David Peixotto via lldb-commits lldb-commits at lists.llvm.org
Fri Sep 19 13:18:44 PDT 2025


================
@@ -2,14 +2,34 @@ add_lldb_unittest(EmulatorTests
   ARM64/TestAArch64Emulator.cpp
   LoongArch/TestLoongArchEmulator.cpp
   RISCV/TestRISCVEmulator.cpp
-
+  
   LINK_COMPONENTS
     Support
+    ${LLVM_TARGETS_TO_BUILD}
   LINK_LIBS
     lldbCore
     lldbSymbol
     lldbTarget
     lldbPluginInstructionARM64
     lldbPluginInstructionLoongArch
     lldbPluginInstructionRISCV
+    lldbPluginProcessUtility
   )
+
+if ("RISCV" IN_LIST LLVM_TARGETS_TO_BUILD)
----------------
dmpots wrote:

Please add a comment here describing why this is split into a separate unit test. Would be good to call out that we leave the other RISCV test in the generic unit test because it does not require the backend to be built.

https://github.com/llvm/llvm-project/pull/159842


More information about the lldb-commits mailing list