craig.topper added a comment.

In D151863#4390172 <https://reviews.llvm.org/D151863#4390172>, @akshaykhadse 
wrote:

> In D151863#4390132 <https://reviews.llvm.org/D151863#4390132>, @craig.topper 
> wrote:
>
>> I still think replacing the `1` with a valid register for the mode is the 
>> better fix.
>
> So, should I create a new registers? Something like `X86::Sym16`, 
> `X86::Sym32` and `X86::Sym64`?

Use RAX and EAX and AX. It just needs to have the right size.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D151863

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to