[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