[PATCH] rs6000: Remove builtin mask check from builtin_decl [PR102347]
Peter Bergner
bergner@linux.ibm.com
Wed Sep 29 00:25:16 GMT 2021
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
More information about the Gcc-patches
mailing list