[PATCH] D83397: [flang] Replace uses of _Complex with std::complex

Wed Jul 8 08:07:04 PDT 2020

schweitz added a comment.

In D83397#2139131 <https://reviews.llvm.org/D83397#2139131>, @echristo wrote:

> In D83397#2139111 <https://reviews.llvm.org/D83397#2139111>, @schweitz wrote:
>
> > Hi Eric,
> >
> > There is an active development branch for the flang middle end. https://github.com/flang-compiler/f18-llvm-project/tree/fir-dev
>
>
> That's not part of the llvm project.
>
> > That code base is being upstreamed piecemeal. Not all of the code is upstreamed at this point. It is simply a false impression that code in the middle of being upstreamed is "unused" or "unnecessary". Since it not all of it is upstreamed, changing interfaces and support code in llvm-project directly is going to cause problems that can become hard to track and resolve while the upstreaming is ongoing.
>
> It very much is unused and unnecessary as there are no pieces of that code in the repository.

We are stuck between a rock and a hard place. Flang needs a middle end, as it currently can't generate code. Work on that middle end is being done. We are following the rules and trying to upstream that code in small "reviewable" chunks as quickly as possible. It's just simply going to be the case because of the interdependencies involved and the small chunks process that some code will merely appear to be temporally unused.

If you have a better solution to how to change the upstream process, then our group would be happy to hear it.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83397