Added exception handling for allocation on CPU (74ce83d0) · Commits · TNL / tnl-dev

src/Benchmarks/SpMV/spmv.h

+18 −8

Original line number	Diff line number	Diff line
		@@ -102,15 +102,20 @@ benchmarkSpMV( Benchmark & benchmark,
		// incorrect formatting: The "!" marks are not at the same line
		// as the message and sometimes they're omitted altogether.
		// benchmark.addErrorMessage( "Failed to read matrix!", 1 );

		// CORRECT? MatrixReader can fail for other reasons than Host Allocation issues, is this throw ok?
		throw Exceptions::HostBadAlloc();
		return false;
		}
		}
		catch( std::bad_alloc )
		// HOW? How does this work if the "if" statement above fails.
		catch( Exceptions::HostBadAlloc e )
		{
		// FIXME: Adds the message to the log file, HOWEVER, it does so with
		// incorrect formatting: The "!" marks are not at the same line
		// as the message and sometimes they're omitted altogether.
		// benchmark.addErrorMessage( "Failed to allocate memory for matrix!", 1 );
		e.what();
		return false;
		}

		@@ -150,15 +155,20 @@ benchmarkSpMV( Benchmark & benchmark,
		// incorrect formatting: The "!" marks are not at the same line
		// as the message and sometimes they're omitted altogether.
		// benchmark.addErrorMessage( "Failed to read matrix!", 1 );

		// CORRECT? MatrixReader can fail for other reasons than Host Allocation issues, is this throw ok?
		throw Exceptions::HostBadAlloc();
		return false;
		}
		}
		catch( std::bad_alloc )
		// HOW? How does this work if the "if" statement above fails.
		catch( Exceptions::HostBadAlloc e )
		{
		// FIXME: Adds the message to the log file, HOWEVER, it does so with
		// incorrect formatting: The "!" marks are not at the same line
		// as the message and sometimes they're omitted altogether.
		// benchmark.addErrorMessage( "Failed to allocate memory for matrix!", 1 );
		e.what();
		return false;
		}