Re: [patch,i386] Cannot inline sse*.* functions into avx functions

2013-07-02 Thread Sriraman Tallam
On Mon, Jul 1, 2013 at 2:12 PM, Uros Bizjak wrote: > On Mon, Jul 1, 2013 at 8:01 PM, Sriraman Tallam wrote: > >>So, something like the patch attached? >> >> * config/i386/i386.c (ix86_option_override_internal): Turn >> on all -mavx target flags by default as they are dependent

Re: [patch,i386] Cannot inline sse*.* functions into avx functions

2013-07-01 Thread Uros Bizjak
On Mon, Jul 1, 2013 at 8:01 PM, Sriraman Tallam wrote: >So, something like the patch attached? > > * config/i386/i386.c (ix86_option_override_internal): Turn > on all -mavx target flags by default as they are dependent > on AVX anyway. Yes, but please also add: Index

Re: [patch,i386] Cannot inline sse*.* functions into avx functions

2013-07-01 Thread Sriraman Tallam
Hi, So, something like the patch attached? * config/i386/i386.c (ix86_option_override_internal): Turn on all -mavx target flags by default as they are dependent on AVX anyway. Thanks Sri On Sun, Jun 30, 2013 at 2:56 AM, Uros Bizjak wrote: > On Sun, Jun 30, 2013 at

Re: [patch,i386] Cannot inline sse*.* functions into avx functions

2013-06-30 Thread Uros Bizjak
On Sun, Jun 30, 2013 at 11:47 AM, Jan Hubicka wrote: >> >> What target flags are enabled by AVX? Assumming that all target flags are >> positive seems incorrect to me (like -mno-red-zone function can not be >> inlined >> into -mred-zone). Does those conditionally enabled AVX codegen flags have

Re: [patch,i386] Cannot inline sse*.* functions into avx functions

2013-06-30 Thread Jan Hubicka
> > What target flags are enabled by AVX? Assumming that all target flags are > positive seems incorrect to me (like -mno-red-zone function can not be inlined > into -mred-zone). Does those conditionally enabled AVX codegen flags have any Actually -mred-zone seems right, but stuff like -msseregp

Re: [patch,i386] Cannot inline sse*.* functions into avx functions

2013-06-30 Thread Jan Hubicka
> On Sat, Jun 29, 2013 at 12:55 AM, Sriraman Tallam wrote: > > > Inlining sse* functions into avx is broken? Here is an example: > > > > __attribute__((always_inline,target("sse3"))) > > inline int callee () > > { > > return 0; > > } > > > > __attribute__((target("avx"))) > > inline int caller

Re: [patch,i386] Cannot inline sse*.* functions into avx functions

2013-06-30 Thread Uros Bizjak
On Sat, Jun 29, 2013 at 12:55 AM, Sriraman Tallam wrote: > Inlining sse* functions into avx is broken? Here is an example: > > __attribute__((always_inline,target("sse3"))) > inline int callee () > { > return 0; > } > > __attribute__((target("avx"))) > inline int caller () > { > return calle

[patch,i386] Cannot inline sse*.* functions into avx functions

2013-06-28 Thread Sriraman Tallam
Hi, Inlining sse* functions into avx is broken? Here is an example: __attribute__((always_inline,target("sse3"))) inline int callee () { return 0; } __attribute__((target("avx"))) inline int caller () { return callee (); } main () { return caller (); } $ g++ -O2 foo.cc error: inlining f