[PATCH] D68244: [InstCombine] don't assume 'inbounds' for bitcast pointer to GEP transform (PR43501)

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 30 13:59:39 PDT 2019


spatel created this revision.
spatel added reviewers: nlopes, efriedma, regehr.
Herald added subscribers: hiraditya, mcrosier.
Herald added a project: LLVM.

I don't know the overall impact of this change, or if it goes far enough, but based on:
https://bugs.llvm.org/show_bug.cgi?id=43501
...we can't declare the GEP 'inbounds' in general, so I just removed that bit to try to make this safe.


https://reviews.llvm.org/D68244

Files:
  clang/test/CodeGen/aapcs-bitfield.c
  clang/test/CodeGenCXX/microsoft-abi-dynamic-cast.cpp
  clang/test/CodeGenCXX/microsoft-abi-typeid.cpp
  llvm/lib/Transforms/InstCombine/InstCombineCasts.cpp
  llvm/test/Transforms/InstCombine/addrspacecast.ll
  llvm/test/Transforms/InstCombine/cast.ll
  llvm/test/Transforms/InstCombine/load-bitcast-vec.ll
  llvm/test/Transforms/InstCombine/memset.ll
  llvm/test/Transforms/InstCombine/unpack-fca.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D68244.222494.patch
Type: text/x-patch
Size: 32282 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190930/6c5bd1b7/attachment-0001.bin>


More information about the llvm-commits mailing list