[all-commits] [llvm/llvm-project] ef2f84: Revert "[InstCombine] Check inbounds in load/store...

Nikita Popov via All-commits all-commits at lists.llvm.org
Thu Dec 24 03:37:21 PST 2020


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: ef2f843347baca1ece69066a3c3a7afa45532079
      https://github.com/llvm/llvm-project/commit/ef2f843347baca1ece69066a3c3a7afa45532079
  Author: Nikita Popov <nikita.ppv at gmail.com>
  Date:   2020-12-24 (Thu, 24 Dec 2020)

  Changed paths:
    M llvm/lib/Transforms/InstCombine/InstCombineLoadStoreAlloca.cpp
    M llvm/test/Transforms/InstCombine/load.ll
    M llvm/test/Transforms/InstCombine/store.ll

  Log Message:
  -----------
  Revert "[InstCombine] Check inbounds in load/store of gep null transform (PR48577)"

This reverts commit 899faa50f206073cdd8eeaaa130ffa15f850e656.

Upon further consideration, this does not fix the right issue.
Doing this fold for non-inbounds GEPs is legal, because the
resulting pointer is still based-on null, which has no associated
address range, and as such and access to it is UB.

https://bugs.llvm.org/show_bug.cgi?id=48577#c3




More information about the All-commits mailing list