[PATCH] Schedule discriminators pass.

Diego Novillo dnovillo at google.com
Tue Feb 18 14:19:29 PST 2014


On Tue, Feb 18, 2014 at 5:16 PM, Eric Christopher <echristo at gmail.com> wrote:
> On Tue, Feb 18, 2014 at 2:14 PM, Diego Novillo <dnovillo at google.com> wrote:
>> On Tue, Feb 18, 2014 at 5:09 PM, Eric Christopher <echristo at gmail.com> wrote:
>>>
>>>   Last change doesn't seem to be required to schedule the pass.
>>
>> It is. Without it, that test will fail. The pass that adds
>> discriminators generates a new lexical block in that test.
>>
>
> Prereq rather than required to go in at the same time? I.e. will
> anything break if you just commit it with the other code?

Required to go in at the same time. Both D2691 and D2692 should be
committed together.

Diego.



More information about the llvm-commits mailing list