llvmbot wrote:

<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-lldb

Author: None (kper)

<details>
<summary>Changes</summary>

As discussed with @<!-- -->DavidSpickett in discord. Running the test runner 
caused the following issue:

```
gmake: Entering directory 
'/home/kper/oss/llvm-project/build/lldb-test-build.noindex/functionalities/thread/concurrent_events/TestConcurrentSignalWatch.test'
/home/kper/oss/llvm-project/build/bin/clang++  -std=c++11 -g -O0  -mriscv 
-I/home/kper/oss/llvm-project/lldb/packages/Python/lldbsuite/test/make/../../../../..//include
 -I/home/kper/oss/llvm-project/build/tools/lldb/include 
-I/home/kper/oss/llvm-project/lldb/test/API/functionalities/thread/concurrent_events
 -I/home/kper/oss/llvm-project/lldb/packages/Python/lldbsuite/test/make 
-include 
/home/kper/oss/llvm-project/lldb/packages/Python/lldbsuite/test/make/test_common.h
 -fno-limit-debug-info      --driver-mode=g++ -MT main.o -MD -MP -MF main.d -c 
-o main.o 
/home/kper/oss/llvm-project/lldb/test/API/functionalities/thread/concurrent_events/main.cpp
clang++: error: unknown argument: '-mriscv'
gmake: *** [Makefile.rules:619: main.o] Error 1
```

By overriding the flags, we avoid the `-mriscv`.

---
Full diff: https://github.com/llvm/llvm-project/pull/124758.diff


1 Files Affected:

- (modified) lldb/packages/Python/lldbsuite/test/make/Makefile.rules (+4) 


``````````diff
diff --git a/lldb/packages/Python/lldbsuite/test/make/Makefile.rules 
b/lldb/packages/Python/lldbsuite/test/make/Makefile.rules
index 2da6ff226b615c..06959f226066ac 100644
--- a/lldb/packages/Python/lldbsuite/test/make/Makefile.rules
+++ b/lldb/packages/Python/lldbsuite/test/make/Makefile.rules
@@ -207,6 +207,10 @@ else
                override ARCH :=
                override ARCHFLAG :=
        endif
+       ifeq "$(ARCH)" "riscv"
+               override ARCH :=
+               override ARCHFLAG :=
+       endif
        ifeq "$(findstring mips,$(ARCH))" "mips"
                override ARCHFLAG := -
        endif

``````````

</details>


https://github.com/llvm/llvm-project/pull/124758
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to