[PATCH] D73194: [FPEnv][ARM] Add lowering of STRICT_FSETCC and STRICT_FSETCCS
Dave Green via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sun Feb 2 07:59:16 PST 2020
dmgreen accepted this revision.
dmgreen added a comment.
This revision is now accepted and ready to land.
OK. It looks like that is at least not going to be simple to sort out immediately.
In the meantime, not crashing is certainly better than crashing, and I think the rest of this code looks good. It's just the strict ordering that we are missing. Please see if you can follow up with either adding a chain or not using glue.
LGTM otherwise.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D73194/new/
https://reviews.llvm.org/D73194
More information about the llvm-commits
mailing list