[PATCH] D53872: [WebAssembly] Add V128 to WebAssemblyInstrInfo::copyPhysReg

Heejin Ahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 7 13:56:02 PST 2018


aheejin accepted this revision.
aheejin added a comment.
This revision is now accepted and ready to land.

Otherwise LGTM



================
Comment at: test/CodeGen/WebAssembly/regcopy.mir:31
+  - { id: 0, class: i32 }
+  - { id: 1, class: i32 }
+liveins:
----------------
tlively wrote:
> aheejin wrote:
> > I think you can remove these for virtual registers?
> Doesn't seem so, but I removed some other things.
I think you can remove this and add `:i32` to the code itself as you did for `%0`, as in
`%0:i32 = COPY %1:i32`
The same for other `%1` usages in other functions.


Repository:
  rL LLVM

https://reviews.llvm.org/D53872





More information about the llvm-commits mailing list