mirror of
https://github.com/ROCm/ROCm.git
synced 2026-04-05 03:01:17 -04:00
[BUILD] minor fixes (#1676)
Remove unused variables, fix member initializer list order.
This commit is contained in:
@@ -912,7 +912,6 @@ unsigned ModuleAxisInfoAnalysis::getPtrContiguity(Value ptr) {
|
||||
if (!tensorTy)
|
||||
return 1;
|
||||
auto layout = tensorTy.getEncoding();
|
||||
auto shape = tensorTy.getShape();
|
||||
|
||||
// Here order should be ordered by contiguous first, so the first element
|
||||
// should have the largest contiguous.
|
||||
|
||||
@@ -103,8 +103,8 @@ struct ReturnOpConversion : public ConvertOpToLLVMPattern<triton::ReturnOp> {
|
||||
struct FuncOpConversion : public FuncOpConversionBase {
|
||||
FuncOpConversion(LLVMTypeConverter &converter, int numWarps,
|
||||
ModuleAllocation &allocation, PatternBenefit benefit)
|
||||
: FuncOpConversionBase(converter, benefit), allocation(allocation),
|
||||
numWarps(numWarps) {}
|
||||
: FuncOpConversionBase(converter, benefit), numWarps(numWarps),
|
||||
allocation(allocation) {}
|
||||
|
||||
triton::FuncOp amendFuncOp(triton::FuncOp funcOp,
|
||||
ConversionPatternRewriter &rewriter) const {
|
||||
@@ -185,7 +185,7 @@ struct CallOpConversion : public ConvertOpToLLVMPattern<triton::CallOp> {
|
||||
CallOpConversion(LLVMTypeConverter &converter, int numWarps,
|
||||
ModuleAllocation &allocation, PatternBenefit benefit)
|
||||
: ConvertOpToLLVMPattern<triton::CallOp>(converter, benefit),
|
||||
allocation(allocation), numWarps(numWarps) {}
|
||||
numWarps(numWarps), allocation(allocation) {}
|
||||
|
||||
LogicalResult
|
||||
matchAndRewrite(triton::CallOp callOp,
|
||||
|
||||
@@ -556,7 +556,6 @@ public:
|
||||
LogicalResult
|
||||
matchAndRewrite(triton::CallOp op, OpAdaptor adaptor,
|
||||
ConversionPatternRewriter &rewriter) const override {
|
||||
auto converter = getTypeConverter();
|
||||
auto newOp = rewriter.replaceOpWithNewOp<triton::CallOp>(
|
||||
op, op.getCallee(), op.getResultTypes(), adaptor.getOperands());
|
||||
addNamedAttrs(newOp, adaptor.getAttributes());
|
||||
@@ -571,9 +570,7 @@ public:
|
||||
LogicalResult
|
||||
matchAndRewrite(ReturnOp op, ReturnOp::Adaptor adaptor,
|
||||
ConversionPatternRewriter &rewriter) const override {
|
||||
auto converter = getTypeConverter();
|
||||
auto newOp =
|
||||
rewriter.replaceOpWithNewOp<ReturnOp>(op, adaptor.getOperands());
|
||||
rewriter.replaceOpWithNewOp<ReturnOp>(op, adaptor.getOperands());
|
||||
return success();
|
||||
}
|
||||
};
|
||||
|
||||
Reference in New Issue
Block a user