This is an automated email from the ASF dual-hosted git repository.

adar pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/kudu.git


The following commit(s) were added to refs/heads/master by this push:
     new c0cb960  Fix loop unroll pragma for GCC < 8
c0cb960 is described below

commit c0cb960bd39fcbf573247596e81acf5068b924db
Author: Todd Lipcon <t...@apache.org>
AuthorDate: Wed Mar 25 11:11:10 2020 -0700

    Fix loop unroll pragma for GCC < 8
    
    The 'GCC unroll' pragma was only introduced in GCC 8. Add appropriate
    preprocessor checks to avoid warnings on earlier gcc versions.
    
    Change-Id: I0278aba3a7e319a93cdefccc27b63a9e5450a12c
    Reviewed-on: http://gerrit.cloudera.org:8080/15558
    Tested-by: Todd Lipcon <t...@apache.org>
    Reviewed-by: Adar Dembo <a...@cloudera.com>
---
 src/kudu/util/bitmap.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/kudu/util/bitmap.h b/src/kudu/util/bitmap.h
index 5a937c0..54a2409 100644
--- a/src/kudu/util/bitmap.h
+++ b/src/kudu/util/bitmap.h
@@ -217,7 +217,7 @@ inline void ForEachBit(const uint8_t* bitmap,
     int tot_count = Bits::CountOnes64withPopcount(w);
 #ifdef __clang__
 #pragma unroll(3)
-#else
+#elif __GNUC__ >= 8
 #pragma GCC unroll 3
 #endif
     for (int i = 0; i < tot_count; i++) {

Reply via email to