[PATCH] D130956: [X86][MC] Always emit `rep` prefix for `bsf`

Kan Shengchen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 1 19:15:48 PDT 2022


skan added a comment.

In D130956#3692473 <https://reviews.llvm.org/D130956#3692473>, @pengfei wrote:

> In D130956#3692469 <https://reviews.llvm.org/D130956#3692469>, @craig.topper wrote:
>
>> Assembler should faithful to the original source. This needs to be done at codegen only I would think.
>
> I see, how about changing the assembly string to `"rep bsf{w}\t{$src, $dst|$dst, $src}"`?

No, I don't think it is a good way. It breaks the support of original bsf.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D130956/new/

https://reviews.llvm.org/D130956



More information about the llvm-commits mailing list