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

Andrzej Warzynski via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 25 11:29:46 PST 2022


awarzynski added a comment.

@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.


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