[PATCH] D145893: [X86][MC]Fix wrong action for encode movdir64b

Kan Shengchen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Mar 12 23:10:23 PDT 2023


skan added a comment.

> Now llvm-mc can encode the asm 'movdir64b (%edx), rcx' but the result is the same as
> 'movdir64b (%edx), ecx', which offend users' intention, while gcc will object this
> action and give a warning.

I am confused about this statement.

1. What does the "Now" mean here? Before this patch or after this patch?
2. I can only see memory operand change in this patch, why it affects the register operand?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D145893



More information about the llvm-commits mailing list