teemperor added a comment.

Woops, my bad, seems like I tested just before you fixed it.

Because I read in the documentation that we could also pass include paths here, 
we could also try something like this:

  STRIP_FROM_INC_PATH    = @abs_srcdir@/..
  STRIP_FROM_INC_PATH    += @abs_srcdir@/../include/clang
  STRIP_FROM_INC_PATH    += @abs_srcdir@/../include/clang-c

That way we have the actual compiler include paths in there and we probably get 
the fully working `#include`s one has to do for a certain class. Could you test 
that? Something like `#include "Sema/Sema.h"` for the `Sema` class would be 
IMHO the best solution.

Not sure what STRIP_FROM_PATH is doing, but I think we can leave it at it's 
current value?

Otherwise I think this review is ready to go.


https://reviews.llvm.org/D32113



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to