gaspersic/triton
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

[Triton-MLIR] Minor fixes to enable fused-softmax and layer-norm tutorials (#835)

Browse Source
This commit is contained in:
Qingyi Liu
2022-11-09 10:18:56 +08:00
committed by GitHub
parent 2da71b2aaa
commit e517b58d59
4 changed files with 5 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
include/triton/Conversion/TritonGPUToLLVM/PtxAsmFormat.h
View File

@@ -320,7 +320,7 @@ struct PTXInstrExecution {
// Prefix a !predicate to the instruction.
PTXInstrExecution &predicateNot(mlir::Value value, StringRef constraint) {
pred = instr->builder->newOperand(value, constraint);
pred->repr = [](int idx) { return "@!%" + std::to_string(idx); };
pred->repr = [](int idx) { return "@!$" + std::to_string(idx); };
return *this;
}

2
lib/Conversion/TritonGPUToLLVM/PtxAsmFormat.cpp
View File

@@ -125,7 +125,7 @@ std::string PTXBuilder::dump() const {
lines.push_back(exec->dump());
}
return strJoin(lines, "\r\n");
return strJoin(lines, "\n\t");
}
PTXInstrExecution &PTXInstrCommon::call(ArrayRef<Operand *> oprs) {

2
lib/Conversion/TritonGPUToLLVM/TritonGPUToLLVM.cpp
View File

@@ -1051,7 +1051,7 @@ struct LoadOpConversion
if (other) {
for (size_t ii = 0; ii < nWords; ++ii) {
PTXInstr &mov = *ptxBuilder.create<>("mov");
mov.o("u", width);
mov.o("u" + std::to_string(width));
size_t size = width / valueElemNbits;

4
unittest/Conversion/TritonGPUToLLVM/PtxAsmFormatTest.cpp
View File

@@ -112,8 +112,8 @@ TEST_F(PtxAsmFormatTest, MultiLinePTX) {
mov(valVal1, constVal);
mov(valVal1, valVal0);
EXPECT_EQ(builder.dump(), "mov $0, 0x1;\r\n"
"mov $1, 0x1;\r\n"
EXPECT_EQ(builder.dump(), "mov $0, 0x1;\n\t"
"mov $1, 0x1;\n\t"
"mov $1, $0;");
auto values = builder.getAllMLIRArgs();