On Thu, May 16, 2019 at 1:18 PM Martin Liška <mli...@suse.cz> wrote: > > Hi. > > We should not allow target_clones being combined with alias attribute. > > Patch can bootstrap on x86_64-linux-gnu and survives regression tests. > > Ready to be installed?
So that's because an alias cannot be turned into a dispatcher and still be an alias, correct? So a way around this would be to turn the alias into the dispatcher and clone the alias target, leaving the plain alias target as default variant not going through the dispatcher? Of course in the testcase the body of the alias target isn't available but that's a general issue, not special to aliases? Richard. > Thanks, > Martin > > gcc/ChangeLog: > > 2019-05-16 Martin Liska <mli...@suse.cz> > > PR lto/90500 > * multiple_target.c (expand_target_clones): Do not allow > target_clones being used with a symbol that is an alias. > > gcc/testsuite/ChangeLog: > > 2019-05-16 Martin Liska <mli...@suse.cz> > > PR lto/90500 > * gcc.target/i386/pr90500-1.c: New test. > * gcc.target/i386/pr90500-2.c: New test. > --- > gcc/multiple_target.c | 5 ++++- > gcc/testsuite/gcc.target/i386/pr90500-1.c | 8 ++++++++ > gcc/testsuite/gcc.target/i386/pr90500-2.c | 7 +++++++ > 3 files changed, 19 insertions(+), 1 deletion(-) > create mode 100644 gcc/testsuite/gcc.target/i386/pr90500-1.c > create mode 100644 gcc/testsuite/gcc.target/i386/pr90500-2.c > >
