On 12/1/21 11:21 AM, Segher Boessenkool wrote: > Hi! > > On Wed, Dec 01, 2021 at 09:29:42AM -0600, Bill Schmidt wrote: >> Recently Kewen fixed a problem in the old builtins support where >> rs6000_builtin_decl prematurely indicated that a target builtin is >> unavailable. This also needs to be done for the new builtins support, but in >> this case we have to ensure the error message is still produced from the >> overload support in rs6000-c.c. Unfortunately, this is less straightforward >> than it could be, because header file includes need to be adjusted to make >> this >> happen. Someday we'll consolidate all the builtin code in one file and this >> won't have to be so ugly. >> >> Bootstrapped and tested on powerpc64le-linux-gnu with no regressions. Is >> this >> okay for trunk? > This is okay for trunk. Thanks! > > Is there some place we can store what original builtin was used when > some overload is resolved? Just in the new builtin code, don't spend > time on the old stuff :-)
I think we can do much better, with a little work. The macros are a little problematic, but I have ideas about how to make this better in a future patch. (There's a fair amount of test suite fallout, so it should wait.) Thanks for the review! Bill > > > Segher