[PATCH] D138449: support: fix getProcessTriple in universal builds
Keith Smiley via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Nov 28 17:12:04 PST 2022
keith added a comment.
I'm not super familiar with the history behind `LLVM_HOST_TRIPLE`, but one thing that definitely worries me about this specific change is there are other consumers of that variable, but only this 1 gets this fixup. I feel like the core issue is the mismatch between having a single `LLVM_HOST_TRIPLE`, but having cmake build a fat binary.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D138449/new/
https://reviews.llvm.org/D138449
More information about the llvm-commits
mailing list