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

Renato Golin via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 10 02:25:07 PST 2015


rengolin accepted this revision.
rengolin added a comment.
This revision is now accepted and ready to land.

Hi Sanjay,

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.

cheers,
--renato


http://reviews.llvm.org/D14469





More information about the llvm-commits mailing list