[PATCH] Add a case to LiveIntervalAnalysis::HandleMoveUp

Jakob Stoklund Olesen stoklund at 2pi.dk
Tue Mar 5 13:09:57 PST 2013


On Mar 5, 2013, at 11:20 AM, Vincent Lejeune <vljn at ovi.com> wrote:

> I reworked my patch a bit so that it now updates the read-undef flags.
> I planned to let MachineScheduler do it however updating the flag inside HandleMove allows to spot error quickier.
> 
> Can you have a look at it please ? 

Did you read my previous mail about the problem with live-out values? You didn't address the problem.

Andy said he is going to add a -verify-scheduler command line option that runs the machine code verifier before and after the MI scheduler. You should have some test cases that contains multiple basic blocks, and exercise this code with -stress-scheduler and -verify-sceuduler.

You should also make sure that the LLVM nightly test suite still works for x86 with -enable-misched -verify-scheduler -stress-scheduler.

I would recommend that you read the code in LiveRangeCalc.cpp to better understand how live intervals work.

/jakob




More information about the llvm-commits mailing list