[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