[lld] [llvm] [llvm-objcopy][WebAssembly] Allow --strip-debug to operate on relocatable files. (PR #102978)

via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 12 14:00:41 PDT 2024


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 a2acea5c380534430a01843698d4bf46b018110f 2a09d4c812efff1f7e8b12e8b28a003398aa15ac --extensions h,cpp -- lld/wasm/Writer.cpp llvm/lib/ObjCopy/wasm/WasmObjcopy.cpp llvm/lib/ObjCopy/wasm/WasmObject.cpp llvm/lib/ObjCopy/wasm/WasmObject.h llvm/lib/ObjCopy/wasm/WasmReader.cpp
``````````

</details>

<details>
<summary>
View the diff from clang-format here.
</summary>

``````````diff
diff --git a/llvm/lib/ObjCopy/wasm/WasmObject.cpp b/llvm/lib/ObjCopy/wasm/WasmObject.cpp
index bee9c26064..eb672cbc62 100644
--- a/llvm/lib/ObjCopy/wasm/WasmObject.cpp
+++ b/llvm/lib/ObjCopy/wasm/WasmObject.cpp
@@ -32,7 +32,7 @@ void Object::removeSections(function_ref<bool(const Section &)> ToRemove) {
     // relocation sections here instead.
     for (auto &Sec : Sections) {
       if (ToRemove(Sec)) {
-        //static std::vector<char> empty;
+        // static std::vector<char> empty;
         Sec.Name = ".objcopy.removed";
         Sec.SectionType = wasm::WASM_SEC_CUSTOM;
         Sec.Contents = {};

``````````

</details>


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


More information about the llvm-commits mailing list