On Fri, Aug 23, 2024 at 5:50 AM Richard Biener <richard.guent...@gmail.com> wrote: > > On Fri, Aug 23, 2024 at 2:36 PM H.J. Lu <hjl.to...@gmail.com> wrote: > > > > obj.found is the number of LTO symbols. We should include the offload > > section when it is used by linker even if there are no LTO symbols. > > OK. > > > PR lto/116361 > > * lto-plugin.c (claim_file_handler_v2): Don't check obj.found > > for the offload section. > > > > Signed-off-by: H.J. Lu <hjl.to...@gmail.com> > > --- > > lto-plugin/lto-plugin.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/lto-plugin/lto-plugin.c b/lto-plugin/lto-plugin.c > > index 61b0de62f52..c564b36eb92 100644 > > --- a/lto-plugin/lto-plugin.c > > +++ b/lto-plugin/lto-plugin.c > > @@ -1320,7 +1320,7 @@ claim_file_handler_v2 (const struct > > ld_plugin_input_file *file, > > if (*can_be_claimed && !obj.offload && offload_files_last_lto == NULL) > > offload_files_last_lto = offload_files_last; > > > > - if (obj.offload && known_used && obj.found > 0) > > + if (obj.offload && known_used) > > { > > /* Add file to the list. The order must be exactly the same as the > > final > > order after recompilation and linking, otherwise host and target > > tables > > -- > > 2.46.0 > >
OK to backport https://gcc.gnu.org/git/?p=gcc.git;a=commit;h=a98dd536b1017c2b814a3465206c6c01b2890998 and this patch to release branches? Thanks. -- H.J.