Skip to content

Commit e173ccf

Browse files
authored
[SYCLomatic] Refine migration of thrust::transform_if using Rewriter framework(#826)
Signed-off-by: chenwei.sun <[email protected]>
1 parent e0cea72 commit e173ccf

File tree

4 files changed

+10
-3
lines changed

4 files changed

+10
-3
lines changed

clang/lib/DPCT/APINamesMapThrust.inc

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,6 @@
2121
****************************************************************************/
2222
ENTRY_HOST("thrust::sequence", MapNames::getDpctNamespace() + "iota", "oneapi::dpl::execution::sycl")
2323
ENTRY_HOST("thrust::sort", "oneapi::dpl::sort", "oneapi::dpl::execution::sycl")
24-
ENTRY_HOST("thrust::transform_if", MapNames::getDpctNamespace() + "transform_if", "oneapi::dpl::execution::sycl")
2524
ENTRY_HOST("thrust::reduce", "std::reduce", "oneapi::dpl::execution::sycl")
2625
ENTRY_HOST("thrust::device_free", MapNames::getDpctNamespace() + "free_device", "")
2726
ENTRY_HOST("thrust::device_malloc", MapNames::getDpctNamespace() + "malloc_device", "")

clang/lib/DPCT/APINamesThrust.inc

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -113,6 +113,15 @@ thrustFactory("thrust::scatter",
113113
{{5,PolicyState::HasPolicy,4,MapNames::getDpctNamespace()+"scatter", HelperFeatureEnum::DplExtrasAlgorithm_scatter },
114114
{4,PolicyState::NoPolicy ,4,MapNames::getDpctNamespace()+"scatter", HelperFeatureEnum::DplExtrasAlgorithm_scatter }}),
115115

116+
// thrust::transform_if
117+
thrustFactory("thrust::transform_if",
118+
{{6,PolicyState::HasPolicy,3,MapNames::getDpctNamespace()+"transform_if", HelperFeatureEnum::DplExtrasAlgorithm_transform_if},
119+
{5,PolicyState::NoPolicy,3,MapNames::getDpctNamespace()+"transform_if", HelperFeatureEnum::DplExtrasAlgorithm_transform_if},
120+
{7,PolicyState::HasPolicy,4,MapNames::getDpctNamespace()+"transform_if", HelperFeatureEnum::DplExtrasAlgorithm_transform_if},
121+
{6,PolicyState::NoPolicy,4,MapNames::getDpctNamespace()+"transform_if", HelperFeatureEnum::DplExtrasAlgorithm_transform_if},
122+
{8,PolicyState::HasPolicy,4,MapNames::getDpctNamespace()+"transform_if", HelperFeatureEnum::DplExtrasAlgorithm_transform_if},
123+
{7,PolicyState::NoPolicy,4,MapNames::getDpctNamespace()+"transform_if", HelperFeatureEnum::DplExtrasAlgorithm_transform_if}}),
124+
116125
// thrust::unique_by_key_copy
117126
FEATURE_REQUEST_FACTORY(HelperFeatureEnum::DplExtrasAlgorithm_unique_copy,
118127
CONDITIONAL_FACTORY_ENTRY(

clang/lib/DPCT/ThrustAPIMigration.cpp

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -89,7 +89,6 @@ void ThrustAPIRule::thrustFuncMigration(const MatchFinder::MatchResult &Result,
8989

9090
auto ReplInfo =
9191
MapNames::ThrustFuncNamesMap.find(ThrustFuncNameWithNamespace);
92-
9392
// For the API migration defined in APINamesThrust.inc
9493
if (ReplInfo == MapNames::ThrustFuncNamesMap.end()) {
9594
dpct::ExprAnalysis EA;

clang/test/dpct/test_api_level/DplExtrasAlgorithm/api_test12.cu

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
// RUN: FileCheck --input-file %T/DplExtrasAlgorithm/api_test12_out/count.txt --match-full-lines %s
66
// RUN: rm -rf %T/DplExtrasAlgorithm/api_test12_out
77

8-
// CHECK: 6
8+
// CHECK: 39
99
// TEST_FEATURE: DplExtrasAlgorithm_transform_if
1010

1111
#include <thrust/transform.h>

0 commit comments

Comments
 (0)