Re: [PATCH][GCC] Make stackalign test LTO proof

2015-12-04 Thread Bernd Schmidt
On 12/04/2015 04:18 PM, Andre Vieira wrote: Reworked following Joern's suggestion. Is this OK? Yes. Bernd

Re: [PATCH][GCC] Make stackalign test LTO proof

2015-12-04 Thread Andre Vieira
On 17/11/15 16:30, Andre Vieira wrote: On 17/11/15 12:29, Bernd Schmidt wrote: On 11/16/2015 04:48 PM, Andre Vieira wrote: On 16/11/15 15:34, Joern Wolfgang Rennecke wrote: I just happened to stumble on this problem with another port. The volatile & test solution doesn't work, though. What do

Re: [PATCH][GCC] Make stackalign test LTO proof

2015-11-17 Thread Andre Vieira
On 17/11/15 12:29, Bernd Schmidt wrote: On 11/16/2015 04:48 PM, Andre Vieira wrote: On 16/11/15 15:34, Joern Wolfgang Rennecke wrote: I just happened to stumble on this problem with another port. The volatile & test solution doesn't work, though. What does work, however, is: __asm__ ("" : : "

Re: [PATCH][GCC] Make stackalign test LTO proof

2015-11-17 Thread Bernd Schmidt
On 11/16/2015 04:48 PM, Andre Vieira wrote: On 16/11/15 15:34, Joern Wolfgang Rennecke wrote: I just happened to stumble on this problem with another port. The volatile & test solution doesn't work, though. What does work, however, is: __asm__ ("" : : "" (dummy)); I can confirm that Joern's

Re: [PATCH][GCC] Make stackalign test LTO proof

2015-11-16 Thread Andre Vieira
On 16/11/15 15:34, Joern Wolfgang Rennecke wrote: I just happened to stumble on this problem with another port. The volatile & test solution doesn't work, though. What does work, however, is: __asm__ ("" : : "" (dummy)); I can confirm that Joern's solution works for me too.

Re: [PATCH][GCC] Make stackalign test LTO proof

2015-11-16 Thread Joern Wolfgang Rennecke
I just happened to stumble on this problem with another port. The volatile & test solution doesn't work, though. What does work, however, is: __asm__ ("" : : "" (dummy));

Re: [PATCH][GCC] Make stackalign test LTO proof

2015-11-16 Thread Richard Biener
On Mon, Nov 16, 2015 at 3:08 PM, Andre Vieira wrote: > On 16/11/15 13:33, Richard Biener wrote: >> >> On Mon, Nov 16, 2015 at 12:43 PM, Andre Vieira >> wrote: >>> >>> On 13/11/15 10:34, Richard Biener wrote: On Thu, Nov 12, 2015 at 4:07 PM, Andre Vieira wrote: > > >>>

Re: [PATCH][GCC] Make stackalign test LTO proof

2015-11-16 Thread Andre Vieira
On 16/11/15 13:33, Richard Biener wrote: On Mon, Nov 16, 2015 at 12:43 PM, Andre Vieira wrote: On 13/11/15 10:34, Richard Biener wrote: On Thu, Nov 12, 2015 at 4:07 PM, Andre Vieira wrote: Hi, This patch changes this testcase to make sure LTO will not optimize away the assignment of t

Re: [PATCH][GCC] Make stackalign test LTO proof

2015-11-16 Thread Richard Biener
On Mon, Nov 16, 2015 at 12:43 PM, Andre Vieira wrote: > On 13/11/15 10:34, Richard Biener wrote: >> >> On Thu, Nov 12, 2015 at 4:07 PM, Andre Vieira >> wrote: >>> >>> Hi, >>> >>>This patch changes this testcase to make sure LTO will not optimize >>> away >>> the assignment of the local array

Re: [PATCH][GCC] Make stackalign test LTO proof

2015-11-16 Thread Andre Vieira
On 13/11/15 10:34, Richard Biener wrote: On Thu, Nov 12, 2015 at 4:07 PM, Andre Vieira wrote: Hi, This patch changes this testcase to make sure LTO will not optimize away the assignment of the local array to a global variable which was introduced to make sure stack space was made available

Re: [PATCH][GCC] Make stackalign test LTO proof

2015-11-13 Thread Richard Biener
On Thu, Nov 12, 2015 at 4:07 PM, Andre Vieira wrote: > Hi, > > This patch changes this testcase to make sure LTO will not optimize away > the assignment of the local array to a global variable which was introduced > to make sure stack space was made available for the test to work. > > This is