To prepare continuous changes, simplify uimm6 and simm7 operands.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Comment Actions
Update to run check-llvm correctly. I had a problem on my test environment
and I didn't notice check-llvm errors.
Comment Actions
Good catch!
Minor nit. Otw, LGTM.
llvm/lib/Target/VE/VEInstrInfo.td | ||
---|---|---|
29 | How come there is a DecoderMethod for simm7 but not for uimm6? I am not saying this is wrong but i'd like to understand why this works. |
llvm/lib/Target/VE/VEInstrInfo.td | ||
---|---|---|
29 | We don't need decoder functions for unsigned values since the default decoder method works correctly for unsigned immediate values. In order to sign-extend values, we need particular functions. |
How come there is a DecoderMethod for simm7 but not for uimm6? I am not saying this is wrong but i'd like to understand why this works.