[PATCH] [go] Teach the go cmake build functions to funnel the include directories down into the cgo-setup variables of llvm-go.

Chandler Carruth chandlerc at gmail.com
Mon Dec 29 14:50:45 PST 2014


On Mon, Dec 29, 2014 at 2:39 PM, Peter Collingbourne <peter at pcc.me.uk>
wrote:

> ================
> Comment at: cmake/modules/AddLLVM.cmake:636
> @@ +635,3 @@
> +      set(cppflags "${cppflags} -I${d}")
> +      set(cxxflags "${cxxflags} -I${d}")
> +    endforeach(d)
> ----------------
> Are you sure you need to set cxxflags? As far as I know, cppflags should
> be picked up for C and C++.
>

Nope. You're suggestion works fine, going with that.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20141229/950fa732/attachment.html>


More information about the llvm-commits mailing list