[PATCH] D91790: [ARM][LowOverheadLoops] Convert intermediate vpr use assertion to condition

Sam Tebbs via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 19 05:40:25 PST 2020


samtebbs created this revision.
samtebbs added reviewers: dmgreen, samparker, SjoerdMeijer.
Herald added subscribers: llvm-commits, danielkiss, hiraditya, kristof.beyls.
Herald added a project: LLVM.
samtebbs requested review of this revision.

This converts the intermediate VPR use assertion to a condition in the if-statement to protect against assertion failures in case behaviuour is changed.

This is a follow-up to https://reviews.llvm.org/D90935 and implements the post-approval comments.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D91790

Files:
  llvm/lib/Target/ARM/ARMLowOverheadLoops.cpp
  llvm/test/CodeGen/Thumb2/LowOverheadLoops/vcmp-vpst-combination-across-blocks.mir

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D91790.306383.patch
Type: text/x-patch
Size: 7300 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20201119/e3c270af/attachment.bin>


More information about the llvm-commits mailing list