[PATCH] D90103: Add OpenMP for optimization

Abid via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 25 05:07:13 PST 2021


abidmalikwaterloo added a comment.

In D90103#2549979 <https://reviews.llvm.org/D90103#2549979>, @jdoerfert wrote:

> Let's focus on the tests without the optimization code.
>
> On that front:
>
> 1. Avoid unnecessary complexity, i.a., print, and go for simple identifiable calls, like `body1()`
> 2. Remove everything from the test that is not needed, e.g. unrelated functions, attributes, etc.
> 3. Preprocess the tests with common passes such as mem2reg, instcombine, and simplifycfg. We only want to run openmpopt on them later, never O3 <https://reviews.llvm.org/owners/package/3/>.

Is there a set of other common passes that are required for this level of testing? Could not figure out from the other OpenMP IR test cases.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D90103



More information about the llvm-commits mailing list