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