[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