[PATCH] D32445: [LV] Handle external uses of floating-point induction variables

Michael Kuperstein via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 24 13:39:21 PDT 2017


mkuper added a comment.

I'm not sure. Do we currently (I mean, without this patch) do anything with FP IVs that violates spec?

If we do, then, yes, we not be recognizing them - and this transformation is also safe.
If we don't, then it would be a good idea to ask somebody who understands FP better whether this makes sense or not.


https://reviews.llvm.org/D32445





More information about the llvm-commits mailing list