[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