diff options
author | Jonas Devlieghere <jonas@devlieghere.com> | 2018-11-11 00:56:15 +0000 |
---|---|---|
committer | Jonas Devlieghere <jonas@devlieghere.com> | 2018-11-11 00:56:15 +0000 |
commit | 64a26308256adcf9212fd346733a60735f7acc01 (patch) | |
tree | 043986e4820875be83de38922ba301eb68d3a569 /lldb/packages/Python/lldbsuite/test/functionalities/stats_api/TestStatisticsAPI.py | |
parent | d23cdbbeb2d5f9cf23baf62597c8c2a24ab2b8ef (diff) | |
download | bcm5719-llvm-64a26308256adcf9212fd346733a60735f7acc01.tar.gz bcm5719-llvm-64a26308256adcf9212fd346733a60735f7acc01.zip |
Pass the function type instead of the return type to FunctionDecl::Create
Fix places where the return type of a FunctionDecl was being used in
place of the function type
FunctionDecl::Create() takes as its T parameter the type of function
that should be created, not the return type. Passing in the return type
looks to have been copypasta'd around a bit, but the number of correct
usages outweighs the incorrect ones so I've opted for keeping what T is
the same and fixing up the call sites instead.
This fixes a crash in Clang when attempting to compile the following
snippet of code with -fblocks -fsanitize=function -x objective-c++ (my
original repro case):
void g(void(^)());
void f()
{
__block int a = 0;
g(^(){ a++; });
}
as well as the following which only requires -fsanitize=function -x c++:
void f(char * buf)
{
__builtin_os_log_format(buf, "");
}
Patch by: Ben (bobsayshilol)
Differential revision: https://reviews.llvm.org/D53263
llvm-svn: 346601
Diffstat (limited to 'lldb/packages/Python/lldbsuite/test/functionalities/stats_api/TestStatisticsAPI.py')
0 files changed, 0 insertions, 0 deletions