Page MenuHomePhabricator
Differential D94402

[mlir] Fix gpu-to-llvm lowering for gpu.alloc with dynamic sizes.
ClosedPublic
Actions

Authored by csigg on Jan 11 2021, 4:43 AM.

Details

Reviewers
ftynse
herhut
mravishankar
antiagainst
Commits
rGd59ddba77725: [mlir] Fix gpu-to-llvm lowering for gpu.alloc with dynamic sizes.

Diff Detail

Unit TestsFailed
View All

TimeTest
250 msx64 debian > MLIR.Conversion/SPIRVToLLVM::lower-host-to-llvm-calls.mlir
Script: -- : 'RUN: at line 1'; /mnt/disks/ssd0/agent/llvm-project/build/bin/mlir-opt --lower-host-to-llvm /mnt/disks/ssd0/agent/llvm-project/mlir/test/Conversion/SPIRVToLLVM/lower-host-to-llvm-calls.mlir | /mnt/disks/ssd0/agent/llvm-project/build/bin/FileCheck /mnt/disks/ssd0/agent/llvm-project/mlir/test/Conversion/SPIRVToLLVM/lower-host-to-llvm-calls.mlir
130 msx64 windows > MLIR.Conversion/SPIRVToLLVM::lower-host-to-llvm-calls.mlir
Script: -- : 'RUN: at line 1'; c:\ws\w64\llvm-project\premerge-checks\build\bin\mlir-opt.exe --lower-host-to-llvm C:\ws\w64\llvm-project\premerge-checks\mlir\test\Conversion\SPIRVToLLVM\lower-host-to-llvm-calls.mlir | c:\ws\w64\llvm-project\premerge-checks\build\bin\filecheck.exe C:\ws\w64\llvm-project\premerge-checks\mlir\test\Conversion\SPIRVToLLVM\lower-host-to-llvm-calls.mlir

Event Timeline

csigg created this revision.Jan 11 2021, 4:43 AM
Herald added a reviewer: ftynse. · View Herald TranscriptJan 11 2021, 4:43 AM
Herald added subscribers: teijeong, rdzhabarov, tatianashp and 15 others. · View Herald Transcript
csigg requested review of this revision.Jan 11 2021, 4:43 AM
Herald added a reviewer: herhut. · View Herald TranscriptJan 11 2021, 4:43 AM
Herald added a project: Restricted Project. · View Herald Transcript
Herald added subscribers: stephenneuendorffer, nicolasvasilache. · View Herald Transcript
ftynse accepted this revision.Jan 11 2021, 4:44 AM
This revision is now accepted and ready to land.Jan 11 2021, 4:44 AM
Harbormaster completed remote builds in B84663: Diff 315760.Jan 11 2021, 5:17 AM
csigg updated this revision to Diff 315775.Jan 11 2021, 5:46 AM

Fix SPIRV lowering.

Herald added a reviewer: mravishankar. · View Herald TranscriptJan 11 2021, 5:46 AM
Herald added a reviewer: antiagainst. · View Herald Transcript
Herald added subscribers: ThomasRaoux, AlexeySotkin. · View Herald Transcript
Harbormaster completed remote builds in B84673: Diff 315775.Jan 11 2021, 6:20 AM
Closed by commit rGd59ddba77725: [mlir] Fix gpu-to-llvm lowering for gpu.alloc with dynamic sizes. (authored by csigg). · Explain WhyJan 11 2021, 6:56 AM
This revision was automatically updated to reflect the committed changes.
csigg added a commit: rGd59ddba77725: [mlir] Fix gpu-to-llvm lowering for gpu.alloc with dynamic sizes..

Revision Contents
Changeset List

PathSize
mlir/
include/
mlir/
Conversion/
StandardToLLVM/
2 lines
lib/
Conversion/
GPUCommon/
6 lines
StandardToLLVM/
8 lines
test/
Conversion/
GPUCommon/
11 lines

Diff 315760

View Options

mlir/include/mlir/Conversion/StandardToLLVM/ConvertStandardToLLVM.h

Loading...
View Options

mlir/lib/Conversion/GPUCommon/ConvertLaunchFuncToRuntimeCalls.cpp

Loading...
View Options

mlir/lib/Conversion/StandardToLLVM/StandardToLLVM.cpp

Loading...
View Options

mlir/test/Conversion/GPUCommon/lower-alloc-to-gpu-runtime-calls.mlir

Loading...