diff options
author | Marco Antognini <marco.antognini@arm.com> | 2018-11-27 14:54:58 +0000 |
---|---|---|
committer | Marco Antognini <marco.antognini@arm.com> | 2018-11-27 14:54:58 +0000 |
commit | 06d9d070c7ee87ff5165242ebc9fe15a5701ba2d (patch) | |
tree | 9990f6116a33e27b7454d96610990a6db902250b /lldb/packages/Python/lldbsuite/test/python_api/target | |
parent | 02f5511ff422b4ba6fcb75df320774132f23db56 (diff) | |
download | bcm5719-llvm-06d9d070c7ee87ff5165242ebc9fe15a5701ba2d.tar.gz bcm5719-llvm-06d9d070c7ee87ff5165242ebc9fe15a5701ba2d.zip |
Derive builtin return type from its definition
Summary:
Prior to this patch, OpenCL code such as the following would attempt to create
a BranchInst with a non-bool argument:
if (enqueue_kernel(get_default_queue(), 0, nd, ^(void){})) /* ... */
This patch is a follow up on a similar issue with pipe builtin
operations. See commit r280800 and https://bugs.llvm.org/show_bug.cgi?id=30219.
This change, while being conservative on non-builtin functions,
should set the type of expressions invoking builtins to the
proper type, instead of defaulting to `bool` and requiring
manual overrides in Sema::CheckBuiltinFunctionCall.
In addition to tests for enqueue_kernel, the tests are extended to
check other OpenCL builtins.
Reviewers: Anastasia, spatel, rsmith
Reviewed By: Anastasia
Subscribers: kristina, cfe-commits, svenvh
Differential Revision: https://reviews.llvm.org/D52879
llvm-svn: 347658
Diffstat (limited to 'lldb/packages/Python/lldbsuite/test/python_api/target')
0 files changed, 0 insertions, 0 deletions