[llvm-commits] [PATCH] Remove unused parameter from the BoundsChecking pass

Joey Gouly Joey.Gouly at arm.com
Fri Nov 23 02:53:12 PST 2012


Just committed as r168511.

Thanks!

Joey
________________________________________
From: llvm-commits-bounces at cs.uiuc.edu [llvm-commits-bounces at cs.uiuc.edu] On Behalf Of Joey Gouly [joey.gouly at arm.com]
Sent: 22 November 2012 09:19
To: 'Rafael EspĂ­ndola'; nunoplopes at sapo.pt
Cc: llvm-commits at cs.uiuc.edu
Subject: Re: [llvm-commits] [PATCH] Remove unused parameter from the    BoundsChecking pass

Hi Rafael and Nuno,

I'm going to wait until my Clang patch lands before I commit this (so I don't break things.)

Thanks for the review!

Joey

-----Original Message-----
From: Rafael EspĂ­ndola [mailto:rafael.espindola at gmail.com]
Sent: 21 November 2012 14:07
To: Joey Gouly
Cc: llvm-commits at cs.uiuc.edu
Subject: Re: [llvm-commits] [PATCH] Remove unused parameter from the BoundsChecking pass

On 21 November 2012 07:21, Joey Gouly <joey.gouly at arm.com> wrote:
> Hi all,
>
> The penalty parameter has been unused since r158920.
>
> This patch removes it, please review!

You can also remove the "unsigned Penalty" field. LGTM with that.

> Thanks,
> Joey

Cheers,
Rafael





_______________________________________________
llvm-commits mailing list
llvm-commits at cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits





More information about the llvm-commits mailing list