Hi! On Thu, Sep 24, 2020 at 03:35:24PM -0500, will schmidt wrote: > We have extraneous BTM entry (RS6000_BTM_POWERPC64) in the define for > our P10 MISC 2 builtin definition. This does not exist for the '0', > '1' or '3' definitions. It appears to me that this was erroneously > copied from the P7 version of the define which contains a version of the > BU macro both with and without that element. Removing the > RS6000_BTM_POWERPC64 portion of the define does not introduce any obvious > failures, I believe this extra line can be safely removed.
No, it cannot. This is used for pdepd/pextd/cntlzdm/cnttzdm/cfuged, all of which do need 64-bit registers to do anything sane. This should really have defined some new builtin class, and I thought we could just be tricky and take a massive shortcut. Bill has been hit by this already as well, sigh :-( Segher
