[lld] r193300 - [PassManager] add ReaderWriter{Native, YAML} to the Driver.

[Chandler Carruth]

On Thu, Oct 24, 2013 at 1:54 PM, Rui Ueyama <ruiu at google.com> wrote:

> On Thu, Oct 24, 2013 at 1:48 PM, Reid Kleckner <rnk at google.com> wrote:
>
>> On Thu, Oct 24, 2013 at 1:41 PM, Shankar Easwaran <
>> shankare at codeaurora.org> wrote:
>>
>>> On 10/24/2013 3:39 PM, Reid Kleckner wrote:
>>>
>>>> Can you let each flavor opt in to the behavior, rather than breaking
>>>> them?
>>>>
>>> No, we cannot do this, as we need the Atom IR to be stable and no hidden
>>> assumptions are carried over to the writer.
>>
>>
>> If you can't do this, then I think the responsibility for fixing all the
>> other targets is on you.  Sorry.
>>
>
> Actually we can do that. I don't know why Shankar thinks we can't, but we
> can definitely propagate some information to the location where the passes
> are added, and enable them only for specific ports. After fixing all the
> ports we can remove the code to propagate the information.
>

The mechanism to selectively use the new behavior is somewhat orthogonal
though.

At the end of the day, the person who needs the change may well end up
needing to write the code for all of the targets in order to get it there.
All of these mechanisms are just ways of extending the amount of time, and
getting incremental patches.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20131024/95a62984/attachment.html>