Re: [PATCH] AArch64: Add support for __builtin_roundeven[f] [PR100966]

2021-06-23 Thread Richard Sandiford via Gcc-patches
Wilco Dijkstra writes: > Hi Richard, > >> So rather than have two patterns that generate frintn, I think >> it would be better to change the existing frint_pattern entry to >> "roundeven" instead, and fix whatever the fallout is. Hopefully it >> shouldn't be too bad, since we already use the opta

Re: [PATCH] AArch64: Add support for __builtin_roundeven[f] [PR100966]

2021-06-22 Thread Wilco Dijkstra via Gcc-patches
Hi Richard, > So rather than have two patterns that generate frintn, I think > it would be better to change the existing frint_pattern entry to > "roundeven" instead, and fix whatever the fallout is. Hopefully it > shouldn't be too bad, since we already use the optab names for the > other UNSPEC_

Re: [PATCH] AArch64: Add support for __builtin_roundeven[f] [PR100966]

2021-06-18 Thread Richard Sandiford via Gcc-patches
Wilco Dijkstra writes: > Enable __builtin_roundeven[f] by adding roundeven as an alias to the > existing frintn support. Although frint_pattern maps UNSPEC_FRINTN to "frintn", "frintn" doesn't mean anything to the target-independent part of the compiler. I assume it was just picked as a placehold