[Lldb-commits] [lldb-commits] [patch] [review request] Generalize host vendor and os reporting

Kaylor, Andrew andrew.kaylor at intel.com
Tue Oct 9 16:30:00 PDT 2012

We came across a problem on Linux where target matching wasn't working because the vendor was being reported as 'gnu' instead of 'pc'.  Looking at the code, it seems that the vendor being reported for both Linux and FreeBSD was invalid.  The OS strings, while OK, were also hard-coded where they didn't need to be.  The attached patch removes ifdef's so that all cases use the host triple.

This problem is orthogonal to the patch I submitted last week to work around an inappropriate default triple on Linux.  Both are needed.

Thanks in advance for any review feedback.

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20121009/ee498174/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: host-vendor.patch
Type: application/octet-stream
Size: 1256 bytes
Desc: host-vendor.patch
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20121009/ee498174/attachment.obj>

More information about the lldb-commits mailing list