[lld] r193300 - [PassManager] add ReaderWriter{Native, YAML} to the Driver.
Chandler Carruth
chandlerc at google.com
Thu Oct 24 13:10:39 PDT 2013
On Thu, Oct 24, 2013 at 1:04 PM, Nick Kledzik <kledzik at apple.com> wrote:
> There is a catch-22 here. Without the new passes it is hard for anyone to
> fix the other problems in lld that were causing the test case failures.
>
> Can we do this in steps:
> 1) commit the code changes with the lines that use the two new passes
> commented out
> 2) owners of the broken areas can uncomment the code (enabling the passes)
> and fix their issues
> 3) the final commit is to enable the two new passes when all test cases
> pass
>
If folks want this, cool. I usually use a macro to enable/disable.
That said, I think that Shankar should make a reasonable effort to fix the
other targets. I don't think its likely that hard. He hadn't even asked for
help due to it being harder than expected when he landed this patch.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20131024/6c108179/attachment.html>
More information about the llvm-commits
mailing list