aaron.ballman added a comment.

In D68377#1698399 <https://reviews.llvm.org/D68377#1698399>, @xbolva00 wrote:

> Current solution does not work
>  /home/xbolva00/LLVM/llvm/tools/clang/include/clang/Basic/Builtins.h:50:34: 
> error: redefinition of ‘BImemccpy’
>
>   #define BUILTIN(ID, TYPE, ATTRS) BI##ID,
>                                    ^
>
> /home/xbolva00/LLVM/llvm/tools/clang/include/clang/Basic/Builtins.h:50:34: 
> note: in definition of macro ‘BUILTIN’
>
>   #define BUILTIN(ID, TYPE, ATTRS) BI##ID,
>   
>
> @aaron.ballman, maybe you can help me how to represent that memccpy exists 
> for GNU and MS?


We may not have the machinery needed for doing this yet, in which case, I think 
it's fine to ignore the MS builtin.


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

https://reviews.llvm.org/D68377



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to