[PATCH] D118112: [flang][tco] Remove unneeded dependencies

Eric Schweitz via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 25 13:03:40 PST 2022


schweitz accepted this revision.
schweitz added a comment.

In D118112#3270237 <https://reviews.llvm.org/D118112#3270237>, @awarzynski wrote:

> @schweitz `${LLVM_TARGETS_TO_BUILD}` and `InitializeAllTargets();` are for initializing LLVM backends (i.e. backends that generate machine code). IIUC, this is irrelevant in `tco` (it's not meant to generate machine code). Also, I've tested `tco` before and after this change and I see no difference in behavior.

Thanks for checking, Andrzej.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D118112



More information about the llvm-commits mailing list