<div dir="ltr">Just in case the below came out with a resolution of "Let's remove it", I submitted this for review:<div><br></div><div>    <a href="http://reviews.llvm.org/D11971">http://reviews.llvm.org/D11971</a></div><div><br></div><div> - Bruce<br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Aug 12, 2015 at 11:21 AM, Bruce Mitchener <span dir="ltr"><<a href="mailto:bruce.mitchener@gmail.com" target="_blank">bruce.mitchener@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hello!<div><br></div><div>This is probably for the Apple People since it relates to debugserver...</div><div><br></div><div>I was going to fix an issue with the cmake build on Mac OS X related to the mig generation step and DNBConfig.h, but when I started digging into things, I noticed that DNBConfig.h isn't needed any longer.</div><div><br></div><div>The only test that it performs is for whether or not 64 bit Mach exceptions can be used.</div><div><br></div><div>The code that actually handled non-64 bit Mach exceptions was removed in 2011 by this commit:</div><div><br></div><blockquote style="margin:0 0 0 40px;border:none;padding:0px"><div><div>commit 0e8147bd867e4cdaae9400f56d02c7aacd40a9b3</div></div><div><div>Author: Greg Clayton <<a href="mailto:gclayton@apple.com" target="_blank">gclayton@apple.com</a>></div></div><div><div>Date:   Fri Oct 28 22:59:14 2011 +0000</div></div><div><div><br></div></div><div><div>    Enabled the "printf" attribute on all debugserver logging functions and fixed</div></div><div><div>    the ensuing mayhem.</div></div><div><div><br></div></div><div><div><br></div></div><div><div><br></div></div><div><div>    git-svn-id: <a href="https://llvm.org/svn/llvm-project/llvdb/trunk@143244" target="_blank">https://llvm.org/svn/llvm-project/llvdb/trunk@143244</a> 91177308-0d34-0410-b5e6-96231b3b80d8</div></div></blockquote><div><br></div><div>So, I'm wondering what the right next step is:</div><div><br></div><div><ul><li>Just fix the issue that I was going to fix in the cmake build system.<br></li><li>Remove the unused test for 64 bit Mach exception handling and that usage of DNBConfig.h</li><li>Remove DNBConfig.h entirely along with the machinery for generating it since nothing actually needs or uses it.</li><li>Something else.</li></ul></div><div>I'm happy to do any of the above options.</div><div><br></div><div>Cheers,</div><div><br></div><div> - Bruce</div><div><br></div><div>PS. HavAVX.s is missing an EOL at EOF, and so it generates a warning. Can I just commit an EOL at EOF without going through Phabricator? That won't break anything, right?</div><div><br></div></div>
</blockquote></div><br></div></div></div>