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

Rui Ueyama ruiu at google.com
Thu Oct 24 15:21:46 PDT 2013


By pushing I'm not referencing to this particular thing.

Do you think you can enable the tests per-port basis, as Reid suggested? I
think that's better than guarding it with #ifdef, as you can enable it now
for ELF (so that it won't bit rotten while being disabled by #ifdef) and
other port owners can fix it independently.


On Thu, Oct 24, 2013 at 2:35 PM, Shankar Easwaran
<shankare at codeaurora.org>wrote:

> On 10/24/2013 4:27 PM, Rui Ueyama wrote:
>
>> LLD is still in an early stage of development. We've got many things to
>> do,
>> and this is not the highest priority among them that needs to be done by
>> next week or otherwise we should break Mach-O and COFF ports. I thank you
>> for putting your effort on this new test scheme, but I can't promise we
>> can
>> fix them in such a short period of time.
>>
>> I hate to say but I'm honestly disappointed. We should respect each other
>> in open source development, rather than forcing each other to do
>> something.
>> If you find someone have a different opinion, you need to elaborate why
>> you
>> think in a different way, rather than replying with a short email
>> repeating
>> basically the same thing. These days, virtually you and me submitted 90%
>> of
>> patches to LLD (I think I submitted like 60%), but I sometimes found that
>> you pushed your opinion too hard without enough explanation. That's not
>> good. Please respect each other, or this development style does not work.
>>
> I am so sorry If I implied that, I was only pushing it because this was
> one of the design goals (the Atom IR) of lld.
>
>
> Thanks
>
> Shankar Easwaran
>
> --
> Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted
> by the Linux Foundation
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20131024/d55a2342/attachment.html>


More information about the llvm-commits mailing list