[libc-commits] [libc] d9bbad2 - [libc][Obvious][NFC] A bunch of cosmetic cleanup.

Siva Chandra Reddy via libc-commits libc-commits at lists.llvm.org
Fri Nov 19 09:03:58 PST 2021


Author: Siva Chandra Reddy
Date: 2021-11-19T17:02:55Z
New Revision: d9bbad277c44a305b72d7d63faac4887bd65a564

URL: https://github.com/llvm/llvm-project/commit/d9bbad277c44a305b72d7d63faac4887bd65a564
DIFF: https://github.com/llvm/llvm-project/commit/d9bbad277c44a305b72d7d63faac4887bd65a564.diff

LOG: [libc][Obvious][NFC] A bunch of cosmetic cleanup.

* Added missing header guards.
* Fixed license header format in a few files.
* Renamed files to more suitable names.

Added: 
    libc/fuzzing/stdlib/atof_differential_fuzz.cpp

Modified: 
    libc/fuzzing/math/RemQuoDiff.h
    libc/fuzzing/stdlib/CMakeLists.txt
    libc/include/sys/mman.h.def
    libc/include/sys/syscall.h.def
    libc/src/__support/architectures.h
    libc/src/__support/common.h
    libc/src/__support/endian.h
    libc/src/__support/sanitizer.h

Removed: 
    libc/fuzzing/stdlib/atof_fuzz.cpp


################################################################################
diff  --git a/libc/fuzzing/math/RemQuoDiff.h b/libc/fuzzing/math/RemQuoDiff.h
index f37cf3ceebd44..b934bc8858769 100644
--- a/libc/fuzzing/math/RemQuoDiff.h
+++ b/libc/fuzzing/math/RemQuoDiff.h
@@ -6,6 +6,9 @@
 //
 //===----------------------------------------------------------------------===//
 
+#ifndef LLVM_LIBC_FUZZING_MATH_REMQUO_H
+#define LLVM_LIBC_FUZZING_MATH_REMQUO_H
+
 #include "src/__support/FPUtil/FPBits.h"
 
 #include <math.h>
@@ -46,3 +49,5 @@ void RemQuoDiff(RemQuoFunc<T> func1, RemQuoFunc<T> func2, const uint8_t *data,
   if (bits1.uintval() != bits2.uintval())
     __builtin_trap();
 }
+
+#endif // LLVM_LIBC_FUZZING_MATH_REMQUO_H

diff  --git a/libc/fuzzing/stdlib/CMakeLists.txt b/libc/fuzzing/stdlib/CMakeLists.txt
index 436125b6cd7ad..0016f4f7ccbfd 100644
--- a/libc/fuzzing/stdlib/CMakeLists.txt
+++ b/libc/fuzzing/stdlib/CMakeLists.txt
@@ -7,9 +7,9 @@ add_libc_fuzzer(
 )
 
 add_libc_fuzzer(
-  atof_fuzz
+  atof_
diff erential_fuzz
   SRCS
-    atof_fuzz.cpp
+    atof_
diff erential_fuzz.cpp
   HDRS
     StringParserOutputDiff.h
   DEPENDS

diff  --git a/libc/fuzzing/stdlib/atof_fuzz.cpp b/libc/fuzzing/stdlib/atof_
diff erential_fuzz.cpp
similarity index 100%
rename from libc/fuzzing/stdlib/atof_fuzz.cpp
rename to libc/fuzzing/stdlib/atof_
diff erential_fuzz.cpp

diff  --git a/libc/include/sys/mman.h.def b/libc/include/sys/mman.h.def
index 738aa3eaea3ad..23435c768cb0a 100644
--- a/libc/include/sys/mman.h.def
+++ b/libc/include/sys/mman.h.def
@@ -6,6 +6,11 @@
 //
 //===----------------------------------------------------------------------===//
 
+#ifndef LLVM_LIBC_SYS_MMAN_H
+#define LLVM_LIBC_SYS_MMAN_H
+
 #include <__llvm-libc-common.h>
 
 %%public_api()
+
+#endif // LLVM_LIBC_SYS_MMAN_H

diff  --git a/libc/include/sys/syscall.h.def b/libc/include/sys/syscall.h.def
index 5262f07858209..5cb6a20da77be 100644
--- a/libc/include/sys/syscall.h.def
+++ b/libc/include/sys/syscall.h.def
@@ -6,7 +6,12 @@
 //
 //===----------------------------------------------------------------------===//
 
+#ifndef LLVM_LIBC_SYS_SYSCALL_H
+#define LLVM_LIBC_SYS_SYSCALL_H
+
 <!> If syscall.h were a linux only file, then we do not need this indirection.
 <!> However, to keep the option of a non-linux OS requiring a syscall.h file,
 <!> with its own special syscall numbers, we use this indirection.
 %%include_file(${syscall_numbers})
+
+#endif // LLVM_LIBC_SYS_SYSCALL_H

diff  --git a/libc/src/__support/architectures.h b/libc/src/__support/architectures.h
index 485faae47c7e3..4975d229422f6 100644
--- a/libc/src/__support/architectures.h
+++ b/libc/src/__support/architectures.h
@@ -1,4 +1,4 @@
-//===-- Compile time architecture detection -------------------------------===//
+//===-- Compile time architecture detection ---------------------*- C++ -*-===//
 //
 // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
 // See https://llvm.org/LICENSE.txt for license information.
@@ -6,6 +6,9 @@
 //
 //===----------------------------------------------------------------------===//
 
+#ifndef LLVM_LIBC_SUPPORT_ARCHITECTURES_H
+#define LLVM_LIBC_SUPPORT_ARCHITECTURES_H
+
 #if defined(__pnacl__) || defined(__CLR_VER)
 #define LLVM_LIBC_ARCH_VM
 #endif
@@ -33,3 +36,5 @@
 #if (defined(LLVM_LIBC_ARCH_AARCH64) || defined(LLVM_LIBC_ARCH_ARM))
 #define LLVM_LIBC_ARCH_ANY_ARM
 #endif
+
+#endif // LLVM_LIBC_SUPPORT_ARCHITECTURES_H

diff  --git a/libc/src/__support/common.h b/libc/src/__support/common.h
index 53a63fc2e9173..17c155e3a8e44 100644
--- a/libc/src/__support/common.h
+++ b/libc/src/__support/common.h
@@ -1,4 +1,4 @@
-//===-- Common internal contructs -----------------------------------------===//
+//===-- Common internal contructs -------------------------------*- C++ -*-===//
 //
 // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
 // See https://llvm.org/LICENSE.txt for license information.

diff  --git a/libc/src/__support/endian.h b/libc/src/__support/endian.h
index e1d52ca468d34..5f977aa466c5b 100644
--- a/libc/src/__support/endian.h
+++ b/libc/src/__support/endian.h
@@ -1,4 +1,4 @@
-//===-- Endianness support ------------------------------------------------===//
+//===-- Endianness support --------------------------------------*- C++ -*-===//
 //
 // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
 // See https://llvm.org/LICENSE.txt for license information.

diff  --git a/libc/src/__support/sanitizer.h b/libc/src/__support/sanitizer.h
index 16fe6ecc0644f..e7ae05f070b5f 100644
--- a/libc/src/__support/sanitizer.h
+++ b/libc/src/__support/sanitizer.h
@@ -1,4 +1,4 @@
-//===-- Convenient sanitizer macros ---------------------------------------===//
+//===-- Convenient sanitizer macros -----------------------------*- C++ -*-===//
 //
 // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
 // See https://llvm.org/LICENSE.txt for license information.


        


More information about the libc-commits mailing list