r321855 - Reapply r321781: [Modules] Allow modules specified by -fmodule-map-file to shadow implicitly found ones

[Eric Fiselier via cfe-commits]

Thanks for the correction Richard.

/Eric

On Tue, Jan 9, 2018 at 8:03 PM, Richard Smith <richard at metafoo.co.uk> wrote:

> On 9 January 2018 at 18:46, Bruno Cardoso Lopes via cfe-commits <
> cfe-commits at lists.llvm.org> wrote:
>
>> Hi Eric,
>>
>> On Mon, Jan 8, 2018 at 9:31 PM, Eric Fiselier <eric at efcs.ca> wrote:
>> > I haven't done a ton of investigation yet, but I suspect this commit is
>> > responsible for breaking my LLVM module selfhost build, which uses
>> libc++ as
>> > the STL, and builds LLD (No bots appear to test this configuration).
>> >
>> > The error message can be found here:
>> > https://gist.github.com/EricWF/fb5e1d18eb1069d23788469b5fb15442
>> >
>> > I'll investigate further tomorrow, but I wanted to send this your way in
>> > case it looks familiar.
>>
>> Haven't seen this one before, maybe because of this particular
>> configuration. I'm happy to help out fixing it, I can try a similar
>> setup and see what I get. It would be nice if you can confirm this is
>> the culprit commit though.
>>
>
> See r321395, which recently added this warning, and is a much more likely
> culprit than this change.
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20180111/c12a4df6/attachment.html>