[all-commits] [llvm/llvm-project] e72f16: [Test] Add a JumpThreading test exposing a bug in ...

Daniil Suchkov via All-commits all-commits at lists.llvm.org
Mon Jun 7 14:41:02 PDT 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: e72f16b7e66256dba5496a32546417d438bb035c
      https://github.com/llvm/llvm-project/commit/e72f16b7e66256dba5496a32546417d438bb035c
  Author: Daniil Suchkov <dsuchkov at azul.com>
  Date:   2021-06-07 (Mon, 07 Jun 2021)

  Changed paths:
    A llvm/test/Transforms/JumpThreading/aa-crash-phi-no-args.ll

  Log Message:
  -----------
  [Test] Add a JumpThreading test exposing a bug in BasicAA.


  Commit: d32cc150feb72f315a5bbd34f92e7beca21a50da
      https://github.com/llvm/llvm-project/commit/d32cc150feb72f315a5bbd34f92e7beca21a50da
  Author: Daniil Suchkov <dsuchkov at azul.com>
  Date:   2021-06-07 (Mon, 07 Jun 2021)

  Changed paths:
    M llvm/lib/Analysis/BasicAliasAnalysis.cpp
    M llvm/test/Transforms/JumpThreading/aa-crash-phi-no-args.ll

  Log Message:
  -----------
  [BasicAA] Handle PHIs without incoming values gracefully

Fix a bug introduced by f6f6f6375d1a4bced8a6e79a78726ab32b8dd879.
Now for empty PHIs, instead of crashing on assert(hasVal()) in
Optional's internals, we'll return NoAlias, as we did before that patch.

Differential Revision: https://reviews.llvm.org/D103831


Compare: https://github.com/llvm/llvm-project/compare/9145a3d4ab7e...d32cc150feb7


More information about the All-commits mailing list