[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:11:25 PDT 2019


gtbercea added a comment.

In D60907#1479298 <https://reviews.llvm.org/D60907#1479298>, @gtbercea wrote:

> 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?
>
>
> This solution is following Alexey's suggestions. This solution allows the optimization of math calls if they apply (example: pow(x,2) => x*x ) which was one of the issues in the previous solution I implemented.





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