[llvm-branch-commits] [llvm] b9029ed - Extend GCC workaround to GCC < 8.4 for llvm::iterator_range ctor (#82643)

via llvm-branch-commits llvm-branch-commits at lists.llvm.org
Thu Feb 22 17:49:26 PST 2024


Author: Thomas Preud'homme
Date: 2024-02-22T21:08:58Z
New Revision: b9029ed0053b93d94e0f133e6a44250f82bf0dc2

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

LOG: Extend GCC workaround to GCC < 8.4 for llvm::iterator_range ctor (#82643)

GCC SFINAE error with decltype was fixed in commit
ac5e28911abdfb8d9bf6bea980223e199bbcf28d which made it into GCC 8.4.
Therefore adjust GCC version test accordingly.

(cherry picked from commit 7f71fa909a10be182b82b9dfaf0fade6eb84796c)

Added: 
    

Modified: 
    llvm/include/llvm/ADT/iterator_range.h

Removed: 
    


################################################################################
diff  --git a/llvm/include/llvm/ADT/iterator_range.h b/llvm/include/llvm/ADT/iterator_range.h
index 2dc227935984b1..7d288ea4506ba5 100644
--- a/llvm/include/llvm/ADT/iterator_range.h
+++ b/llvm/include/llvm/ADT/iterator_range.h
@@ -43,8 +43,8 @@ class iterator_range {
   IteratorT begin_iterator, end_iterator;
 
 public:
-#if __GNUC__ == 7
-  // Be careful no to break gcc-7 on the mlir target.
+#if __GNUC__ == 7 || (__GNUC__ == 8 && __GNUC_MINOR__ < 4)
+  // Be careful no to break gcc-7 and gcc-8 < 8.4 on the mlir target.
   // See https://github.com/llvm/llvm-project/issues/63843
   template <typename Container>
 #else


        


More information about the llvm-branch-commits mailing list