[PATCH] D89366: [WebAssembly] v128.load{8, 16, 32, 64}_lane instructions

Thomas Lively via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Oct 14 18:13:47 PDT 2020


tlively added a comment.

In D89366#2331154 <https://reviews.llvm.org/D89366#2331154>, @aheejin wrote:

> Do loads need the vector argument? The proposal has that but I'm wondering why... Can that possibly be an error from the author's side?

Oops, I meant to answer that before but I did not. This is not a mistake because the purpose of the load_lane instructions is to replace just one lane of a vector. The vector to be be modified needs to be taken as an argument.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89366



More information about the cfe-commits mailing list