Skip to content

Commit b27a756

Browse files
authored
[SYCLomatic] Fix compile warning (#2707)
Signed-off-by: Jiang, Zhiwei <zhiwei.jiang@intel.com>
1 parent 2f7d361 commit b27a756

File tree

9 files changed

+130
-97
lines changed

9 files changed

+130
-97
lines changed

clang/lib/DPCT/PreProcessor.cpp

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,6 @@
2020
#include "clang/AST/Type.h"
2121
#include "clang/AST/TypeLoc.h"
2222
#include "clang/Analysis/AnalysisDeclContext.h"
23-
#include "clang/Analysis/CallGraph.h"
2423
#include "clang/Basic/Cuda.h"
2524
#include "clang/Basic/SourceLocation.h"
2625
#include "clang/DPCT/DpctOptions.h"

clang/lib/DPCT/RuleInfra/CallExprRewriterCommon.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2185,7 +2185,7 @@ const std::string MipmapNeedBindlessImage =
21852185
DOES_FIRST_LEVEL_POINTER_NEED_CONST)
21862186
#define NEW(...) makeNewDeleteExprCreator(true, __VA_ARGS__)
21872187
#define DELETE(...) makeNewDeleteExprCreator(false, __VA_ARGS__)
2188-
#define DECL(TYPE, VAR, ...) makeDeclCreator(TYPE, VAR, __VA_ARGS__)
2188+
#define DECLARE(TYPE, VAR, ...) makeDeclCreator(TYPE, VAR, __VA_ARGS__)
21892189
#define TYPENAME(SUBEXPR) makeTypenameExprCreator(SUBEXPR)
21902190
#define ZERO_INITIALIZER(SUBEXPR) makeZeroInitializerCreator(SUBEXPR)
21912191
#define SUBGROUP \

clang/lib/DPCT/RuleInfra/TypeLocRewriters.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -114,7 +114,7 @@ class CheckTemplateArgCount {
114114
return Num == Count;
115115
}
116116
size_t NoneDefaultNum = 0;
117-
for (int i = 0; i < Num; i++) {
117+
for (size_t i = 0; i < Num; i++) {
118118
if (!TSTL.getArgLoc(i).getArgument().getIsDefaulted()) {
119119
NoneDefaultNum++;
120120
}

clang/lib/DPCT/RulesLang/RulesLang.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7317,7 +7317,7 @@ TypeMmberRule::findTokenEndBeforeColonColon(SourceLocation TokStart,
73177317

73187318
bool FoundColonColon = false;
73197319
// Find coloncolon
7320-
while (TokPtr && (TokPtr - 1)) {
7320+
while (TokPtr) {
73217321
if (*TokPtr == ':' && *(TokPtr - 1) == ':') {
73227322
TokPtr = TokPtr - 2;
73237323
FoundColonColon = true;

clang/lib/DPCT/RulesLang/RulesLangCooperativeGroups.cpp

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,6 @@
2222
#include "clang/AST/TypeLoc.h"
2323
#include "clang/ASTMatchers/ASTMatchers.h"
2424
#include "clang/Analysis/AnalysisDeclContext.h"
25-
#include "clang/Analysis/CallGraph.h"
2625
#include "clang/Basic/SourceLocation.h"
2726
#include "clang/Lex/MacroArgs.h"
2827

clang/lib/DPCT/RulesLang/RulesLangNoneAPIAndType.cpp

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,6 @@
2121
#include "clang/AST/TypeLoc.h"
2222
#include "clang/ASTMatchers/ASTMatchers.h"
2323
#include "clang/Analysis/AnalysisDeclContext.h"
24-
#include "clang/Analysis/CallGraph.h"
2524
#include "clang/Basic/Cuda.h"
2625
#include "clang/Basic/SourceLocation.h"
2726
#include "clang/Lex/MacroArgs.h"

clang/lib/DPCT/RulesLang/RulesLangTexture.cpp

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,6 @@
2424
#include "clang/AST/TypeLoc.h"
2525
#include "clang/ASTMatchers/ASTMatchers.h"
2626
#include "clang/Analysis/AnalysisDeclContext.h"
27-
#include "clang/Analysis/CallGraph.h"
2827
#include "clang/Basic/Cuda.h"
2928
#include "clang/Basic/SourceLocation.h"
3029
#include "clang/Lex/MacroArgs.h"

clang/lib/DPCT/RulesMathLib/APINamesCUBLAS.inc

Lines changed: 127 additions & 83 deletions
Large diffs are not rendered by default.

clang/lib/DPCT/RulesMathLib/BLASAPIMigration.cpp

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -437,13 +437,6 @@ void BLASFunctionCallRule::runRule(const MatchFinder::MatchResult &Result) {
437437
bool NeedUseLambda = isConditionOfFlowControl(
438438
CE, OriginStmtType, CanAvoidUsingLambda, OuterInsertLoc);
439439
bool IsInReturnStmt = isInReturnStmt(CE, OuterInsertLoc);
440-
bool CanAvoidBrace = false;
441-
const CompoundStmt *CS = findImmediateBlock(CE);
442-
if (CS && (CS->size() == 1)) {
443-
const Stmt *S = *(CS->child_begin());
444-
if (CE == S || dyn_cast<ReturnStmt>(S))
445-
CanAvoidBrace = true;
446-
}
447440

448441
if (NeedUseLambda) {
449442
PrefixInsertLoc = FuncNameBegin;

0 commit comments

Comments
 (0)