[llvm-commits] PATCH: Add machine post dominator pass

Andrew Trick atrick at apple.com
Fri Sep 14 13:43:26 PDT 2012


On Sep 14, 2012, at 11:12 AM, Tom Stellard <tom at stellard.net> wrote:

> Hi,
> 
> The attached patch adds a machine post dominator pass.  This is used by
> both the R600 and AMDIL backends.

Tom,

Your patch looks fine to me. Seems pretty obvious--I wonder if we ever had MachinePostDom and it was ripped out.

I would prefer that you remove the "inline" modifiers. I know it's copied from PostDominators.h, but I still think it's silly.

-Andy



More information about the llvm-commits mailing list