[Lldb-commits] [lldb] Add no-op handing for HLSLAttributedResource switch cases (PR #106698)
https://github.com/dklimkin created
https://github.com/llvm/llvm-project/pull/106698
New value added in e00e9a3f8294c9b96cb0328bf136fab72aeec749
>From 241469a4e639a95caed22605d59728e605d09db5 Mon Sep 17 00:00:00 2001
From: Danial Klimkin
Date: Fri, 30 Aug 2024 11:36:16 +0200
Subject: [PATCH 1/3] Add no-op handing for HLSLAttributedResource switch case
New value added in e00e9a3f8294c9b96cb0328bf136fab72aeec749
---
lldb/source/Plugins/TypeSystem/Clang/TypeSystemClang.cpp | 4
1 file changed, 4 insertions(+)
diff --git a/lldb/source/Plugins/TypeSystem/Clang/TypeSystemClang.cpp
b/lldb/source/Plugins/TypeSystem/Clang/TypeSystemClang.cpp
index 695801da9da69a..2368d6f8aa5b7f 100644
--- a/lldb/source/Plugins/TypeSystem/Clang/TypeSystemClang.cpp
+++ b/lldb/source/Plugins/TypeSystem/Clang/TypeSystemClang.cpp
@@ -4242,6 +4242,10 @@
TypeSystemClang::GetTypeClass(lldb::opaque_compiler_type_t type) {
case clang::Type::PackIndexing:
break;
}
+
+ case clang::Type::HLSLAttributedResource:
+break;
+
// We don't know hot to display this type...
return lldb::eTypeClassOther;
}
>From dd29bc792588fd23000985120e832242c14c07fb Mon Sep 17 00:00:00 2001
From: Danial Klimkin
Date: Fri, 30 Aug 2024 11:42:03 +0200
Subject: [PATCH 2/3] Two more cases
---
.../Plugins/TypeSystem/Clang/TypeSystemClang.cpp | 11 +--
1 file changed, 9 insertions(+), 2 deletions(-)
diff --git a/lldb/source/Plugins/TypeSystem/Clang/TypeSystemClang.cpp
b/lldb/source/Plugins/TypeSystem/Clang/TypeSystemClang.cpp
index 2368d6f8aa5b7f..6efa1b2c4b7859 100644
--- a/lldb/source/Plugins/TypeSystem/Clang/TypeSystemClang.cpp
+++ b/lldb/source/Plugins/TypeSystem/Clang/TypeSystemClang.cpp
@@ -4241,11 +4241,11 @@
TypeSystemClang::GetTypeClass(lldb::opaque_compiler_type_t type) {
// We don't handle pack indexing yet
case clang::Type::PackIndexing:
break;
- }
case clang::Type::HLSLAttributedResource:
break;
-
+ }
+
// We don't know hot to display this type...
return lldb::eTypeClassOther;
}
@@ -5152,6 +5152,9 @@ lldb::Encoding
TypeSystemClang::GetEncoding(lldb::opaque_compiler_type_t type,
// We don't handle pack indexing yet
case clang::Type::PackIndexing:
break;
+
+ case clang::Type::HLSLAttributedResource:
+break;
}
count = 0;
return lldb::eEncodingInvalid;
@@ -5314,6 +5317,10 @@ lldb::Format
TypeSystemClang::GetFormat(lldb::opaque_compiler_type_t type) {
case clang::Type::PackIndexing:
break;
}
+
+ case clang::Type::HLSLAttributedResource:
+break;
+
// We don't know hot to display this type...
return lldb::eFormatBytes;
}
>From af49efdf83539b2819d23626a16f2388b012446a Mon Sep 17 00:00:00 2001
From: Danial Klimkin
Date: Fri, 30 Aug 2024 11:48:24 +0200
Subject: [PATCH 3/3] Update TypeSystemClang.cpp
---
lldb/source/Plugins/TypeSystem/Clang/TypeSystemClang.cpp | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/lldb/source/Plugins/TypeSystem/Clang/TypeSystemClang.cpp
b/lldb/source/Plugins/TypeSystem/Clang/TypeSystemClang.cpp
index 6efa1b2c4b7859..b0f49ebf2d2cbb 100644
--- a/lldb/source/Plugins/TypeSystem/Clang/TypeSystemClang.cpp
+++ b/lldb/source/Plugins/TypeSystem/Clang/TypeSystemClang.cpp
@@ -4245,7 +4245,6 @@
TypeSystemClang::GetTypeClass(lldb::opaque_compiler_type_t type) {
case clang::Type::HLSLAttributedResource:
break;
}
-
// We don't know hot to display this type...
return lldb::eTypeClassOther;
}
@@ -5316,11 +5315,10 @@ lldb::Format
TypeSystemClang::GetFormat(lldb::opaque_compiler_type_t type) {
// We don't handle pack indexing yet
case clang::Type::PackIndexing:
break;
- }
case clang::Type::HLSLAttributedResource:
break;
-
+ }
// We don't know hot to display this type...
return lldb::eFormatBytes;
}
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
[Lldb-commits] [lldb] Add no-op handing for HLSLAttributedResource switch cases (PR #106698)
https://github.com/dklimkin closed https://github.com/llvm/llvm-project/pull/106698 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
[Lldb-commits] [lldb] Update test past bdbad0d07bb600301cb324e87a6be37ca4af591a (PR #84889)
https://github.com/dklimkin created
https://github.com/llvm/llvm-project/pull/84889
None
>From 04886b73fc9ca7b4a6adc52f5306cb6addc85e5c Mon Sep 17 00:00:00 2001
From: Danial Klimkin
Date: Tue, 12 Mar 2024 11:02:32 +0100
Subject: [PATCH] Update test past bdbad0d07bb600301cb324e87a6be37ca4af591a
---
.../data-formatter/builtin-formats/TestBuiltinFormats.py| 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
a/lldb/test/API/functionalities/data-formatter/builtin-formats/TestBuiltinFormats.py
b/lldb/test/API/functionalities/data-formatter/builtin-formats/TestBuiltinFormats.py
index 8c3bdabeaac1ba..4d6f44db0195b4 100644
---
a/lldb/test/API/functionalities/data-formatter/builtin-formats/TestBuiltinFormats.py
+++
b/lldb/test/API/functionalities/data-formatter/builtin-formats/TestBuiltinFormats.py
@@ -308,5 +308,5 @@ def test_pointer(self):
@no_debug_info_test
def test_instruction(self):
self.assertIn(
-" addq 0xa(%rdi), %r8\n", self.getFormatted("instruction",
"0x0a47034c")
+"= addq 0xa(%rdi), %r8\n", self.getFormatted("instruction",
"0x0a47034c")
)
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
[Lldb-commits] [lldb] Update test past bdbad0d07bb600301cb324e87a6be37ca4af591a (PR #84889)
https://github.com/dklimkin closed https://github.com/llvm/llvm-project/pull/84889 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
[Lldb-commits] [lldb] Fix lock guads in PipePosix.cpp (PR #90572)
https://github.com/dklimkin created
https://github.com/llvm/llvm-project/pull/90572
Guard object destroyed immediately after creation without naming.
>From 7be6091ee5a4509bfec231adfdab3bc135112070 Mon Sep 17 00:00:00 2001
From: Danial Klimkin
Date: Tue, 30 Apr 2024 10:05:02 +0200
Subject: [PATCH] Fix lock guads in PipePosix.cpp
Guard object destroyed immediately after creation without naming.
---
lldb/source/Host/posix/PipePosix.cpp | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/lldb/source/Host/posix/PipePosix.cpp
b/lldb/source/Host/posix/PipePosix.cpp
index afd3fe39059ac1..f35c348990df62 100644
--- a/lldb/source/Host/posix/PipePosix.cpp
+++ b/lldb/source/Host/posix/PipePosix.cpp
@@ -108,7 +108,7 @@ Status PipePosix::CreateNew(bool child_processes_inherit) {
}
Status PipePosix::CreateNew(llvm::StringRef name, bool child_process_inherit) {
- std::scoped_lock (m_read_mutex, m_write_mutex);
+ std::scoped_lock guard(m_read_mutex, m_write_mutex);
if (CanReadUnlocked() || CanWriteUnlocked())
return Status("Pipe is already opened");
@@ -146,7 +146,7 @@ Status PipePosix::CreateWithUniqueName(llvm::StringRef
prefix,
Status PipePosix::OpenAsReader(llvm::StringRef name,
bool child_process_inherit) {
- std::scoped_lock (m_read_mutex, m_write_mutex);
+ std::scoped_lock guard(m_read_mutex, m_write_mutex);
if (CanReadUnlocked() || CanWriteUnlocked())
return Status("Pipe is already opened");
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
[Lldb-commits] [lldb] Fix lock guads in PipePosix.cpp (PR #90572)
https://github.com/dklimkin closed https://github.com/llvm/llvm-project/pull/90572 ___ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
