On Tue, Dec 09, 2014 at 03:32:33PM +0300, Ilya Verbin wrote:
> On 04 Dec 20:52, Jakub Jelinek wrote:
> > On Thu, Dec 04, 2014 at 10:35:19PM +0300, Ilya Verbin wrote:
> > > This issue can be resolved by forcing output of such variables.
> > > Is this fix ok?  Should I add a testcase?
> > 
> > Yes, with proper ChangeLog.  Yes.
> 
> Here is updated patch, ok to commit?
> 
> However, I don't see -flto option in the build log.  It seems that
> check_effective_target_lto isn't working inside libgomp/ directory.
> Maybe because ENABLE_LTO is defined only in gcc/configure.ac ?
> 
> 
> gcc/
>       * varpool.c (varpool_node::get_create): Force output of vars with
>       "omp declare target" attribute.
> libgomp/
>       * testsuite/libgomp.c/target-9.c: New test.

Ok, though please try to find out why effective target lto check doesn't
work in libgomp.  Perhaps you just need to include some further *.exp
file?

        Jakub

Reply via email to