On Thu, Apr 25, 2019 at 9:51 AM Martin Liška <mli...@suse.cz> wrote:
> On 4/23/19 10:29 AM, Martin Liška wrote: > > On 4/18/19 7:44 PM, H.J. Lu wrote: > >> On Thu, Apr 18, 2019 at 4:07 AM Martin Liška <mli...@suse.cz> wrote: > >>> > >>> Hi. > >>> > >>> I'm sending updated version of that patch. The patch rejects usage of > AVX512 ISAs (except AVX512F) > >>> for target attribute for C++ multiversioning and for target_clone > attribute. > >>> > >>> Patch can bootstrap on x86_64-linux-gnu and survives regression tests. > >>> > >>> Ready to be installed? > >>> Thanks, > >>> Martin > >> > >> Your patch doesn't handle cmov nor gfni properly, which aren't AVX512. > >> I prefer this patch. > >> > > > > I like the patch. Thanks for working on that. > > > > Martin > > > > > > Btw. can we get that patch into GCC 9.1? > > Adding port maintainers to CC. > HJ knows ISA interdependencies, and the benefit of the patch outweights the (small) risk, so from the maintaner PoV, OK for the mainline unless RM vetoes the decision soon. HJ, please double check the patch for eventual inconsistencies or possible regressions before committing. Thanks, Uros.
