On 9/28/21 2:24 PM, Bill Schmidt via Gcc-patches wrote: > Unless you are planning to do a backport, I think the proper way forward > here is to just wait for the new builtin support to land. In the new code, > we initialize all built-ins up front, and check properly at expansion time > whether the builtin is enabled in the environment that obtains during expand.
Bill, have you tried the test case in the bugzilla with your builtin rewrite and it works? If so, then I think the correct thing would be to skip "fixing" trunk and wait for your builtin rewrite to land there. That said, this does fail on GCC11 and GCC10 (not sure about earlier, haven't tried them yet), so we will need some type of fix for the releases. I do think your concern about not having some checking elsewhere is valid, unless there already is some checking there and you and I are just not aware of it. Peter
