[all-commits] [llvm/llvm-project] a00ad8: [Test] rewrite inline_nossp.ll

Nick Desaulniers via All-commits all-commits at lists.llvm.org
Mon Jun 28 13:53:15 PDT 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: a00ad8599045ce7d0b5d2a64a8d22267df37bd14
      https://github.com/llvm/llvm-project/commit/a00ad8599045ce7d0b5d2a64a8d22267df37bd14
  Author: Nick Desaulniers <ndesaulniers at google.com>
  Date:   2021-06-28 (Mon, 28 Jun 2021)

  Changed paths:
    M llvm/test/Transforms/Inline/inline_nossp.ll

  Log Message:
  -----------
  [Test] rewrite inline_nossp.ll

While adding remark based tests in D104944, I noticed that the tests
that we were passing were passing for the wrong reason. They were
passing because the dynamic allocas were preventing inlining, not the
code I added in D91816.

Rewrite and simplify the test. Add remark based checks to validate we're
preventing inline substitutions for the right reasons.

Reviewed By: MaskRay

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




More information about the All-commits mailing list