[PATCH] D121855: [scudo] Fix static and unused function type annotations
Vitaly Buka via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Mar 16 17:22:54 PDT 2022
vitalybuka added inline comments.
================
Comment at: compiler-rt/lib/scudo/standalone/size_class_map.h:348
-template <typename SCMap> static void validateMap() {
+template <typename SCMap> static UNUSED void validateMap() {
for (uptr C = 0; C < SCMap::NumClasses; C++) {
----------------
================
Comment at: compiler-rt/lib/scudo/standalone/tests/checksum_test.cpp:35
// different checksum. We do not use random data to avoid flakyness.
-template <ComputeChecksum F> void verifyChecksumFunctionBitFlip() {
+template <ComputeChecksum F> static void verifyChecksumFunctionBitFlip() {
scudo::uptr Array[sizeof(scudo::u64) / sizeof(scudo::uptr)];
----------------
it's inline as it's template, so why do we need static?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D121855/new/
https://reviews.llvm.org/D121855
More information about the llvm-commits
mailing list