[all-commits] [llvm/llvm-project] 5d12b9: [ValueTracking] Don't assume readonly function wil...

[Nikita Popov via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 5d12b976b004e5022b523d3bc368aa6794aad988
      https://github.com/llvm/llvm-project/commit/5d12b976b004e5022b523d3bc368aa6794aad988
  Author: Nikita Popov <nikita.ppv at gmail.com>
  Date:   2021-01-24 (Sun, 24 Jan 2021)

  Changed paths:
    M llvm/lib/Analysis/ValueTracking.cpp
    M llvm/test/Transforms/Attributor/nocapture-1.ll
    M llvm/test/Transforms/Attributor/nonnull.ll
    M llvm/test/Transforms/Attributor/read_write_returned_arguments_scc.ll
    M llvm/test/Transforms/FunctionAttrs/nonnull.ll
    M llvm/test/Transforms/GVNHoist/hoist-convergent.ll
    M llvm/test/Transforms/GVNHoist/hoist-pr31891.ll
    M llvm/test/Transforms/Inline/ret_attr_update.ll
    M llvm/test/Transforms/JumpThreading/assume.ll
    M llvm/test/Transforms/JumpThreading/guards.ll
    M llvm/test/Transforms/OpenMP/parallel_deletion.ll
    M llvm/unittests/Analysis/ValueTrackingTest.cpp

  Log Message:
  -----------
  [ValueTracking] Don't assume readonly function will return

This is similar to D94106, but for the
isGuaranteedToTransferExecutionToSuccessor() helper. We should not
assume that readonly functions will return, as this is only true for
mustprogress functions (in which case we already infer willreturn).
As with the DCE change, for now continue assuming that readonly
intrinsics will return, as not all target intrinsics have been
annotated yet.

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