[PATCH] D13142: [PowerPC] Replace cntlz[.] with cntlzw[.]
Bill Wendling via llvm-commits
llvm-commits at lists.llvm.org
Fri Sep 25 16:55:10 PDT 2015
Unfortunately, I only vaguely recall this patch. Is there a reason not to
have support for both? I don't think we support POWER in LLVM though (I
could be wrong there) ... You may need to ask one of the ppc experts here.
On Fri, Sep 25, 2015, 5:46 AM Hal Finkel <hfinkel at anl.gov> wrote:
> [+David Fang; maybe you can comment on Darwin]
> ----- Original Message -----
> > From: hfinkel at anl.gov
> > To: tom at bumblecow.com, isanbard at gmail.com, hfinkel at anl.gov
> > Cc: llvm-commits at lists.llvm.org
> > Sent: Friday, September 25, 2015 7:44:12 AM
> > Subject: Re: [PATCH] D13142: [PowerPC] Replace cntlz[.] with cntlzw[.]
> > 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
> > 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
> Hal Finkel
> Assistant Computational Scientist
> Leadership Computing Facility
> Argonne National Laboratory
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the llvm-commits