[PATCH] D27395: Fixed store operation for a vector of i1.

Elena Demikhovsky via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 6 01:15:07 PST 2016


delena abandoned this revision.
delena added a comment.

> You don't need load-modify-store for an unmasked store; we're allowed to clobber the padding bits, so you can do something like this:

I said that we need load-modify-store for the **masked ** version. (or to ban it at all, I don't really like this idea)

I'm abandoning this review, since we need to make high-level decision about the form of the store.
I'll send RFC later to the dev list.


Repository:
  rL LLVM

https://reviews.llvm.org/D27395





More information about the llvm-commits mailing list