[PATCH] Canonicalise Windows target triple spellings
abdulras at fb.com
Tue Mar 4 15:33:42 PST 2014
On Mar 4, 2014, at 1:59 PM, Chandler Carruth <chandlerc at gmail.com> wrote:
> On Tue, Mar 4, 2014 at 11:53 AM, Saleem Abdulrasool <abdulras at fb.com> wrote:
> i686-pc-win32 => i686-pc-windows-msvc
> i686-pc-mingw32 => i686-pc-windows-gnu
> i686-pc-cygwin => i686-pc-windows-cygnus
> I don't understand the use of 'pc' as vendor -- is this just that vendor is preserved? I would hope that the default is "unknown" much the way it is on Linux.
Yes, that is supposed to indicate that the vendor is preserved. If you do not specify one, one is not inserted, and I have tests to ensure that. I would love to see unknown become the default here as it is on Linux.
abdulras (at) fb (dot) com
More information about the llvm-commits