<p dir="ltr">I'd probably split it into two subclasses of Verify. Or split out adding a new tool name based on compile time option and adding each action with the specific tool based on the command line. </p>
<p dir="ltr">Sorry for terseness, on a phone on a bus. If you need more I'll try to get it in a bit. </p>
<div class="gmail_quote">On Feb 5, 2014 3:29 PM, "Ben Langmuir" <<a href="mailto:blangmuir@apple.com">blangmuir@apple.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div style="word-wrap:break-word">So would the change you’re envisioning be to subclass VerifyJobAction and/or add a new ActionClass?  All of the other ActionClasses are unambiguous in what tool they need.<div><br></div><div>
Ben<br><div><br><div><div>On Feb 5, 2014, at 2:35 PM, Ben Langmuir <<a href="mailto:blangmuir@apple.com" target="_blank">blangmuir@apple.com</a>> wrote:</div><br><blockquote type="cite"><div style="word-wrap:break-word">
Sounds reasonable, I’ll look into it.<div><br></div><div>Ben<br><div><br><div><div>On Feb 5, 2014, at 2:33 PM, Eric Christopher <<a href="mailto:echristo@gmail.com" target="_blank">echristo@gmail.com</a>> wrote:</div>
<br><blockquote type="cite"><p dir="ltr">The idea was that the hook could be used for any verification, just a specific one of debug at the moment. It can easily be changed to support more verifications (and should). :)</p>
<p dir="ltr">-eric </p>
<div class="gmail_quote">On Feb 5, 2014 2:32 PM, "Ben Langmuir" <<a href="mailto:blangmuir@apple.com" target="_blank">blangmuir@apple.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<br>
On Feb 5, 2014, at 11:00 AM, Eric Christopher <<a href="mailto:echristo@gmail.com" target="_blank">echristo@gmail.com</a>> wrote:<br>
<br>
> Out of curiosity would it work to have this as a Verify action rather<br>
> than a Compile action that has no output?<br>
><br>
> -eric<br>
<br>
VerifyJobAction seems to be specific to verifying debug information.  Perhaps that should change?  Or maybe it should be renamed to VerifyDebugInfoJobAction?  I’m not sure.<br>
<br>
Ben</blockquote></div>
</blockquote></div><br></div></div></div>_______________________________________________<br>cfe-commits mailing list<br><a href="mailto:cfe-commits@cs.uiuc.edu" target="_blank">cfe-commits@cs.uiuc.edu</a><br><a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
</blockquote></div><br></div></div></div></blockquote></div>