<div><br></div><div><br><div class="gmail_quote"><div dir="ltr">On Mon, Jan 7, 2019 at 8:39 AM <<a href="mailto:paul.robinson@sony.com">paul.robinson@sony.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
<br>
> -----Original Message-----<br>
> From: llvm-dev [mailto:<a href="mailto:llvm-dev-bounces@lists.llvm.org" target="_blank">llvm-dev-bounces@lists.llvm.org</a>] On Behalf Of Alex<br>
> Bradbury via llvm-dev<br>
> Sent: Monday, January 07, 2019 11:33 AM<br>
> To: Mehdi AMINI<br>
> Cc: LLVM Dev<br>
> Subject: Re: [llvm-dev] Removing LLVM_ALWAYS_INLINE from ADT classes<br>
> <br>
> On Sun, 6 Jan 2019 at 00:42, Mehdi AMINI via llvm-dev<br>
> <<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>> wrote:<br>
> ><br>
> ><br>
> ><br>
> > On Fri, Jan 4, 2019 at 3:52 PM Paweł Bylica via llvm-dev <llvm-<br>
> <a href="mailto:dev@lists.llvm.org" target="_blank">dev@lists.llvm.org</a>> wrote:<br>
> >><br>
> >> On Sat, Jan 5, 2019 at 12:38 AM Duncan P. N. Exon Smith via llvm-dev<br>
> <<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>> wrote:<br>
> >>><br>
> >>> This makes sense to me.<br>
> >>><br>
> >>> One concern is that this in itself will slow down the build, since<br>
> tablegen will get even slower.  Ideally, there would be some (perhaps<br>
> default?) configuration where we build the tablegen binaries with<br>
> optimizations on and then use them in the build, as if we were cross-<br>
> compiling.<br>
> >><br>
> >><br>
> >> Maybe something simple like -DLLVM_OPTIMIZE_TABLEGEN=ON (enabled by<br>
> default for Debug builds). You would need to set it to off only if you<br>
> want to debug tablegen.<br>
> ><br>
> ><br>
> > I proposed this a while back, but it think there were some problems with<br>
> it, I don't remember what exactly but maybe related to bootstrapping / 2-<br>
> stages builds?<br>
> ><br>
> > + Chris, maybe he remembers something?<br>
> <br>
> LLVM_OPTIMIZED_TABLEGEN=ON is a handy way of speeding up build times,<br>
> but it really shouldn't be recommended for anyone who is modifying .td<br>
> files. You run the risk of making .td modifications that trigger<br>
> asserts for those not using LLVM_OPTIMIZED_TABLEGEN.<br>
<br>
Can we make LLVM_OPTIMIZED_TABLEGEN=ON build it as Release+Asserts?</blockquote><div dir="auto"><br></div><div dir="auto">There is (use to be) some very expensive assertions in llvm-tblgen.</div><div dir="auto"><br></div><div dir="auto">— </div><div dir="auto">Mehdi</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"></blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
--paulr<br>
<br>
> <br>
> Best,<br>
> <br>
> Alex<br>
> _______________________________________________<br>
> LLVM Developers mailing list<br>
> <a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
> <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br>
</blockquote></div></div>