From ac98d4a443f46ce2775f415f8d806df335cc3b80 Mon Sep 17 00:00:00 2001 From: Andrew Noyes Date: Thu, 8 Aug 2024 10:44:54 -0700 Subject: [PATCH] Remove always_inline attribute - it wasn't affecting codegen --- ConflictSet.cpp | 10 +++------- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/ConflictSet.cpp b/ConflictSet.cpp index 2dcc3bd..03f1c3a 100644 --- a/ConflictSet.cpp +++ b/ConflictSet.cpp @@ -2104,13 +2104,9 @@ bool scan16(const InternalVersionT *vs, const uint8_t *is, int begin, int end, } // Returns true if v[i] <= readVersion for all i such that begin <= i < end -// -// always_inline So that we can optimize when begin or end is a constant. -// gcovr exclude annotation necessary because of always_inline? template -inline __attribute__((always_inline)) bool -scan16(const InternalVersionT *vs, int begin, int end, // GCOVR_EXCL_LINE - InternalVersionT readVersion) { // GCOVR_EXCL_LINE +bool scan16(const InternalVersionT *vs, int begin, int end, + InternalVersionT readVersion) { assert(0 <= begin && begin < 16); assert(0 <= end && end <= 16); assert(begin <= end); @@ -2873,7 +2869,7 @@ bool checkRangeRead(Node *n, std::span begin, template __attribute__((target("avx512f"))) bool scan16(const InternalVersionT *vs, const uint8_t *is, int begin, int end, InternalVersionT readVersion); -template __attribute__((always_inline, target("avx512f"))) bool +template __attribute__((target("avx512f"))) bool scan16(const InternalVersionT *vs, int begin, int end, InternalVersionT readVersion); template __attribute__((target("avx512f"))) bool