Skip to content

Commit 604456e

Browse files
committed
Remove unintended code from MLIRTranslation.
1 parent 9244fd1 commit 604456e

File tree

1 file changed

+6
-17
lines changed

1 file changed

+6
-17
lines changed

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

Lines changed: 6 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -827,12 +827,12 @@ static void collectReductionInfo(
827827
atomicGen = owningAtomicReductionGens[i];
828828
llvm::Value *variable =
829829
moduleTranslation.lookupValue(loop.getReductionVars()[i]);
830-
reductionInfos.push_back(llvm::OpenMPIRBuilder::ReductionInfo(
831-
moduleTranslation.convertType(reductionDecls[i].getType()), variable,
832-
privateReductionVariables[i],
833-
/*EvaluationKind=*/llvm::OpenMPIRBuilder::EvaluationKindTy::Scalar,
834-
owningReductionGens[i],
835-
/*ReductionGenClang=*/nullptr, atomicGen));
830+
reductionInfos.push_back(
831+
{moduleTranslation.convertType(reductionDecls[i].getType()), variable,
832+
privateReductionVariables[i],
833+
/*EvaluationKind=*/llvm::OpenMPIRBuilder::EvaluationKindTy::Scalar,
834+
owningReductionGens[i],
835+
/*ReductionGenClang=*/nullptr, atomicGen});
836836
}
837837
}
838838

@@ -2908,17 +2908,6 @@ LogicalResult OpenMPDialectLLVMIRTranslationInterface::amendOperation(
29082908
}
29092909
return failure();
29102910
})
2911-
.Case("omp.target",
2912-
[&](Attribute attr) {
2913-
if (auto targetAttr = attr.dyn_cast<omp::TargetAttr>()) {
2914-
llvm::OpenMPIRBuilderConfig &config =
2915-
moduleTranslation.getOpenMPBuilder()->Config;
2916-
config.TargetCPU = targetAttr.getTargetCpu();
2917-
config.TargetFeatures = targetAttr.getTargetFeatures();
2918-
return success();
2919-
}
2920-
return failure();
2921-
})
29222911
.Default([](Attribute) {
29232912
// Fall through for omp attributes that do not require lowering.
29242913
return success();

0 commit comments

Comments
 (0)