[PATCH] Remove LoopVectorizationCostModel::AT

[Hal Finkel]

Sorry about that everyone; was pulled away from my computer soon after I committed. Thanks for taking care of it!

 -Hal

----- Original Message -----
> From: "David Blaikie" <dblaikie at gmail.com>
> To: "Ehsan Akhgari" <ehsan.akhgari at gmail.com>, "Rafael Ávila de Espíndola" <rafael.espindola at gmail.com>
> Cc: llvm-commits at cs.uiuc.edu
> Sent: Tuesday, October 14, 2014 7:03:57 PM
> Subject: Re: [PATCH] Remove LoopVectorizationCostModel::AT
> 
> 
> 
> & I see this might've been recently introduced in 219741 - in that
> case, it's usually best to reply to the commit mail to point it out
> to the original author so they can clean it up.
> 
> 
> On Tue, Oct 14, 2014 at 5:02 PM, David Blaikie < dblaikie at gmail.com >
> wrote:
> 
> 
> Is the warning emitted in a +Asserts build of LLVM? (usually such
> warnings don't persist long in the codebase, because so many of us
> build with -Werror +Asserts, so that leaves the clang warnings for
> -Asserts builds, but you have to be careful that you don't break the
> +Asserts build when you fix these (eg: the variable may be unused in
> -Asserts, but used in +Asserts, so removing it will fix the -Asserts
> warning, but break the +Asserts build))
> 
> http://reviews.llvm.org/D5788
> 
> 
> 
> 
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
> 

-- 
Hal Finkel
Assistant Computational Scientist
Leadership Computing Facility
Argonne National Laboratory