[PATCH] D117781: [flang] Update tco tool pipline and add translation to LLVM IR

[Andrzej Warzynski via Phabricator via llvm-commits]

awarzynski added a comment.

In D117781#3267662 <https://reviews.llvm.org/D117781#3267662>, @mehdi_amini wrote:

> It is a bit surprising that this linking dependency is declared on the tool instead of on the actual library that needs it.

IIUC, these dependencies are not needed at all. `tco` is not meant for interfacing with the backend (i.e. machine code generation). I've submitted an update: https://reviews.llvm.org/D118112

Apologies for not pointing this out earlier. I was away for a bit and when I looked at this yesterday I focused on fixing the build failure.

Btw, I agree that merging `fir-opt` and `tco` into one tool would be a sensible direction.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D117781