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 0175a1e4d33720ed7e827b3db5a36f88bdd790a3 86232bd5dd6fe28ea830f0b5fe3ffa4f48d545ce -- clang-tools-extra/clang-apply-replacements/lib/Tooling/ApplyReplacements.cpp compiler-rt/lib/hwasan/hwasan_report.cpp `````````` </details> <details> <summary> View the diff from clang-format here. </summary> ``````````diff diff --git a/clang-tools-extra/clang-apply-replacements/lib/Tooling/ApplyReplacements.cpp b/clang-tools-extra/clang-apply-replacements/lib/Tooling/ApplyReplacements.cpp index 20bafef724..dece201d37 100644 --- a/clang-tools-extra/clang-apply-replacements/lib/Tooling/ApplyReplacements.cpp +++ b/clang-tools-extra/clang-apply-replacements/lib/Tooling/ApplyReplacements.cpp @@ -41,7 +41,8 @@ namespace replace { namespace detail { -static constexpr std::array<StringRef, 3> AllowedExtensions = {".yaml", ".yml", ""}; +static constexpr std::array<StringRef, 3> AllowedExtensions = {".yaml", ".yml", + ""}; template <typename TranslationUnits> static std::error_code collectReplacementsFromDirectory( @@ -60,9 +61,9 @@ static std::error_code collectReplacementsFromDirectory( continue; } - if (!is_contained(AllowedExtensions, extension(I->path()))) + if (!is_contained(AllowedExtensions, extension(I->path()))) continue; - + TUFiles.push_back(I->path()); ErrorOr<std::unique_ptr<MemoryBuffer>> Out = @@ -271,5 +272,5 @@ bool deleteReplacementFiles(const TUReplacementFiles &Files, return Success; } -} -} +} // namespace replace +} // namespace clang diff --git a/compiler-rt/lib/hwasan/hwasan_report.cpp b/compiler-rt/lib/hwasan/hwasan_report.cpp index 6765888ce7..2ee7ac520c 100644 --- a/compiler-rt/lib/hwasan/hwasan_report.cpp +++ b/compiler-rt/lib/hwasan/hwasan_report.cpp @@ -12,7 +12,9 @@ //===----------------------------------------------------------------------===// #include "hwasan_report.h" + #include <dlfcn.h> + #include "hwasan.h" #include "hwasan_allocator.h" #include "hwasan_globals.h" @@ -148,9 +150,9 @@ class SavedStackAllocations { u32 thread_id_; }; -class Decorator: public __sanitizer::SanitizerCommonDecorator { +class Decorator : public __sanitizer::SanitizerCommonDecorator { public: - Decorator() : SanitizerCommonDecorator() { } + Decorator() : SanitizerCommonDecorator() {} const char *Access() { return Blue(); } const char *Allocation() const { return Magenta(); } const char *Origin() const { return Magenta(); } @@ -164,7 +166,8 @@ static bool FindHeapAllocation(HeapAllocationsRingBuffer *rb, uptr tagged_addr, HeapAllocationRecord *har, uptr *ring_index, uptr *num_matching_addrs, uptr *num_matching_addrs_4b) { - if (!rb) return false; + if (!rb) + return false; *num_matching_addrs = 0; *num_matching_addrs_4b = 0; @@ -188,9 +191,7 @@ static bool FindHeapAllocation(HeapAllocationsRingBuffer *rb, uptr tagged_addr, // Measure the number of heap ring buffer entries that would have matched // if we only had 4 tag bits, which is the case for MTE. - auto untag_4b = [](uptr p) { - return p & ((1ULL << 60) - 1); - }; + auto untag_4b = [](uptr p) { return p & ((1ULL << 60) - 1); }; if (untag_4b(h.tagged_addr) <= untag_4b(tagged_addr) && untag_4b(h.tagged_addr) + h.requested_size > untag_4b(tagged_addr)) { ++*num_matching_addrs_4b; @@ -1058,25 +1059,25 @@ void ReportRegisters(const uptr *frame, uptr pc) { // result in a new logcat line, irrespective of whether a newline is present, // and so we wish to reduce the number of Printf() calls we have to make. #if defined(__aarch64__) - Printf(" x0 %016llx x1 %016llx x2 %016llx x3 %016llx\n", - frame[0], frame[1], frame[2], frame[3]); + Printf(" x0 %016llx x1 %016llx x2 %016llx x3 %016llx\n", frame[0], + frame[1], frame[2], frame[3]); #elif SANITIZER_RISCV64 Printf(" sp %016llx x1 %016llx x2 %016llx x3 %016llx\n", reinterpret_cast<const u8 *>(frame) + 256, frame[1], frame[2], frame[3]); #endif - Printf(" x4 %016llx x5 %016llx x6 %016llx x7 %016llx\n", - frame[4], frame[5], frame[6], frame[7]); - Printf(" x8 %016llx x9 %016llx x10 %016llx x11 %016llx\n", - frame[8], frame[9], frame[10], frame[11]); - Printf(" x12 %016llx x13 %016llx x14 %016llx x15 %016llx\n", - frame[12], frame[13], frame[14], frame[15]); - Printf(" x16 %016llx x17 %016llx x18 %016llx x19 %016llx\n", - frame[16], frame[17], frame[18], frame[19]); - Printf(" x20 %016llx x21 %016llx x22 %016llx x23 %016llx\n", - frame[20], frame[21], frame[22], frame[23]); - Printf(" x24 %016llx x25 %016llx x26 %016llx x27 %016llx\n", - frame[24], frame[25], frame[26], frame[27]); + Printf(" x4 %016llx x5 %016llx x6 %016llx x7 %016llx\n", frame[4], + frame[5], frame[6], frame[7]); + Printf(" x8 %016llx x9 %016llx x10 %016llx x11 %016llx\n", frame[8], + frame[9], frame[10], frame[11]); + Printf(" x12 %016llx x13 %016llx x14 %016llx x15 %016llx\n", frame[12], + frame[13], frame[14], frame[15]); + Printf(" x16 %016llx x17 %016llx x18 %016llx x19 %016llx\n", frame[16], + frame[17], frame[18], frame[19]); + Printf(" x20 %016llx x21 %016llx x22 %016llx x23 %016llx\n", frame[20], + frame[21], frame[22], frame[23]); + Printf(" x24 %016llx x25 %016llx x26 %016llx x27 %016llx\n", frame[24], + frame[25], frame[26], frame[27]); // hwasan_check* reduces the stack pointer by 256, then __hwasan_tag_mismatch // passes it to this function. #if defined(__aarch64__) `````````` </details> https://github.com/llvm/llvm-project/pull/79900 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits