[PATCH] D72699: [ARM][MVE] VTP Block fix

Sjoerd Meijer via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 14 05:56:50 PST 2020


SjoerdMeijer created this revision.
SjoerdMeijer added reviewers: samparker, dmgreen.
Herald added subscribers: hiraditya, kristof.beyls.
Herald added a project: LLVM.

Fix for the case that was missing where we create 2 VPT blocks predicated on the same instruction. The problem was that for each VPT block, we save the predicate statement to a list so that we can remove them all later in one go, but the same instruction was added twice. Thus, we were running in an assert trying to remove the same instruction twice. To avoid this, the instructions are now saved to a set.


https://reviews.llvm.org/D72699

Files:
  llvm/lib/Target/ARM/MVEVPTBlockPass.cpp
  llvm/test/CodeGen/Thumb2/mve-vpt-2-blocks-1-pred.mir

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D72699.237956.patch
Type: text/x-patch
Size: 5159 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200114/ad960aa8/attachment.bin>


More information about the llvm-commits mailing list