OK I've have a look, thanks
Christian On 04/30/2015 10:27 AM, Bin.Cheng wrote: > On Mon, Apr 27, 2015 at 8:01 PM, Uros Bizjak <ubiz...@gmail.com> wrote: >> On Wed, Feb 4, 2015 at 2:21 PM, Christian Bruel <christian.br...@st.com> >> wrote: >>> While trying to reduce the PR64835 case for ARM and x86, I noticed that the >>> alignment flags are cleared for x86 when attribute optimized is used. >>> >>> With the attached testcases, the visible effects are twofold : >>> >>> 1) Functions compiled in with attribute optimize (-O2) are not aligned as if >>> they were with the -O2 flag. >>> >>> 2) can_inline_edge_p fails because opts_for_fn (caller->decl) != opts_for_fn >>> (callee->decl)) even-though they are compiled with the same optimization >>> level. >> >> 2015-02-06 Christian Bruel <christian.br...@st.com> >> >> PR target/64835 >> * config/i386/i386.c (ix86_default_align): New function. >> (ix86_override_options_after_change): Call ix86_default_align. >> (TARGET_OVERRIDE_OPTIONS_AFTER_CHANGE): New hook. >> (ix86_override_options_after_change): New function. >> >> 2015-02-06 Christian Bruel <christian.br...@st.com> >> >> PR target/64835 >> * gcc.dg/ipa/iinline-attr.c: New test. >> * gcc.target/i386/iinline-attr-2.c: New test. >> >> OK for mainline. > > Hi Christian, > I noticed case gcc.dg/ipa/iinline-attr.c failed on aarch64. The > original patch is x86 specific, while the case is added as general > one. Could you please have a look at this? > > FAIL: gcc.dg/ipa/iinline-attr.c scan-ipa-dump inline > "hooray[^\\n]*inline copy in test" > > Thanks, > bin >> >> Thanks, >> Uros
