<div dir="rtl"><div dir="ltr">Hi Rafael,</div><div dir="ltr"><br></div><div dir="ltr">Do you mean Generic_GCC::GCCInstallationDetector ?</div><div dir="ltr"><br></div><div dir="ltr">Yaron</div><div dir="ltr"><br></div></div>
<div class="gmail_extra"><br><br><div class="gmail_quote"><div dir="ltr">2014-03-23 14:10 GMT+02:00 Rafael Avila de Espindola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span>:</div>
<blockquote class="gmail_quote" style="margin:0 .8ex;border-left:1px #ccc solid;border-right:1px #ccc solid;padding-left:1ex;padding-right:1ex"><div dir="auto"><div><br><br>Sent from my iPhone</div><div class=""><div><br>
On Mar 23, 2014, at 6:54, Reid Kleckner <<a href="mailto:rnk@google.com" target="_blank">rnk@google.com</a>> wrote:<br><br></div><blockquote type="cite"><div><p dir="ltr">That makes total sense, given that (I think) we do roughly the same on Linux.</p>
</div></blockquote><div><br></div></div><div>We do a search in the driver and pass the result down. Doing the same for mingw would be nice.</div><div><div class="h5"><div><br></div><br><blockquote type="cite"><div><p dir="ltr">
Sent from phone</p>
<div class="gmail_quote">On Mar 23, 2014 5:40 AM, "Yaron Keren" <<a href="mailto:yaron.keren@gmail.com" target="_blank">yaron.keren@gmail.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="rtl"><div dir="ltr">Hi,</div><div dir="ltr"><br></div><div dir="ltr">The current setup of MinGW include directories in InitHeaderSearch.cpp has several problems:</div><div dir="ltr"><br></div><div dir="ltr">* New versions are added manually to the code.</div>
<div dir="ltr">* All existing versions of MinGW directories are added regardless of what is actually installed (and pruned later), wasted work.</div><div dir="ltr">* Two <a href="http://MinGW.org" target="_blank">MinGW.org</a> C includes are missing.</div>
<div dir="ltr">
* If multiple MinGW versions are installed clang will add directories for all of them leading to unpredictable results.</div><div dir="ltr"><br></div><div dir="ltr">I'd like to suggest that clang will add include directories only for the gcc.exe which is on the system PATH based on the list below for both distributions. </div>
<div dir="ltr">This way:</div><div dir="ltr">* clang will not add include diectories for every possible version, only for this one.</div><div dir="ltr">* clang will correctly hanlde multiple based on the dveloper choice which gcc.exe is on the path</div>
<div dir="ltr">* We won't have to manually update new versions in InitHeaderSearch.cpp.</div><div dir="ltr"><br></div><div dir="ltr">What do you think?<br></div><div dir="ltr"><br></div><div dir="ltr">Yaron</div><div dir="ltr">
<br></div><div dir="ltr"><div dir="ltr"><a href="http://mingw.org" target="_blank">mingw.org</a></div><div dir="ltr">---------</div><div dir="ltr">gcc location</div><div dir="ltr">BASEPATH\bin\gcc.exe</div><div dir="ltr">
<br></div><div dir="ltr">
C32 includes</div><div dir="ltr">BASEPATH\lib\gcc\mingw32\X.Y.Z\include</div><div dir="ltr">BASEPATH\lib\gcc\mingw32\X.Y.Z\include-fixed</div><div dir="ltr">BASEPATH\include</div><div dir="ltr"> </div><div dir="ltr">C++32 includes</div>
<div dir="ltr">BASEPATH\lib\gcc\mingw32\X.Y.Z\include\c++</div><div dir="ltr">BASEPATH\lib\gcc\mingw32\X.Y.Z\include\c++\mingw32</div><div dir="ltr">BASEPATH\lib\gcc\mingw32\X.Y.Z\include\c++\backward</div><div dir="ltr">
<br></div><div dir="ltr">mingw-w64</div><div dir="ltr">---------</div><div dir="ltr">gcc includes</div><div dir="ltr">BASEPATH\bin\gcc.exe</div><div dir="ltr"><br></div><div dir="ltr">C32 includes</div><div dir="ltr">BASEPATH\lib\gcc\i686-w64-mingw32\X.Y.Z\include</div>
<div dir="ltr">BASEPATH\lib\gcc\i686-w64-mingw32\X.Y.Z\include-fixed</div><div dir="ltr">BASEPATH\i686-w64-mingw32\include</div><div dir="ltr"><br></div><div dir="ltr">C64 includes</div><div dir="ltr">BASEPATH\lib\gcc\x86_64-w64-mingw32\X.Y.Z\include</div>
<div dir="ltr">BASEPATH\lib\gcc\x86_64-w64-mingw32\X.Y.Z\include-fixed</div><div dir="ltr">BASEPATH\x86_64-w64-mingw32\include</div><div dir="ltr"><br></div><div dir="ltr">C++32 includes</div><div dir="ltr">BASEPATH\i686-w64-mingw32\include\c++</div>
<div dir="ltr"><br></div><div dir="ltr">C++64 includes</div><div dir="ltr">BASEPATH\x86_64-w64-mingw32\include\c++</div><div><br></div></div><div dir="ltr"><br></div><div dir="ltr"><br></div></div>
<br>_______________________________________________<br>
cfe-dev mailing list<br>
<a href="mailto:cfe-dev@cs.uiuc.edu" target="_blank">cfe-dev@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev</a><br>
<br></blockquote></div>
</div></blockquote><blockquote type="cite"><div><span>_______________________________________________</span><br><span>cfe-dev mailing list</span><br><span><a href="mailto:cfe-dev@cs.uiuc.edu" target="_blank">cfe-dev@cs.uiuc.edu</a></span><br>
<span><a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev</a></span><br></div></blockquote></div></div></div></blockquote></div><br></div>