aaron.ballman added a comment.

Thank you for this! The revert LGTM, but please wait for @mstorsjo to confirm 
this won't cause problems for MinGW folks.

In terms of solving the underlying issue with aux triple so that we can re-land 
this, I went digging to see if there's an existing issue tracking this and I 
could not find one. However, related to the functionality in this patch, I did 
see: https://github.com/llvm/llvm-project/issues/60383. I filed 
https://github.com/llvm/llvm-project/issues/64436 to track the aux-triple 
issue; hopefully we can find someone with the bandwidth to look into that issue 
(I'll see if I can get someone at Intel to look into it, but if someone else 
wanted to tackle it first, that would be great).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D157115

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

Reply via email to