[cfe-commits] r164177 - in /cfe/trunk: lib/Driver/ test/Driver/ test/Driver/Inputs/freescale_ppc_tree/ test/Driver/Inputs/freescale_ppc_tree/lib/ test/Driver/Inputs/freescale_ppc_tree/usr/ test/Driver/Inputs/freescale_ppc_tree/usr/lib/ test/Driver/Inputs/freescale_ppc_tree/usr/lib/powerpc-fsl-linux/ test/Driver/Inputs/freescale_ppc_tree/usr/lib/powerpc-fsl-linux/4.6.2/

Tobias von Koch via cfe-commits cfe-commits at lists.llvm.org
Thu Feb 9 14:46:21 PST 2017


On Wed, Feb 8, 2017 at 7:21 PM, Chandler Carruth <chandlerc at gmail.com>
wrote:

>
>> +    // The Freescale PPC SDK has the gcc libraries in
>> +    // <sysroot>/usr/lib/<triple>/x.y.z so have a look there as well.
>> +    "/" + CandidateTriple.str(),
>>
>
> So, this is really bad it turns out.
>
> We use this directory to walk every installed GCC version. But because
> this is just a normal lib directory on many systems (ever Debian and Ubuntu
> system for example) this goes very badly. It goes even more badly because
> of the (questionable) design of LLVM's directory iterator:
>

Wow, this is pretty bad, but it really sounds like the iterator should be
fixed rather than trying to hack around it. Doesn't this happen for the
other directories as well (which, admittedly, will have less entries)?

FWIW, I am not aware of anyone at NXP/Freescale or elsewhere actually using
LLVM on NXP Power platforms these days. The team I was a part of back then
doesn't exist anymore, so I'm not even sure who to ask. I may get more
insight into this once a certain proposed acquisition by my current
employer completes ;) but for now I wouldn't be opposed if you just want to
remove this.

Tobias
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20170209/cb00b747/attachment.html>


More information about the cfe-commits mailing list