[PATCH] D109890: Flang OpenMP Report Plugin

[Andrzej Warzynski via Phabricator via llvm-commits]

awarzynski added a comment.

Apologies, I forgot to mention this one before.

In `FlangOmpReport::ExecuteAction`, you do some filename manipulation in order to prevent the plugin from generating output files next to the input. You can avoid that and leverage the "existing" mechanism, e.g.:

- `flang-new -o <output-filename> <input-filename>` will use `<output-filename>` as the location and name of the output file

In particular,`flang-new -o - <intput-filename>` will print to stdout. If you use this, then all you have to do is write your `RUN` lines in tests like this:

  ! RUNL %flang_fc1 -load %llvmshlibdir/flangOmpReport.so -plugin flang-omp-report -fopenmp %S/Inputs/omp-declarative-directive.f90 -o - | FileCheck %s

So, less code in tests and in `ExecuteAction` :)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109890