[PATCH] D147324: [OpenMP][MLIR][Flang][bbc][Driver] Add OpenMP RTL Flags to Flang and generate omp.FlagsAttr from them

Wed Apr 5 10:16:34 PDT 2023

agozillon added a comment.

In D147324#4245932 <https://reviews.llvm.org/D147324#4245932>, @kiranchandramohan wrote:

> LG. This portion of the patch was accepted in https://reviews.llvm.org/D145264.
>
> See inline comment about location of LLVM Dialect tests. You may either remove the LLVM dialect tests from this patch and land it. Or you can submit this and move the tests in a separate NFC patch and submit without review.

Hi Kiran,

Thanks for the quick reply. I'll do the former and remove the tests and land the patch today (back tracking on the previous statement of leaving landing it till later), I don't believe they add anything to the test suite as there is no modifications between FIR -> LLVM Dialect as far as this attribute and the ModuleOp go.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D147324