Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(11)

Issue 4559064: [pph] Add new C test case (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
13 years, 10 months ago by Diego Novillo
Modified:
13 years, 10 months ago
Reviewers:
CC:
Lawrence Crowl, collinwinter, gcc-patches_gcc.gnu.org
Visibility:
Public.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+17 lines, -0 lines) Patch
A gcc/testsuite/g++.dg/pph/c1return-5.h View 1 chunk +16 lines, -0 lines 0 comments Download
A gcc/testsuite/g++.dg/pph/c1return-5.cc View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 1
Diego Novillo
13 years, 10 months ago (2011-06-01 15:41:12 UTC) #1
This test case from the C testsuite started working after the last
merge from trunk.

Collin, this is the test I was referring to yesterday.  I'm going to
add some more C test cases to the testsuite that are currently not
working.  I think that's going to make our fixing job easier
(otherwise, it's not trivial to test these test cases outside my own
local tree).

Tested on x86_64.  Committed to pph.

Diego.


	* g++.dg/pph/c1return-5.cc: New.
	* g++.dg/pph/c1return-5.h: New.

diff --git a/gcc/testsuite/g++.dg/pph/c1return-5.cc
b/gcc/testsuite/g++.dg/pph/c1return-5.cc
new file mode 100644
index 0000000..804e113
--- /dev/null
+++ b/gcc/testsuite/g++.dg/pph/c1return-5.cc
@@ -0,0 +1 @@
+#include "c1return-5.h"
diff --git a/gcc/testsuite/g++.dg/pph/c1return-5.h
b/gcc/testsuite/g++.dg/pph/c1return-5.h
new file mode 100644
index 0000000..8b72a51
--- /dev/null
+++ b/gcc/testsuite/g++.dg/pph/c1return-5.h
@@ -0,0 +1,16 @@
+#ifndef __PPH_GUARD_H
+#define __PPH_GUARD_H
+/* { dg-options "-mpreferred-stack-boundary=4" } */
+/* { dg-final { scan-assembler-not "and\[lq\]?\[^\\n\]*-64,\[^\\n\]*sp" } } */
+
+/* This compile only test is to detect an assertion failure in stack branch
+   development.  */
+struct bar
+{
+  int x;
+} __attribute__((aligned(64)));
+
+
+struct bar
+foo (void) { }
+#endif

--
This patch is available for review at http://codereview.appspot.com/4559064
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b