Skip to content

[lldb] Include <cstring> for memset#196335

Merged
Nerixyz merged 1 commit into
llvm:mainfrom
Nerixyz:fix/lldb-arm-include
May 7, 2026
Merged

[lldb] Include <cstring> for memset#196335
Nerixyz merged 1 commit into
llvm:mainfrom
Nerixyz:fix/lldb-arm-include

Conversation

@Nerixyz

@Nerixyz Nerixyz commented May 7, 2026

Copy link
Copy Markdown
Contributor

Fixes the failure on lldb-aarch64-ubuntu (https://lab.llvm.org/buildbot/#/builders/59/builds/34161) after #195514.

@Nerixyz Nerixyz requested a review from JDevlieghere as a code owner May 7, 2026 14:49
@Nerixyz Nerixyz enabled auto-merge (squash) May 7, 2026 14:49
@llvmorg-github-actions

Copy link
Copy Markdown

@llvm/pr-subscribers-lldb

Author: Nerixyz (Nerixyz)

Changes

Fixes the failure on lldb-aarch64-ubuntu (https://lab.llvm.org/buildbot/#/builders/59/builds/34161) after #195514.


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

1 Files Affected:

  • (modified) lldb/test/API/lang/cpp/char1632_t/main.cpp (+1)
diff --git a/lldb/test/API/lang/cpp/char1632_t/main.cpp b/lldb/test/API/lang/cpp/char1632_t/main.cpp
index e38188cf85dee..9520b52711008 100644
--- a/lldb/test/API/lang/cpp/char1632_t/main.cpp
+++ b/lldb/test/API/lang/cpp/char1632_t/main.cpp
@@ -1,4 +1,5 @@
 #include <assert.h>
+#include <cstring>
 #include <string>
 
 #define UASZ 64

@Nerixyz Nerixyz merged commit 655b7a4 into llvm:main May 7, 2026
12 checks passed
yebinchon pushed a commit to yebinchon/llvm-project that referenced this pull request May 7, 2026
@DavidSpickett

Copy link
Copy Markdown
Contributor

I did this a few hours ago #196300 but thanks anyway! I caught it locally because the bot has been a bit behind lately.

Didn't realise that GitHub would merge the PR if the result was an empty commit. No reason not to I suppose.

moar55 pushed a commit to moar55/llvm-project that referenced this pull request May 12, 2026
pedroMVicente pushed a commit to pedroMVicente/llvm-project that referenced this pull request May 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants