On Tue, Aug 6, 2013 at 2:37 PM, Jean-Daniel Dupas <span dir="ltr"><<a href="mailto:devlists@shadowlab.org" target="_blank">devlists@shadowlab.org</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div style="word-wrap:break-word"><div>Hello,</div><div><br></div><div>While I was playing with the new clang module functionality, I encountered an issue with the 'convert #include in import' feature.</div><div><br>
</div><div>Apparently, when including a file using an absolute path, clang does not try at all to map it to possibly existing included module. It results in some inconsistencies when compiling with and without module enabled.</div>
</div></blockquote><div><br></div><div>This is sort of expected. We look for the module.map file in the base of the include path where the header was found (the path given to -I or -isystem). If you give an absolute path, then there is no such path.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><div>Take for instance this source file:</div><div><br></div><div>------------ include.c</div>
<div><br></div><div><div style="margin:0px;font-size:11px;font-family:Menlo;color:rgb(209,47,27)"><span style="color:#78492a">#include </span>"/usr/include/stdio.h"</div><div style="margin:0px;font-size:11px;font-family:Menlo;min-height:13px">
<br></div><div style="margin:0px;font-size:11px;font-family:Menlo;color:rgb(120,73,42)">#include <span style="color:#d12f1b"><stdio.h></span></div></div><div><br></div><div>-------------------------</div><div><br></div>
<div>Obviously, as the stdio.h file contains include guards, this file compile fine without modules.</div><div>But if I enable modules, clang does not figures both includes points to the same files, and so the same module, and it try to import the module, while the stdio.h was already included.</div>
<div>And it result in a rather ugly mess:</div><div><br></div><div>$ clang -fsyntax-only -fmodules include.c</div><div><br></div><div>usr/include/stdio.h:96:8: error: redefinition of '__sbuf'</div><div>struct __sbuf {</div>
<div>       ^</div><div>/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.9.sdk/usr/include/stdio.h:86:8: note: previous definition is here</div><div>struct __sbuf {</div><div>       ^</div>
<div>In file included from include.c:2:</div><div>/usr/include/stdio.h:130:16: error: redefinition of '__sFILE'</div><div>typedef struct __sFILE {</div><div>               ^</div><div>/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.9.sdk/usr/include/stdio.h:120:16: note: previous definition is here</div>
<div>typedef struct __sFILE {</div><div>               ^</div><div><br></div><div>… and so on, on many lines.</div><div><br></div><div>Of course, I don't expect someone write this kind of include, but this is what Xcode and clang synthesize when using prefix headers.</div>
<div>Xcode convert the header path to an absolute path, and pass it to clang using the "-include" parameter.</div><div>Then clang interpret the '-include' argument as an #include directive, and try to import the file using the absolute path.</div>
<div><br></div><div>Is this a clang bug, or is it an expected behaviors for the modules ? If this is a bug, I'd be glad to have a look at what is wrong, and propose patches.</div><span class="HOEnZb"><font color="#888888"><div>
<br></div><div>
<span style="border-collapse:separate;border-spacing:0px"><div>-- Jean-Daniel</div><div><br></div><div><br></div></span><br>

</div>
<br></font></span></div><br>_______________________________________________<br>
cfe-dev mailing list<br>
<a href="mailto:cfe-dev@cs.uiuc.edu">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><br>