diff --git a/rocclr/compiler/lib/utils/v0_8/libUtils.cpp b/rocclr/compiler/lib/utils/v0_8/libUtils.cpp index 1f89e38013..136ab1b329 100644 --- a/rocclr/compiler/lib/utils/v0_8/libUtils.cpp +++ b/rocclr/compiler/lib/utils/v0_8/libUtils.cpp @@ -459,20 +459,23 @@ appendLogToCL(aclCompiler *cl, const std::string &logStr) if (logStr.empty()) { return; } - unsigned size = logStr.size() + cl->logSize; + std::string log = logStr; + if ('\n' != log[log.size()-1]) { + log.append("\n"); + } + unsigned size = cl->logSize + log.size(); if (!size) { return; } char *tmpBuildLog = reinterpret_cast(aclutAlloc(cl)(size + 2)); memset(tmpBuildLog, 0, size + 2); if (cl->logSize) { - std::copy(cl->buildLog,cl->buildLog + cl->logSize, tmpBuildLog); - std::copy(logStr.begin(), logStr.end(), tmpBuildLog + cl->logSize + 1); - tmpBuildLog[cl->logSize] = '\n'; + std::copy(cl->buildLog, cl->buildLog + cl->logSize, tmpBuildLog); + std::copy(log.begin(), log.end(), tmpBuildLog + cl->logSize); } else { - std::copy(logStr.begin(), logStr.end(), tmpBuildLog); + std::copy(log.begin(), log.end(), tmpBuildLog); } - cl->logSize += (unsigned int)logStr.size(); + cl->logSize += (unsigned int)log.size(); if (cl->buildLog) { aclutFree(cl)(cl->buildLog); } diff --git a/rocclr/runtime/device/device.hpp b/rocclr/runtime/device/device.hpp index 8d3ed27d1f..66930f93f6 100644 --- a/rocclr/runtime/device/device.hpp +++ b/rocclr/runtime/device/device.hpp @@ -918,12 +918,15 @@ public: workGroupInfo_.preferredSizeMultiple_ = size; } + //! Return the build log + const std::string& buildLog() const { return buildLog_; } + protected: std::string name_; //!< kernel name WorkGroupInfo workGroupInfo_; //!< device kernel info structure amd::KernelSignature* signature_; //!< kernel signature bool hsa_; //!< True if HSA kernel on GPU - + std::string buildLog_; //!< build log private: //! Disable default copy constructor Kernel(const Kernel&); diff --git a/rocclr/runtime/device/gpu/gpukernel.cpp b/rocclr/runtime/device/gpu/gpukernel.cpp index cdf91b789a..39661c7bf5 100644 --- a/rocclr/runtime/device/gpu/gpukernel.cpp +++ b/rocclr/runtime/device/gpu/gpukernel.cpp @@ -3524,6 +3524,7 @@ HSAILKernel::init() void* shader_isa = NULL; error = aclCompile(dev().hsaCompiler(), prog().binaryElf(), options.c_str(), ACL_TYPE_CG, ACL_TYPE_ISA, NULL); + buildLog_ += aclGetCompilerLog(dev().hsaCompiler()); if (error != ACL_SUCCESS) { LogError("Failed to finalize"); return false; diff --git a/rocclr/runtime/device/gpu/gpukernel.hpp b/rocclr/runtime/device/gpu/gpukernel.hpp index 10b3ec0e10..b2558bc3c7 100644 --- a/rocclr/runtime/device/gpu/gpukernel.hpp +++ b/rocclr/runtime/device/gpu/gpukernel.hpp @@ -424,9 +424,6 @@ public: //! Returns GPU device object, associated with this kernel const NullProgram& nullProg() const { return prog_; } - //! Returns the kernel's build log - const std::string& buildLog() const { return buildLog_; } - //! Returns the kernel's build error const cl_int buildError() const { return buildError_; } @@ -477,7 +474,6 @@ protected: //! Returns UAV arena index for this kernel uint uavArena() const { return uavArena_; } - std::string buildLog_; //!< Kernel's build log cl_int buildError_; //!< Kernel's build error std::string ilSource_; //!< IL source code of this kernel diff --git a/rocclr/runtime/device/gpu/gpuprogram.cpp b/rocclr/runtime/device/gpu/gpuprogram.cpp index 26ea61b683..896e307d8c 100644 --- a/rocclr/runtime/device/gpu/gpuprogram.cpp +++ b/rocclr/runtime/device/gpu/gpuprogram.cpp @@ -2049,6 +2049,7 @@ HSAILProgram::linkImpl(amd::option::Options* options) std::string curOptions = options->origOptionStr + hsailOptions(); errorCode = aclCompile(dev().hsaCompiler(), binaryElf_, curOptions.c_str(), continueCompileFrom, ACL_TYPE_CG, NULL); + buildLog_ += aclGetCompilerLog(dev().hsaCompiler()); break; } } @@ -2108,6 +2109,7 @@ HSAILProgram::linkImpl(amd::option::Options* options) if (!aKernel->init() ) { return false; } + buildLog_ += aKernel->buildLog(); aKernel->setUniformWorkGroupSize(options ->oVariables->UniformWorkGroupSize); kernels()[kernelName] = aKernel;