[PATCH] Sema: Accept pointers to any address space for builtin functions

Tom Stellard thomas.stellard at amd.com
Thu Mar 19 08:06:27 PDT 2015


Hi, here is an updated patch.  I've addressed all comments except I did not add a DensMap to rewriteBuiltinFunctionDecl(), I wasn't sure exactly how to use this to speed up the function.  I can still add this if people think it will help.  I did some profiling of a function with 100,000 __builtin_memcpy() calls and this patch doesn't seem to increase compile time.

I also did not add handling for return types.  I wasn't able to figure out how to get the actual call expression which had the return type used in the code.  I added a TODO for this.


REPOSITORY
  rL LLVM

http://reviews.llvm.org/D8082

Files:
  include/clang/Basic/Builtins.def
  include/clang/Basic/Builtins.h
  lib/Sema/SemaExpr.cpp
  test/CodeGenOpenCL/memcpy.cl
  test/Sema/builtins.cl

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D8082.22255.patch
Type: text/x-patch
Size: 7235 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20150319/7bec21cc/attachment.bin>


More information about the cfe-commits mailing list