[PATCH] D63009: [OpenMP] Add target task alloc function with device ID

Alexey Bataev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Jun 15 13:58:48 PDT 2019


ABataev added a comment.

In D63009#1544900 <https://reviews.llvm.org/D63009#1544900>, @gtbercea wrote:

> In D63009#1544758 <https://reviews.llvm.org/D63009#1544758>, @Hahnfeld wrote:
>
> > Am I correct that the second to last revision ("- Fix tests.") removed all checks for the actual `device_id` argument from the tests? From my point of view that's not fixing but weakening the tests! Can you explain why they needed "fixing"?
>
>
> When I was just passing the default value the LLVM-IR was: i64 -1 i.e. constant, easy to check.
>
> With the latest change the emitted code is: i64 %123 i.e. where %123 is a local derived from the expression of the device ID.


If the value is constant, check for the constant. And at least several tests with the expressions should check for the correct value of the expression.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D63009





More information about the cfe-commits mailing list