<p dir="ltr">On 12 Dec 2014 11:24, "Justin Bogner" <<a href="mailto:mail@justinbogner.com">mail@justinbogner.com</a>> wrote:<br>
><br>
> Reid Kleckner <<a href="mailto:reid@kleckner.net">reid@kleckner.net</a>> writes:<br>
> > Author: rnk<br>
> > Date: Fri Dec 12 13:13:04 2014<br>
> > New Revision: 224145<br>
> ><br>
> > URL: <a href="http://llvm.org/viewvc/llvm-project?rev=224145&view=rev">http://llvm.org/viewvc/llvm-project?rev=224145&view=rev</a><br>
> > Log:<br>
> > Allow module deps to be printed in an arbitrary order<br>
> ><br>
> > The order is different between Windows and Unix for reasons unknown, but<br>
> > the compiler output appears to still be determinstic.<br>
><br>
> I don't think this is right. It looks to me like r224055 changed<br>
> ModuleDependencyListener's semantics unintentionally. I'll look into it.</p>
<p dir="ltr">Thankyou!</p>
<p dir="ltr">I think the change you're referring to is that it removed the pruning of /x/../. That was not unintentional - there were no tests for that behaviour, and it's wrong if x is a symlink - but I had intended to investigate further prior to checkin and forgot.</p>
<p dir="ltr">You should be about to revert the changes to ModuleDependecyCollector.cpp to get back the old behaviour, but that behaviour seems to be wrong (and will probably assert in some cases).</p>
<p dir="ltr">> > Modified:<br>
> >     cfe/trunk/test/Modules/dependency-dump.m<br>
> ><br>
> > Modified: cfe/trunk/test/Modules/dependency-dump.m<br>
> > URL:<br>
> > <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Modules/dependency-dump.m?rev=224145&r1=224144&r2=224145&view=diff">http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Modules/dependency-dump.m?rev=224145&r1=224144&r2=224145&view=diff</a><br>
> > ==============================================================================<br>
> > --- cfe/trunk/test/Modules/dependency-dump.m (original)<br>
> > +++ cfe/trunk/test/Modules/dependency-dump.m Fri Dec 12 13:13:04 2014<br>
> > @@ -6,10 +6,10 @@<br>
> >  // expected-no-diagnostics<br>
> ><br>
> >  // RUN: FileCheck %s -check-prefix=VFS -input-file %t/vfs/vfs.yaml<br>
> > -// VFS: 'name': "SubFramework.h"<br>
> > -// VFS: 'name': "Treasure.h"<br>
> > -// VFS: 'name': "Module.h"<br>
> > -// VFS: 'name': "Sub.h"<br>
> > -// VFS: 'name': "Sub2.h"<br>
> > +// VFS-DAG: 'name': "SubFramework.h"<br>
> > +// VFS-DAG: 'name': "Treasure.h"<br>
> > +// VFS-DAG: 'name': "Module.h"<br>
> > +// VFS-DAG: 'name': "Sub.h"<br>
> > +// VFS-DAG: 'name': "Sub2.h"<br>
> ><br>
> >  @import Module;<br>
> ><br>
> ><br>
> > _______________________________________________<br>
> > cfe-commits mailing list<br>
> > <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
> > <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
> _______________________________________________<br>
> cfe-commits mailing list<br>
> <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
</p>