ilya-biryukov added a comment.

Sorry about the confusion, I can now see why the original version of the patch 
was actually simpler.
I was put off by the fact that we override by adding command-line arguments 
instead of passing things around in the code, but it appears that's actually 
simpler than adding yet another mechanism to override target tripple in the 
driver.

The only suggestion from me would be maybe try moving this to a layer that 
would also make also benefit other compilation-db-based things like 
`clang-tidy`, etc?
Any ideas on how to do this? Would this potentially break anything?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D63194



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to