[PATCH] D60907: [OpenMP] Add math functions support in OpenMP offloading

Gheorghe-Teodor Bercea via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Apr 25 13:48:20 PDT 2019


gtbercea added a comment.

In D60907#1479142 <https://reviews.llvm.org/D60907#1479142>, @hfinkel wrote:

> In D60907#1479118 <https://reviews.llvm.org/D60907#1479118>, @gtbercea wrote:
>
> > Ping @hfinkel @tra
>
>
> The last two comments in D47849 <https://reviews.llvm.org/D47849> indicated exploration of a different approach, and one which still seems superior to this one. Can you please comment on why you're now pursuing this approach instead?


Hal, as far as I can tell, this solution is similar to yours but with a slightly different implementation. If there are particular aspects about this patch you would like to discuss/give feedback on please let me know.


Repository:
  rC Clang

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

https://reviews.llvm.org/D60907





More information about the cfe-commits mailing list