[all-commits] [llvm/llvm-project] de127d: [InstCombine] Add tests for PR48577 (NFC)

Nikita Popov via All-commits all-commits at lists.llvm.org
Wed Dec 23 12:03:46 PST 2020


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

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

  Log Message:
  -----------
  [InstCombine] Add tests for PR48577 (NFC)


  Commit: 899faa50f206073cdd8eeaaa130ffa15f850e656
      https://github.com/llvm/llvm-project/commit/899faa50f206073cdd8eeaaa130ffa15f850e656
  Author: Nikita Popov <nikita.ppv at gmail.com>
  Date:   2020-12-23 (Wed, 23 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:
  -----------
  [InstCombine] Check inbounds in load/store of gep null transform (PR48577)

If the GEP isn't inbounds, then accessing a GEP of null location
is generally not UB.

While this is a minimal fix, the GEP of null handling should
probably be its own fold.


Compare: https://github.com/llvm/llvm-project/compare/fc41777702aa...899faa50f206


More information about the All-commits mailing list