[PATCH] D81057: [SVE] Fix ubsan issues in DecodeIITType

David Sherwood via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 2 23:58:05 PDT 2020


david-arm created this revision.
david-arm added reviewers: sdesmalen, dsanders.
Herald added subscribers: llvm-commits, psnobl, rkruppe, hiraditya, tschuett.
Herald added a reviewer: efriedma.
Herald added a project: LLVM.

In an earlier patch I removed the need for
IITDescriptor::ScalableVecArgument, which involved changing
DecodeIITType to pull out the last IIT_Info from the list. However,
it turns out this is unsafe and causes ubsan failures. I've tried to
fix this a different way by simply passing the last IIT_Info as an
additional argument to DecodeIITType.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D81057

Files:
  llvm/lib/IR/Function.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D81057.268065.patch
Type: text/x-patch
Size: 4371 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200603/559ece44/attachment.bin>


More information about the llvm-commits mailing list