<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Mon, May 12, 2014 at 5:45 PM, Jon Roelofs <span dir="ltr"><<a href="mailto:jonathan@codesourcery.com" target="_blank">jonathan@codesourcery.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">================<br>
Comment at: lib/builtins/arm/vfp/adddf3vfp.S:10<br>
@@ -9,3 +9,3 @@<br>
<br>
-#include "../assembly.h"<br>
+#include "../../assembly.h"<br>
<br>
----------------<br>
</div><div class="">Saleem Abdulrasool wrote:<br>
> Im tempted to say that we should adjust the CPPFLAGS to include $(top_srcdir)/lib/builtins in the include search path and make this into:<br>
><br>
>     #include "assembly.h"<br>
><br>
> We certainly do similar things on the LLVM side where we assume that $(top_srcdir)/include is in the search path for headers.  I don't think that the minor discomfort of gf no longer working in vim is a huge loss.<br>

</div>Yeah, I'm not a huge fan of '../' in include paths either, but I don't feel comfortable enough with compiler-rt's makefiles to make that change.... they're quite complicated :)<br></blockquote>
<div><br></div><div>You'll also have to change CMake build system :)</div><div>Currently Makefile build system has "/compiler-rt/lib" in its include paths, so you can probably include the headers as "builtins/assembly.h".</div>
<div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<a href="http://reviews.llvm.org/D3731" target="_blank">http://reviews.llvm.org/D3731</a><br>
<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" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div dir="ltr"><div>Alexey Samsonov, Mountain View, CA</div></div>
</div></div>