AaronBallman wrote: > > Marking as requested changes because it's not clear why these changes are > > being proposed. > > This topic was talked in: #113133: Should we add `nsz` top > `__builtin_elementwise_min`. > > @nikic suggested that we should remove float support from > `__builtin_elementwise_min`. > > My suggestion is that we can set `__builtin_elementwise_min` as the the most > relaxed flavor: backend can do anything that they want.
This builtin was released over five years ago in Clang, so changing the semantics with no notice is pretty user hostile. It doesn't sound like the float support we have is wrong, just that someone wanted to change the semantics around negative zero handling for better optimization opportunities. Is that correct? https://github.com/llvm/llvm-project/pull/180885 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
