[all-commits] [llvm/llvm-project] 61b144: [Inliner] Add extra test for callsite noalias meta...

[Nikita Popov via All-commits]

  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 61b1446a9e45beb77705dce7358b511dd0e8800e
      https://github.com/llvm/llvm-project/commit/61b1446a9e45beb77705dce7358b511dd0e8800e
  Author: Nikita Popov <nikita.ppv at gmail.com>
  Date:   2020-10-31 (Sat, 31 Oct 2020)

  Changed paths:
    M llvm/test/Transforms/Inline/noalias-cs.ll

  Log Message:
  -----------
  [Inliner] Add extra test for callsite noalias metadata (NFC)

Test the case where the callee does not use noalias metadata itself.
This case is currently handled inconsistently.


  Commit: 27f647d117087ca11959e232e6443f4aee31e966
      https://github.com/llvm/llvm-project/commit/27f647d117087ca11959e232e6443f4aee31e966
  Author: Nikita Popov <nikita.ppv at gmail.com>
  Date:   2020-10-31 (Sat, 31 Oct 2020)

  Changed paths:
    M llvm/lib/Transforms/Utils/InlineFunction.cpp
    M llvm/test/Transforms/Inline/noalias-cs.ll

  Log Message:
  -----------
  [Inliner] Consistently apply callsite noalias metadata

Previously, !noalias and !alias.scope metadata on the call site was
applied as part of CloneAliasScopeMetadata(), which short-circuits
if the callee does not use any noalias metadata itself. However,
these two things have no relation to each other.

Consistently apply !noalias and !alias.scope metadata by integrating
this into an existing function that handled !llvm.access.group and
!llvm.mem.parallel_loop_access metadata. The handling for all of
these metadata kinds essentially the same.


Compare: https://github.com/llvm/llvm-project/compare/11efd002b1e6...27f647d11708