[PATCH] D155546: [clang][Interp] Implement __builtin_fmin

[Aaron Ballman via Phabricator via cfe-commits]

aaron.ballman added subscribers: jcranmer-intel, hubert.reinterpretcast, rsmith.
aaron.ballman added a comment.

I'd like to see test coverage for treatment of NaNs. According to the C23 standard, a quiet NaN is treated as missing data for fmax and fmin; so if there's a quiet NaN and a numeric value, the numeric value it's what's returned.

It's not yet clear to me what happens when any of these functions encounter a signaling NaN at compile time. CC @hubert.reinterpretcast @jcranmer-intel @rsmith


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D155546/new/

https://reviews.llvm.org/D155546