thopre added a comment. In D96568#2569296 <https://reviews.llvm.org/D96568#2569296>, @jonpa wrote:
>> Sounds good to me. Hopefully I'll get round to __builtin_isinf soon and a >> single hook will make the patch slightly smaller. > > Patch updated to call the new hook testFPKind() and make it take a BuiltinID > as argument (that seems to work at least for the moment - maybe an enum type > will become necessary at some point per your suggestion..?) > > I am not sure if this is "only" or "typically" used in constrained FP mode, > or if the mode should be independent of calling this hook. The patch as it is > asserts that it is called for an FP type but leaves it to the target to > decide based on the FP mode, where SystemZ opts out unless it is constrained > (which I think is what is wanted...). I forgot to ask at the time, why do you restrict this code to the constrained case? Presumably it's a lot faster than fabs+cmp and should be faster in all cases? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D96568/new/ https://reviews.llvm.org/D96568 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits