[PATCH] D14469: [ARM] add overrides for isCheapToSpeculateCttz() and isCheapToSpeculateCtlz()

Sanjay Patel via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 10 08:40:07 PST 2015


spatel added a comment.

In http://reviews.llvm.org/D14469#285988, @rengolin wrote:

> The logic seems sound. I'm curious as to why the lit.local.cfg wasn't there in the first place. :)
>
> LGTM, with maybe a split, committing the lit config first, then the change itself.


Thanks, Renato! The lit config isn't there because the directory isn't there at all. This would be the first ARM-specific regression test under SimplifyCFG.


http://reviews.llvm.org/D14469





More information about the llvm-commits mailing list