Skip to content

Commit d26e309

Browse files
[SYCLomatic][NFC] Move migration rules in ASTTraversal.cpp into RulesLang/RulesLang.cpp/h. (#2477)
1 parent 6b7b5a9 commit d26e309

File tree

15 files changed

+15272
-15205
lines changed

15 files changed

+15272
-15205
lines changed

clang/lib/DPCT/ASTTraversal.cpp

Lines changed: 14 additions & 14125 deletions
Large diffs are not rendered by default.

clang/lib/DPCT/ASTTraversal.h

Lines changed: 0 additions & 1065 deletions
Large diffs are not rendered by default.

clang/lib/DPCT/CMakeLists.txt

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -112,6 +112,7 @@ add_clang_library(DPCT
112112
ASTTraversal.cpp
113113
PreProcessor.cpp
114114
AnalysisInfo.cpp
115+
RulesLang/RulesLang.cpp
115116
RulesLangLib/CUB/CallExprRewriterCUB.cpp
116117
RulesLangLib/CUB/RewriterClassMethods.cpp
117118
RulesLangLib/CUB/RewriterDeviceHistgram.cpp

clang/lib/DPCT/RuleInfra/CallExprRewriterCommon.h

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -563,8 +563,8 @@ makeFuncNameFromDevAttrCreator(unsigned idx) {
563563
auto Arg = CE->getArg(idx)->IgnoreImplicitAsWritten();
564564
if (auto DRE = dyn_cast<DeclRefExpr>(Arg)) {
565565
auto ArgName = DRE->getNameInfo().getAsString();
566-
auto Search = EnumConstantRule::EnumNamesMap.find(ArgName);
567-
if (Search != EnumConstantRule::EnumNamesMap.end()) {
566+
auto Search = MapNames::EnumNamesMap.find(ArgName);
567+
if (Search != MapNames::EnumNamesMap.end()) {
568568
requestHelperFeatureForEnumNames(ArgName);
569569
return Search->second->NewName;
570570
}

clang/lib/DPCT/RuleInfra/ExprAnalysis.cpp

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@
99
#include "RuleInfra/ExprAnalysis.h"
1010

1111
#include "ASTTraversal.h"
12+
#include "RulesLang/RulesLang.h"
1213
#include "AnalysisInfo.h"
1314
#include "RulesLangLib/CUBAPIMigration.h"
1415
#include "RuleInfra/CallExprRewriter.h"
@@ -577,10 +578,10 @@ void ExprAnalysis::analyzeExpr(const DeclRefExpr *DRE) {
577578
}
578579

579580
auto &ReplEnum =
580-
MapNames::findReplacedName(EnumConstantRule::EnumNamesMap, RefString);
581+
MapNames::findReplacedName(MapNames::EnumNamesMap, RefString);
581582
requestHelperFeatureForEnumNames(RefString);
582-
auto ItEnum = EnumConstantRule::EnumNamesMap.find(RefString);
583-
if (ItEnum != EnumConstantRule::EnumNamesMap.end()) {
583+
auto ItEnum = MapNames::EnumNamesMap.find(RefString);
584+
if (ItEnum != MapNames::EnumNamesMap.end()) {
584585
for (auto ItHeader = ItEnum->second->Includes.begin();
585586
ItHeader != ItEnum->second->Includes.end(); ItHeader++) {
586587
DpctGlobalInfo::getInstance().insertHeader(DRE->getBeginLoc(),

clang/lib/DPCT/RuleInfra/MapNames.cpp

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,8 +11,10 @@
1111
#include "RuleInfra/CallExprRewriter.h"
1212
#include "RulesDNN/DNNAPIMigration.h"
1313
#include "FileGenerator/GenFiles.h"
14+
#include "RulesLang/RulesLang.h"
1415
#include <map>
1516

17+
1618
using namespace clang;
1719
using namespace clang::dpct;
1820
// Not use sycl:: namespace explicitly
@@ -42,7 +44,7 @@ MapNames::MapTy MapNames::DeviceRandomGeneratorTypeMap;
4244
std::unordered_map<std::string, std::shared_ptr<TypeNameRule>>
4345
MapNames::CuDNNTypeNamesMap;
4446
std::unordered_map<std::string, std::shared_ptr<EnumNameRule>>
45-
EnumConstantRule::EnumNamesMap;
47+
MapNames::EnumNamesMap;
4648
MapNames::MapTy CuDNNTypeRule::CuDNNEnumNamesMap;
4749
std::map<std::string /*Original API*/, HelperFeatureEnum>
4850
CuDNNTypeRule::CuDNNEnumNamesHelperFeaturesMap;
@@ -1303,7 +1305,7 @@ void MapNames::setExplicitNamespaceMap(
13031305
{"CUDNN_RNN_DATA_LAYOUT_SEQ_MAJOR_PACKED", HelperFeatureEnum::device_ext},
13041306
};
13051307
// Enum constants name mapping.
1306-
EnumConstantRule::EnumNamesMap = {
1308+
MapNames::EnumNamesMap = {
13071309
// ...
13081310
// enum Device Attribute
13091311
// ...

clang/lib/DPCT/RuleInfra/MapNames.h

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -309,6 +309,9 @@ class MapNames {
309309
using SetTy = std::set<std::string>;
310310
using ThrustMapTy = std::map<std::string, ThrustFuncReplInfo>;
311311

312+
313+
static std::unordered_map<std::string, std::shared_ptr<EnumNameRule>>
314+
EnumNamesMap;
312315
static const SetTy SupportedVectorTypes;
313316
static const SetTy VectorTypes2MArray;
314317
static const std::map<std::string, int> VectorTypeMigratedTypeSizeMap;

clang/lib/DPCT/RulesLang/CallExprRewriterMemory.cpp

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@
88

99
#include "RuleInfra/CallExprRewriter.h"
1010
#include "RuleInfra/CallExprRewriterCommon.h"
11+
#include "RulesLang/RulesLang.h"
1112

1213
namespace clang {
1314
namespace dpct {

clang/lib/DPCT/RulesLang/CallExprRewriterWmma.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,12 +26,12 @@ void CallExprRewriterFactoryBase::initRewriterMapWmma() {
2626
// nvcuda::wmma::mem_row_major, this mapping must be protected by option
2727
// "--use-experimental-features=matrix".
2828
if (DpctGlobalInfo::useExtJointMatrix() && !DpctGlobalInfo::useSYCLCompat()) {
29-
EnumConstantRule::EnumNamesMap.insert(
29+
MapNames::EnumNamesMap.insert(
3030
{"nvcuda::wmma::mem_row_major",
3131
std::make_shared<EnumNameRule>(
3232
MapNames::getClNamespace() +
3333
"ext::oneapi::experimental::matrix::layout::row_major")});
34-
EnumConstantRule::EnumNamesMap.insert(
34+
MapNames::EnumNamesMap.insert(
3535
{"nvcuda::wmma::mem_col_major",
3636
std::make_shared<EnumNameRule>(
3737
MapNames::getClNamespace() +

0 commit comments

Comments
 (0)