Page MenuHomePhabricator
Feed Advanced Search

Thu, Jun 10

wvoquine added a comment to D103912: LoadStoreVectorizer: support different operand orders in the add sequence match.

The reported build failure is not related to the current change and is currently faced by other builds as well:

Thu, Jun 10, 3:58 PM · Restricted Project
wvoquine edited reviewers for D103912: LoadStoreVectorizer: support different operand orders in the add sequence match, added: bogner; removed: JustinBorb.
Thu, Jun 10, 3:18 PM · Restricted Project
wvoquine added inline comments to D103912: LoadStoreVectorizer: support different operand orders in the add sequence match.
Thu, Jun 10, 2:48 PM · Restricted Project
wvoquine updated the diff for D103912: LoadStoreVectorizer: support different operand orders in the add sequence match.

Implemented the suggested naming and code style changes.

Thu, Jun 10, 2:48 PM · Restricted Project

Tue, Jun 8

wvoquine added a comment to D103912: LoadStoreVectorizer: support different operand orders in the add sequence match.

The failing test seems to be not related to the change.

Tue, Jun 8, 3:01 PM · Restricted Project
wvoquine updated the diff for D103912: LoadStoreVectorizer: support different operand orders in the add sequence match.

Fixed the code style issue.
Implemented operand order picking loop instead of a sequence of ifs.

Tue, Jun 8, 12:30 PM · Restricted Project
wvoquine set the repository for D103912: LoadStoreVectorizer: support different operand orders in the add sequence match to rG LLVM Github Monorepo.
Tue, Jun 8, 10:15 AM · Restricted Project
wvoquine added reviewers for D103912: LoadStoreVectorizer: support different operand orders in the add sequence match: arsenm, JustinBorb, volkan.
Tue, Jun 8, 10:14 AM · Restricted Project
wvoquine requested review of D103912: LoadStoreVectorizer: support different operand orders in the add sequence match.
Tue, Jun 8, 10:13 AM · Restricted Project

May 12 2021

wvoquine added inline comments to D101680: Change the context instruction for computeKnownBits in LoadStoreVectorizer pass.
May 12 2021, 3:31 PM · Restricted Project

May 11 2021

wvoquine added a comment to D101680: Change the context instruction for computeKnownBits in LoadStoreVectorizer pass.

Thanks for accepting!

May 11 2021, 5:31 PM · Restricted Project

May 10 2021

wvoquine added inline comments to D101680: Change the context instruction for computeKnownBits in LoadStoreVectorizer pass.
May 10 2021, 4:33 PM · Restricted Project

May 6 2021

wvoquine added inline comments to D101680: Change the context instruction for computeKnownBits in LoadStoreVectorizer pass.
May 6 2021, 9:02 PM · Restricted Project
wvoquine updated the diff for D101680: Change the context instruction for computeKnownBits in LoadStoreVectorizer pass.

Update the test ld_v2i8_add_context to align it with the comment put for the test: the llvm.assume should be placed between the two loads.

May 6 2021, 8:55 PM · Restricted Project
wvoquine updated the diff for D101680: Change the context instruction for computeKnownBits in LoadStoreVectorizer pass.

Added tests where the assume intrinsic is placed between the vectorized instructions or even after them in the end of the basic block.

May 6 2021, 8:42 PM · Restricted Project
wvoquine added inline comments to D101680: Change the context instruction for computeKnownBits in LoadStoreVectorizer pass.
May 6 2021, 7:28 PM · Restricted Project
wvoquine added inline comments to D101680: Change the context instruction for computeKnownBits in LoadStoreVectorizer pass.
May 6 2021, 12:03 PM · Restricted Project

May 3 2021

wvoquine added inline comments to D101680: Change the context instruction for computeKnownBits in LoadStoreVectorizer pass.
May 3 2021, 10:43 AM · Restricted Project
wvoquine updated subscribers of D101680: Change the context instruction for computeKnownBits in LoadStoreVectorizer pass.
May 3 2021, 10:31 AM · Restricted Project

Apr 30 2021

wvoquine updated the diff for D101680: Change the context instruction for computeKnownBits in LoadStoreVectorizer pass.
Apr 30 2021, 7:16 PM · Restricted Project
wvoquine added inline comments to D101680: Change the context instruction for computeKnownBits in LoadStoreVectorizer pass.
Apr 30 2021, 5:46 PM · Restricted Project
wvoquine requested review of D101680: Change the context instruction for computeKnownBits in LoadStoreVectorizer pass.
Apr 30 2021, 5:03 PM · Restricted Project
wvoquine added a comment to D101576: Add support for llvm.assume intrinsic to the LoadStoreVectorizer pass.

Could someone merge the change?

Apr 30 2021, 10:12 AM · Restricted Project

Apr 29 2021

wvoquine updated the diff for D101576: Add support for llvm.assume intrinsic to the LoadStoreVectorizer pass.

Fixed the non-legacy pass compilation issue.

Apr 29 2021, 8:01 PM · Restricted Project
wvoquine requested review of D101576: Add support for llvm.assume intrinsic to the LoadStoreVectorizer pass.
Apr 29 2021, 2:51 PM · Restricted Project