[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:32:07 PST 2017
Sorry, didn't answer you question.
On Wed, Dec 6, 2017 at 6:11 AM, Don Hinton <hintonda at gmail.com> wrote:
> 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?).
>>
>
Yes. This was the original motivation, e.g., when passing
-DLLVM_OPTIMIZED_TABLEGEN=On.
>
>> 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/74bd23e3/attachment.html>
More information about the llvm-commits
mailing list