SWDEV-245381 : Updated guarantee error messages to print based on BUILD_TYPE

Change-Id: Ia21039326b440f6d807a6495a9a05dd52b384c76
This commit is contained in:
agodavar
2021-01-05 04:53:59 -05:00
parent 7bcd555602
commit 69a786e8d1
5 changed files with 16 additions and 9 deletions
+2 -2
View File
@@ -98,7 +98,7 @@ void MemObjMap::AddMemObj(const void* k, amd::Memory* v) {
if (!rval.second) {
DevLogPrintfError("Memobj map already has an entry for ptr: 0x%x",
reinterpret_cast<uintptr_t>(k));
guarantee(false);
guarantee(false, "Memobj map already has an entry for ptr");
}
}
@@ -108,7 +108,7 @@ void MemObjMap::RemoveMemObj(const void* k) {
if (rval != 1) {
DevLogPrintfError("Memobj map does not have ptr: 0x%x",
reinterpret_cast<uintptr_t>(k));
guarantee(false);
guarantee(false, "Memobj map does not have ptr");
}
}
+2 -2
View File
@@ -30,14 +30,14 @@ namespace amd {
Kernel::Kernel(Program& program, const Symbol& symbol, const std::string& name)
: program_(program), symbol_(symbol), name_(name) {
parameters_ = new (signature()) KernelParameters(const_cast<KernelSignature&>(signature()));
fixme_guarantee(parameters_ != NULL && "out of memory");
fixme_guarantee(parameters_ != NULL, "out of memory");
name_ += '\0';
}
Kernel::Kernel(const Kernel& rhs)
: program_(rhs.program_()), symbol_(rhs.symbol_), name_(rhs.name_) {
parameters_ = new(signature()) KernelParameters(*rhs.parameters_);
fixme_guarantee(parameters_ != NULL && "out of memory");
fixme_guarantee(parameters_ != NULL, "out of memory");
}
Kernel::~Kernel() {
+1 -1
View File
@@ -175,7 +175,7 @@ int32_t Program::addDeviceProgram(Device& device, const void* image, size_t leng
const device::Program* same_dev_prog = nullptr;
if ((amd::IS_HIP) && (same_prog != nullptr)) {
auto same_dev_prog_map_ = same_prog->devicePrograms();
guarantee(same_dev_prog_map_.size() == 1);
guarantee(same_dev_prog_map_.size() == 1, "For same_prog, devicePrograms size != 1");
same_dev_prog = same_dev_prog_map_.begin()->second;
}
+5 -1
View File
@@ -46,7 +46,11 @@ extern "C" void breakpoint(void) {
void report_fatal(const char* file, int line, const char* message) {
// FIXME_lmoriche: Obfuscate the message string
fprintf(stderr, "%s:%d: %s\n", file, line, message);
#if (defined(DEBUG))
fprintf(stderr, "%s:%d: %s\n", file, line, message);
#else
fprintf(stderr, "%s\n", message);
#endif
::abort();
}
+6 -3
View File
@@ -90,13 +90,16 @@ extern void log_printf(LogLevel level, const char* file, int line, uint64_t *sta
#endif // __INTEL_COMPILER
//! \brief Abort the program if the invariant \a cond is false.
#define guarantee(cond) \
#define guarantee(cond, ...) \
if (!(cond)) { \
amd::report_fatal(__FILE__, __LINE__, "guarantee(" XSTR(cond) ")"); \
if(strlen(#__VA_ARGS__) == 0) \
amd::report_fatal(__FILE__, __LINE__, XSTR(cond) ); \
else \
amd::report_fatal(__FILE__, __LINE__, XSTR(__VA_ARGS__) ); \
amd::breakpoint(); \
}
#define fixme_guarantee(cond) guarantee(cond)
#define fixme_guarantee(cond, ...) guarantee(cond, __VA_ARGS__)
//! \brief Abort the program with a fatal error message.
#define fatal(msg) \