[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