[Openmp-commits] [PATCH] D63009: [OpenMP] Add target task alloc function with device ID
Jonas Hahnfeld via Phabricator via Openmp-commits
openmp-commits at lists.llvm.org
Sat Jun 15 00:33:09 PDT 2019
Hahnfeld added a comment.
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"?
If I had to guess this is because some directives have the `device` clause, so the tests should check that the generated code passes the correct argument in that case.
CHANGES SINCE LAST ACTION
More information about the Openmp-commits