[PATCH] D118618: [flang] Fix invalid diagnostics in wrapper and reduce code dup.

[Yury Gribov via Phabricator via llvm-commits]

ygribov added a comment.

In D118618#3289849 <https://reviews.llvm.org/D118618#3289849>, @kiranchandramohan wrote:

> Thanks @ygribov for this patch and the other contributions made.
> We will take this patch. I just wanted to highlight that the `flang` script is temporary and will be replaced by the `flang-new` driver when codegen is upstreamed. I guess the longer-standing contributions will be in `flang-new`. We are also trying to minimize changes to the `flang` script to keep it stable. Sometimes we have seen minor changes running into issues in a different bash version. Hope that is OK with you.

Hi Kiran,

Thanks for the update. We are just trying to figure out the status of flang's frontend at the moment and fix issues in flang script which we run into. I do realize that this work will be thrown away in few months. If you believe that this is not really useful, I'll keep flang fixes in our local branch.

As a side note, what is the plan about merging codegen into main?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D118618