<div dir="ltr">Here is a patch that adds a handful of options to the Darwin driver. There's only a few because I'd like to know if I'm on the right track or not. I've tried to model the implementation and style off of the various existing bits of drivers and linking contexts for each platform.<div>
<br></div><div style>I have a few questions, I suspect most of which are for Nick:</div><div style>- What should the granularity of the tests in DarwinLdDriverTest.cpp be? For now I've put all of the new dylib flags into the "Dylib" test instead of each flag getting its own test. This seems to mirror WinLinkDriverTest.cpp's "Basic" test, though it also has tests for individual options.</div>
<div style>- Are the doxygen in MachOLinkingContext too verbose? Should they just be basic setters/getter pairs with no comments, like PECOFFLinkingContext?</div><div style>- How should the dylib compatibility and current version flags be stored? The minimum OS version is immediately validated outside of validateImpl and stored into a PackedVersion. There is no getter to access the exact value for that field, just the minOS helper.</div>
<div style>- What are the style guidelines for the tablegen files and the unit tests? The LDOptions.td file seem to disregard the 80 column idea completely.</div><div style><br></div><div style>-- Joe Ranieri</div></div>