[PATCH] D49946: Fix crash in bounds checking

Joel Galenson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 30 08:48:31 PDT 2018


jgalenson updated this revision to Diff 157976.
jgalenson marked an inline comment as done.
jgalenson added a comment.
Herald added a subscriber: jfb.

In https://reviews.llvm.org/D49946#1179935, @Ka-Ka wrote:

> I found another reproducer for the problem. This one is slightly different where a different assert blow, but I guess its only a different symptom for the same fault. As this one is slightly smaller you might want to use this one as a testcase instead (or use both)?


Thanks!  I'll use both it and the first one you gave me.  Luckily, it doesn't crash with this patch.


https://reviews.llvm.org/D49946

Files:
  lib/Transforms/Instrumentation/BoundsChecking.cpp
  test/Instrumentation/BoundsChecking/many-traps-2.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D49946.157976.patch
Type: text/x-patch
Size: 7915 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180730/6f200a1b/attachment.bin>


More information about the llvm-commits mailing list