[PATCH] D40229: [cmake] Remove redundant call to cmake when building host tools.
Don Hinton via llvm-commits
llvm-commits at lists.llvm.org
Wed Dec 6 06:11:21 PST 2017
Sure, that sounds good. Working on patch now. Thanks for reporting this.
On Wed, Dec 6, 2017 at 6:00 AM Pavel Labath <labath at google.com> wrote:
> On 6 December 2017 at 13:40, Don Hinton <hintonda at gmail.com> wrote:
> > Would handling them like the linker flags and only passing them when not
> > cross compiling fix this?
>
> Yes, I believe it will (Is this code ever invoked when *not*
> cross-compiling?).
>
> However, I still think the CROSS_TOOLCHAIN_FLAGS should come last, as
> this gives the user the opportunity to override any of our arguments
> for any reason he sees fit. E.g., I can see how someone might want to
> use CMAKE_BUILD_TYPE=Release for the host tblgen targets, but still
> build the main thing in debug mode.
>
> Is there any reason that can't be done?
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20171206/aee4ab6f/attachment.html>
More information about the llvm-commits
mailing list