[Openmp-commits] [PATCH] D102107: [OpenMP] Codegen aggregate for outlined function captures

Fri Sep 23 13:51:17 PDT 2022

dhruvachak added a comment.

In D102107#3812554 <https://reviews.llvm.org/D102107#3812554>, @dhruvachak wrote:

> Updated llvm tests. The following 3 tests still fail:
>
>   LLVM :: Transforms/OpenMP/spmdization_constant_prop.ll
>   LLVM :: Transforms/OpenMP/spmdization_guarding_two_reaching_kernels.ll
>   LLVM :: Transforms/OpenMP/spmdization_remarks.ll

@jdoerfert @jhuber6 
I updated the LLVM tests except one, Transforms/OpenMP/spmdization_constant_prop.ll. There is no C source snippet in there. Can you help as to how to update it? Please review the diffs for all the updated LLVM tests as well.

In addition, the other 2 tests above fail even after updating. Looks like something is wrong. Can you help as to how to fix them?

Similar to clang tests, we are seeing remarks differences. We already decided to file an issue (after this patch lands) and look at them after-the-fact.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D102107