[PATCH] D70804: [Frontend] Allow OpenMP offloading to aarch64
Jonas Hahnfeld via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Nov 29 00:10:48 PST 2019
Hahnfeld added a comment.
In D70804#1763443 <https://reviews.llvm.org/D70804#1763443>, @bryanpkc wrote:
> In D70804#1763289 <https://reviews.llvm.org/D70804#1763289>, @ABataev wrote:
>
> > Tests are required
>
>
> The tests for this were added in D30644 <https://reviews.llvm.org/D30644>, but they are currently failing on AArch64 due to D32035 <https://reviews.llvm.org/D32035>. This patch allows those tests to pass again. What further testing should I add?
There should be tests in Clang that the target is indeed supported. Tests in libomptarget are fine, but apparently they are not tested on a regular basis when failing for more than two years.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D70804/new/
https://reviews.llvm.org/D70804
More information about the cfe-commits
mailing list