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

Alexander Batashev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed May 29 11:19:01 PDT 2019


alexbatashev added a comment.

In D62580#1521234 <https://reviews.llvm.org/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 **O**penCL. I planned to use 'C' but it is already used.

I'll add docs. Thank you.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D62580/new/

https://reviews.llvm.org/D62580





More information about the cfe-commits mailing list