[OpenACC][CIR] Fix transform inclusive scan init parameter

[xlauko]

This fixes macos build, where otherwise the compilation yields an error: no viable conversion from 'bool' to 'typename iterator_traits<const QualType *>::value_type

[xlauko]

• [CIR] Make all opt tests verify roundtrip #161439
• [CIR] Refactor cir.cast to use uniform assembly form w/o parens, commas #161431
• [OpenACC][CIR] Fix transform inclusive scan init parameter #161428 👈 (View in Graphite)
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[llvmbot]

@llvm/pr-subscribers-clang

@llvm/pr-subscribers-clangir

Author: Henrich Lauko (xlauko)

Changes

This fixes macos build, where otherwise the compilation yields an error: no viable conversion from 'bool' to 'typename iterator_traits<const QualType *>::value_type

---

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

1 Files Affected:

• (modified) clang/lib/CIR/CodeGen/CIRGenOpenACCRecipe.cpp (+1-1)

diff --git a/clang/lib/CIR/CodeGen/CIRGenOpenACCRecipe.cpp b/clang/lib/CIR/CodeGen/CIRGenOpenACCRecipe.cpp
index e41c2d85fbd5d..fc28ac552224c 100644
--- a/clang/lib/CIR/CodeGen/CIRGenOpenACCRecipe.cpp
+++ b/clang/lib/CIR/CodeGen/CIRGenOpenACCRecipe.cpp
@@ -89,7 +89,7 @@ mlir::Value OpenACCRecipeBuilderBase::makeBoundsAlloca(
   std::transform_inclusive_scan(
       resultTypes.begin(), resultTypes.end(),
       std::back_inserter(allocasLeftArr), std::plus<bool>{},
-      [](QualType ty) { return !ty->isConstantArrayType(); });
+      [](QualType ty) { return !ty->isConstantArrayType(); }, false);
 
   // Keep track of the number of 'elements' that we're allocating. Individual
   // allocas should multiply this by the size of its current allocation.

[erichkeane]

Huh, looks like someone's library is missing overload 1 here :)

https://en.cppreference.com/w/cpp/algorithm/transform_inclusive_scan.html

[xlauko]

Merge activity

• Sep 30, 8:24 PM UTC: A user started a stack merge that includes this pull request via Graphite.
• Sep 30, 8:25 PM UTC: @xlauko merged this pull request with Graphite.