On Mon, Mar 20, 2023 at 01:43:41PM -0400, Michael Meissner wrote: > On Fri, Mar 17, 2023 at 02:35:16PM -0500, Segher Boessenkool wrote: > > On Thu, Mar 09, 2023 at 08:40:36PM -0500, Michael Meissner wrote: > > /* { dg-final { scan-assembler {\m__divtc3\M} } } */ > > > > It might well be that we can use a sloppier regexp here, but why would > > we do that? It is a good thing to use the \m and \M constraint escapes > > pretty much always. > > The last time I posted the patch, you said: > > | > +/* { dg-final { scan-assembler "bl __divtc3" } } */ > | > | This name depends on what object format and ABI is in use (some have an > | extra leading underscore, or a dot, or whatever). > > So the patch was an attempt to match the other cases.
I also said you could do {\mbl .?__divtc3\M} or similar, in cases where you need to consider multiple ABIs. Just searching for a substring is very suboptimal always. > > Does this need backports? > > I think we will need backports for GCC 12. The issue exists in GCC 11, but I > don't think that GCC 11 can really work on systems with IEEE long double, > since > a lot of the stuff to really finish up the support was not in GCC 11. I think > I tried dropping the patch into GCC 12, and it looks like something else may > be > needed. I will look into it. Okay, thanks. We'll see. Segher