[Lldb-commits] [lldb] [lldb][DataFormatter] VectorType: fix format for arrays with size not a power-of-2 (PR #68907)
via lldb-commits
lldb-commits at lists.llvm.org
Fri Oct 13 11:40:47 PDT 2023
github-actions[bot] wrote:
<!--LLVM CODE FORMAT COMMENT: {clang-format}-->
:warning: C/C++ code formatter, clang-format found issues in your code. :warning:
<details>
<summary>
You can test this locally with the following command:
</summary>
``````````bash
git-clang-format --diff b98b567c2501540ef4a9d586c26ab8271c6d1f0d d0d6425021e1666b92b1c96accebf9edf1d217b6 -- lldb/source/DataFormatters/VectorType.cpp lldb/test/API/functionalities/data-formatter/vector-types/main.cpp
``````````
</details>
<details>
<summary>
View the diff from clang-format here.
</summary>
``````````diff
diff --git a/lldb/source/DataFormatters/VectorType.cpp b/lldb/source/DataFormatters/VectorType.cpp
index 4b534224d538..2af8a693e635 100644
--- a/lldb/source/DataFormatters/VectorType.cpp
+++ b/lldb/source/DataFormatters/VectorType.cpp
@@ -193,10 +193,9 @@ static lldb::Format GetItemFormatForFormat(lldb::Format format,
/// type 'element_type'. Returns a std::nullopt if the
/// size of the container is not a multiple of 'element_type'
/// or if an error occurs.
-static std::optional<size_t> CalculateNumChildren(
- CompilerType container_elem_type,
- uint64_t num_elements,
- CompilerType element_type) {
+static std::optional<size_t>
+CalculateNumChildren(CompilerType container_elem_type, uint64_t num_elements,
+ CompilerType element_type) {
std::optional<uint64_t> container_elem_size =
container_elem_type.GetByteSize(/* exe_scope */ nullptr);
if (!container_elem_size)
@@ -256,7 +255,8 @@ public:
m_parent_format, element_type,
parent_type.GetTypeSystem().GetSharedPointer());
m_num_children =
- ::CalculateNumChildren(element_type, num_elements, m_child_type).value_or(0);
+ ::CalculateNumChildren(element_type, num_elements, m_child_type)
+ .value_or(0);
m_item_format = GetItemFormatForFormat(m_parent_format, m_child_type);
return false;
}
``````````
</details>
https://github.com/llvm/llvm-project/pull/68907
More information about the lldb-commits
mailing list