[PATCH] D54121: [FPEnv] Add constrained FCMP intrinsic

Cameron McInally via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 9 10:06:46 PST 2018


cameron.mcinally added a comment.

Thinking aloud...

I'm playing with the idea of being able to implement the signaling variants of each operation through the quiet variants. I mean, a signaling compare is really just:

  bool compareSignalingEqual(a, b) {
    return is_ordered(a,b) ? compareQuietEqual(a, b) : signal Invalid;
  }

But it may be difficult to get the flags right. I haven't thought that all the way through yet.

This is probably not worth serious consideration at this point. Just wanted to throw it out there.


Repository:
  rL LLVM

https://reviews.llvm.org/D54121





More information about the llvm-commits mailing list