[PATCH] Remove LoopVectorizationCostModel::AT

[Eric Christopher]

I've gone ahead and committed the fix and followed up to the thread.

Thanks!

-eric

On Tue, Oct 14, 2014 at 5:03 PM, David Blaikie <dblaikie at gmail.com> wrote:
> & 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
>