[PATCH] D25480: __builtin_fpclassify missing one int parameter

2016-10-14 Thread David Sheinkman via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL284277: __builtin_fpclassify missing one int parameter (authored by davidsh). Changed prior to commit: https://reviews.llvm.org/D25480?vs=74377=74740#toc Repository: rL LLVM

[PATCH] D25480: __builtin_fpclassify missing one int parameter

2016-10-11 Thread Hubert Tong via cfe-commits
hubert.reinterpretcast added a comment. In https://reviews.llvm.org/D25480#567552, @ahatanak wrote: > __builtin_fpclassify takes five int arguments followed by one last argument > that is of floating point type. Do you know if there is a way to specify the > last one argument is a floating

[PATCH] D25480: __builtin_fpclassify missing one int parameter

2016-10-11 Thread Akira Hatanaka via cfe-commits
ahatanak added a comment. __builtin_fpclassify takes five int arguments followed by one last argument that is of floating point type. Do you know if there is a way to specify the last one argument is a floating point rather than using '.'? https://reviews.llvm.org/D25480

[PATCH] D25480: __builtin_fpclassify missing one int parameter

2016-10-11 Thread Hubert Tong via cfe-commits
hubert.reinterpretcast added a comment. A test should probably be added as well. https://reviews.llvm.org/D25480 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D25480: __builtin_fpclassify missing one int parameter

2016-10-11 Thread Tania Albarghouthi via cfe-commits
taniabarg created this revision. taniabarg added reviewers: davidsh, hubert.reinterpretcast. taniabarg added a subscriber: cfe-commits. BIF fpclassify has the wrong number of integer parameters specified in Builtins.def. There should be 5 int parameters (each representing the values FP_NAN,