Skip to content

Commit 1be881a

Browse files
committed
Fix comments
1 parent c198b85 commit 1be881a

File tree

1 file changed

+0
-6
lines changed

1 file changed

+0
-6
lines changed

mlir/lib/Target/LLVMIR/Dialect/OpenMP/OpenMPToLLVMIRTranslation.cpp

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1017,15 +1017,10 @@ allocReductionVars(T loop, ArrayRef<BlockArgument> reductionArgs,
10171017
// variable allocated in the inlined region)
10181018
llvm::Value *var = builder.CreateAlloca(
10191019
moduleTranslation.convertType(reductionDecls[i].getType()));
1020-
// var->setName("private_redvar");
10211020

10221021
llvm::Type *ptrTy = llvm::PointerType::getUnqual(builder.getContext());
10231022
llvm::Value *castVar =
10241023
builder.CreatePointerBitCastOrAddrSpaceCast(var, ptrTy);
1025-
// TODO: I (Sergio) just guessed casting phis[0] like it's done for var is
1026-
// what's supposed to happen with this code coming from a merge from main,
1027-
// but I don't actually know. Someone more familiar with it needs to check
1028-
// this.
10291024
llvm::Value *castPhi =
10301025
builder.CreatePointerBitCastOrAddrSpaceCast(phis[0], ptrTy);
10311026

@@ -1039,7 +1034,6 @@ allocReductionVars(T loop, ArrayRef<BlockArgument> reductionArgs,
10391034
"allocaction is implicit for by-val reduction");
10401035
llvm::Value *var = builder.CreateAlloca(
10411036
moduleTranslation.convertType(reductionDecls[i].getType()));
1042-
// var->setName("private_redvar");
10431037

10441038
llvm::Type *ptrTy = llvm::PointerType::getUnqual(builder.getContext());
10451039
llvm::Value *castVar =

0 commit comments

Comments
 (0)