fully qualifies use of detail namespace

[cjdb]

Some TUs apparently end up with an ambiguity between ::llvm::detail and support::detail, so we close that gap at the source.

[llvmbot]

@llvm/pr-subscribers-llvm-adt

Author: Christopher Di Bella (cjdb)

Changes

Some TUs apparently end up with an ambiguity between ::llvm::detail and support::detail, so we close that gap at the source.

---

Full diff: https://github.com/llvm/llvm-project/pull/87536.diff

1 Files Affected:

• (modified) llvm/include/llvm/ADT/iterator_range.h (+2-2)

diff --git a/llvm/include/llvm/ADT/iterator_range.h b/llvm/include/llvm/ADT/iterator_range.h
index 7d288ea4506ba5..021256c545b0af 100644
--- a/llvm/include/llvm/ADT/iterator_range.h
+++ b/llvm/include/llvm/ADT/iterator_range.h
@@ -50,7 +50,7 @@ class iterator_range {
 #else
   template <typename Container,
             std::enable_if_t<explicitly_convertible<
-                detail::IterOfRange<Container>, IteratorT>::value> * = nullptr>
+                llvm::detail::IterOfRange<Container>, IteratorT>::value> * = nullptr>
 #endif
   iterator_range(Container &&c)
       : begin_iterator(adl_begin(c)), end_iterator(adl_end(c)) {
@@ -65,7 +65,7 @@ class iterator_range {
 };
 
 template <typename Container>
-iterator_range(Container &&) -> iterator_range<detail::IterOfRange<Container>>;
+iterator_range(Container &&) -> iterator_range<llvm::detail::IterOfRange<Container>>;
 
 /// Convenience function for iterating over sub-ranges.
 ///

[github-actions]

✅ With the latest revision this PR passed the C/C++ code formatter.

[slackito]

Looks good to me.

[kazutakahirata]

Thank you for fixing this!