This is an archive of the discontinued LLVM Phabricator instance.

Differential D80353

[mlir][mlir-cuda-runner] Fix obvious unit test error introduced in D80167. NFC.
AbandonedPublic

Authored by whchung on May 20 2020, 8:07 PM.

Details

Reviewers
rsmith

Diff Detail

Event Timeline

whchung created this revision.May 20 2020, 8:07 PM
Herald added a project: Restricted Project. · View Herald TranscriptMay 20 2020, 8:08 PM
Herald added subscribers: llvm-commits, jurahul, Kayjukh and 15 others. · View Herald Transcript
whchung retitled this revision from [mlir][mlir-cuda-runner] Fix obvious unit test error introduced in D80167. to [mlir][mlir-cuda-runner] Fix obvious unit test error introduced in D80167. NFC..May 20 2020, 8:19 PM
whchung abandoned this revision.May 20 2020, 9:04 PM
Harbormaster failed remote builds in B57475: Diff 265411!May 20 2020, 9:29 PM
whchung mentioned this in D80167: [mlir][gpu] Refactor ConvertGpuLaunchFuncToCudaCalls pass..May 20 2020, 9:51 PM

Revision Contents

PathSize
mlir/
tools/
mlir-cuda-runner/
4 lines

Diff 265411

mlir/tools/mlir-cuda-runner/cuda-runtime-wrappers.cpp

Show All 31 Lines
extern "C" int32_t mgpuModuleLoad(void **module, void *data) { extern "C" int32_t mgpuModuleLoad(void **module, void *data) {
int32_t err = reportErrorIfAny( int32_t err = reportErrorIfAny(
cuModuleLoadData(reinterpret_cast<CUmodule *>(module), data), cuModuleLoadData(reinterpret_cast<CUmodule *>(module), data),
"ModuleLoad"); "ModuleLoad");
return err; return err;
} }
extern "C" int32_t mcuModuleGetFunction(void **function, void *module, extern "C" int32_t mgpuModuleGetFunction(void **function, void *module,
const char *name) { const char *name) {
return reportErrorIfAny( return reportErrorIfAny(
cuModuleGetFunction(reinterpret_cast<CUfunction *>(function), cuModuleGetFunction(reinterpret_cast<CUfunction *>(function),
reinterpret_cast<CUmodule>(module), name), reinterpret_cast<CUmodule>(module), name),
"GetFunction"); "GetFunction");
} }
// The wrapper uses intptr_t instead of CUDA's unsigned int to match // The wrapper uses intptr_t instead of CUDA's unsigned int to match
// the type of MLIR's index type. This avoids the need for casts in the // the type of MLIR's index type. This avoids the need for casts in the
▲ Show 20 Lines • Show All 68 Lines • Show Last 20 Lines