<div dir="ltr">On Sat, Aug 8, 2015 at 12:10 AM, Justin Bogner via llvm-commits <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Apparently if you make a typo in the argument to CMAKE_BUILD_TYPE, cmake<br>
silently accepts this but doesn't apply any particular build type to<br>
your build. For LLVM, this means you get a build that doesn't really<br>
make any sense - it's sort of a debug build with asserts disabled.<br>
<br>
I'm not really sure why cmake allows this at all and the attached patch<br>
is basically a horrible hack to avoid the issue, but I just spent an<br>
obscene amount of time debugging some bizarre behaviour I ran into<br>
because I set the build type to RelWithDebugInfo instead of<br>
RelWithDebInfo.<br>
<br>
Better suggestions welcome.<br></blockquote><div><br></div><div>Personally, I think that "Invalid value for CMAKE_BUILD_TYPE: ${CMAKE_BUILD_TYPE}" would be a better message.</div><div><br></div><div>I recently ran into this myself (and luckily saw the warning from cmake saying it was invalid and was going to default to something else), so I like this idea very much.</div><div><br></div><div>LGTM (Ill let you decide on the wordsmithing).</div><div> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature">Saleem Abdulrasool<br>compnerd (at) compnerd (dot) org</div>
</div></div>