<div dir="ltr">Hi Justin:<div><br></div><div>The problem was with generators like XCode and MSVC that create multiple configurations, e.g., Release, Debug, etc., and need to rely on passing -DNDEBUG in order to work.  That makes it tough to use a variable like LLVM_ENABLE_DUMP that gets set at configure time..<br></div><div><br></div><div>We could probably do a partial change that doesn't remove the NDEBUG tests, but it still wouldn't work well with XCode and MSVC generators -- would still compile and link, but just not as useful on a per configuration basis.  </div><div><br></div><div>Please let me know if that would be helpful, and I'll create a new diff for you to review.</div><div><br></div><div>thanks...</div><div>don</div><div><br></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Dec 4, 2017 at 4:55 PM, Justin Bogner <span dir="ltr"><<a href="mailto:mail@justinbogner.com" target="_blank">mail@justinbogner.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">Aaron Ballman <<a href="mailto:aaron@aaronballman.com">aaron@aaronballman.com</a>> writes:<br>
> On Mon, Dec 4, 2017 at 5:24 PM, Justin Bogner <<a href="mailto:mail@justinbogner.com">mail@justinbogner.com</a>> wrote:<br>
>> Aaron Ballman via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>> writes:<br>
>>> Author: aaronballman<br>
>>> Date: Sun Oct 15 07:32:27 2017<br>
>>> New Revision: 315854<br>
>>><br>
>>> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=315854&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-<wbr>project?rev=315854&view=rev</a><br>
>>> Log:<br>
>>> Reverting r315590; it did not include changes for llvm-tblgen, which<br>
>>> is causing link errors for several people.<br>
>><br>
>> What's happening with this? Is someone investigating these errors? I'd<br>
>> really like the part of this change that moves LLVM_ENABLE_DUMP into<br>
>> llvm-config.h to get in, as it's currently kind of awkward to use in the<br>
>> half finished state.<br>
>><br>
>>> Error LNK2019 unresolved external symbol "public: void __cdecl<br>
>>> `anonymous namespace'::MatchableInfo::<wbr>dump(void)const "<br>
>>> (?dump@MatchableInfo@?<wbr>A0xf4f1c304@@QEBAXXZ) referenced in function<br>
>>> "public: void __cdecl `anonymous<br>
>>> namespace'::AsmMatcherEmitter:<wbr>:run(class llvm::raw_ostream &)"<br>
>>> (?run@AsmMatcherEmitter@?<wbr>A0xf4f1c304@@QEAAXAEAVraw_<wbr>ostream@llvm@@@Z)<br>
>>> llvm-tblgen D:\llvm\2017\utils\TableGen\<wbr>AsmMatcherEmitter.obj 1<br>
><br>
> I've not been investigating -- I was only trying to get the bot back<br>
> to green. Perhaps Don has worked on this, however.<br>
<br>
</span>Okay, I couldn't find any email trail of followup (not even to mention<br>
the commit was reverted!), so I wasn't sure if I'd just missed the<br>
conversation here.<br>
</blockquote></div><br></div>