[PATCH] D62580: [OpenCL] Use long instead of long long in x86 builtins

2019-06-03 Thread Alexander Batashev via Phabricator via cfe-commits
alexbatashev marked 6 inline comments as done. alexbatashev added a comment. @Ka-Ka done CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62580/new/ https://reviews.llvm.org/D62580 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https:

[PATCH] D62580: [OpenCL] Use long instead of long long in x86 builtins

2019-05-31 Thread Alexander Batashev via Phabricator via cfe-commits
alexbatashev marked an inline comment as done. alexbatashev added a comment. @Ka-Ka good point. Thank you. @Anastasia Would such tests be ok with you? @erichkeane Thank you very much. I think I don't have permissions to commit changes and will need someone's help. Repository: rG LLVM Github M

[PATCH] D62580: [OpenCL] Use long instead of long long in x86 builtins

2019-05-29 Thread Alexander Batashev via Phabricator via cfe-commits
alexbatashev added a comment. In D62580#1521234 , @erichkeane wrote: > 'O' is an interesting choice. Any real justification for it, or just "what > was available"? It definitely needs to be documented in the top of > Builtins.def however. O is for **