[PATCH] D75209: [OPENMP][NVPTX]Fix PR45003: add support for complex functions.

Johannes Doerfert via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Mar 7 09:08:26 PST 2020


jdoerfert added a comment.

In D75209#1910047 <https://reviews.llvm.org/D75209#1910047>, @ABataev wrote:

> In D75209#1909976 <https://reviews.llvm.org/D75209#1909976>, @jdoerfert wrote:
>
> > I am unsure we need this with the proper math support. Sema patch for that is going for review today. I'll try this out soon.
>
>
> It has nothing to do with the math functions support. These functions are required for the definition of __muldc3/__divdc3/__mulsc2/__divsc3 functions emitted for the complex types. CUDA does absolutely the same.


The fact that CUDA does the same is the important part here. Why should we copy it then? See https://reviews.llvm.org/D75788#change-0Ax9LSfw7OAk for working support of complex math functions in target regions without copying what cuda does but by just using it.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75209





More information about the cfe-commits mailing list