[PATCH] D13142: [PowerPC] Replace cntlz[.] with cntlzw[.]

hfinkel@anl.gov via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 25 05:44:12 PDT 2015


hfinkel added a comment.

I think this is fine, although given that we've accepted the older form for a long time (and it was apparently in use, at least on some older Darwin systems), we should be kind to our users and continue to accept it as an alias. Please add the InstAlias defs and associated parsing tests (set InstAlias's Emit parameter to 0).

Also, as best I can tell from https://llvm.org/bugs/show_bug.cgi?id=1095 and http://lists.llvm.org/pipermail/llvm-commits/Week-of-Mon-20070115/042738.html, Darwin also wanted the newer mnemonic spelling. This is confusing, given that we have old tests specifically for the old form. Bill, if you can remember from that long ago, can you comment on what we should do here for Darwin?


http://reviews.llvm.org/D13142





More information about the llvm-commits mailing list