diff mbox

[hsa,testsuite,3/5] Suppress hsa warnings in libgomp tests

Message ID f6a2e3cb9f6eea4cf5df80920cc796621d663f83.1457369363.git.mjambor@suse.cz
State New
Headers show

Commit Message

Martin Jambor Feb. 29, 2016, 4:22 p.m. UTC 
Hi,

just like with the compiler gomp testsuite, we need to add -Wno-hsa to
options when compiling libgomp testcases in order not to have "excess
errors" failures when HSA is enabled.  There are quite many of such
testcases on the trunk because I have disabled the dynamic parallelism
way of executing stuff.

The patch below adds the option to all libgomp testsuite compilations,
so that people who are not interested in HSA do not need to care.  The
patch has been tested both with and without HSA enabled.  OK for
trunk?

Thanks,

Martin


2016-03-04  Martin Jambor  <mjambor@suse.cz>

	* testsuite/lib/libgomp.exp (libgomp_init): Append -Wno-hsa to
	ALWAYS_CFLAGS.
---
 libgomp/testsuite/lib/libgomp.exp | 3 +++
 1 file changed, 3 insertions(+)

Comments

Jakub Jelinek March 7, 2016, 5:57 p.m. UTC | #1 
On Mon, Feb 29, 2016 at 05:22:36PM +0100, Martin Jambor wrote:
> 2016-03-04  Martin Jambor  <mjambor@suse.cz>
> 
> 	* testsuite/lib/libgomp.exp (libgomp_init): Append -Wno-hsa to
> 	ALWAYS_CFLAGS.

Ok.

	Jakub
diff mbox

Patch

diff --git a/libgomp/testsuite/lib/libgomp.exp b/libgomp/testsuite/lib/libgomp.exp
index 154a447..bbc2c26 100644
--- a/libgomp/testsuite/lib/libgomp.exp
+++ b/libgomp/testsuite/lib/libgomp.exp
@@ -237,6 +237,9 @@  proc libgomp_init { args } {
     # Disable caret
     lappend ALWAYS_CFLAGS "additional_flags=-fno-diagnostics-show-caret"
 
+    # Disable HSA warnings by default.
+    lappend ALWAYS_CFLAGS "additional_flags=-Wno-hsa"
+
     # Disable color diagnostics
     lappend ALWAYS_CFLAGS "additional_flags=-fdiagnostics-color=never"