For #2 you can use llvm::GetProcessTriple(), i will think about #1 when I have a chance to look at the code<br><div class="gmail_quote"><div dir="ltr">On Sat, Sep 3, 2016 at 2:38 PM Vyacheslav Karpukhin <<a href="mailto:vyacheslav@karpukhin.com">vyacheslav@karpukhin.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">stigger added a comment.<br>
<br>
In <a href="https://reviews.llvm.org/D20041#533725" rel="noreferrer" target="_blank">https://reviews.llvm.org/D20041#533725</a>, @zturner wrote:<br>
<br>
> How difficult would be it be to eliminate the `PathSyntax` enumeration entirely and update every user of `PathSyntax` to use a triple instead?<br>
<br>
<br>
I've attempted to do it and hit two issues which I'm not sure how to resolve:<br>
<br>
1. SymbolFilePDB.cpp:358 uses FileSpec with explicit `ePathSyntaxWindows`. It's possible to do something like `llvm::Triple("unknown", "unknown", "windows")`, but that kind of smells.<br>
2. HostInfoBase.cpp:63 has a bunch of FileSpecs with `ePathSyntaxHostNative`, but at that moment it is not yet possible to use `HostInfo::GetArchitecture()`, because it's being constructed.<br>
<br>
<br>
<a href="https://reviews.llvm.org/D20041" rel="noreferrer" target="_blank">https://reviews.llvm.org/D20041</a><br>
<br>
<br>
<br>
</blockquote></div>