[all-commits] [llvm/llvm-project] edf56f: [LoopInterchange] Don't rely on ASSERTS build for ...

[Sjoerd Meijer via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: edf56f1fa27dce36c2b537290c26fec1af1a1140
      https://github.com/llvm/llvm-project/commit/edf56f1fa27dce36c2b537290c26fec1af1a1140
  Author: Sjoerd Meijer <smeijer at nvidia.com>
  Date:   2024-11-19 (Tue, 19 Nov 2024)

  Changed paths:
    A llvm/test/Transforms/LoopInterchange/call-instructions-remarks.ll
    M llvm/test/Transforms/LoopInterchange/call-instructions.ll
    A llvm/test/Transforms/LoopInterchange/inner-indvar-depend-on-outer-indvar-remark.ll
    M llvm/test/Transforms/LoopInterchange/inner-indvar-depend-on-outer-indvar.ll
    M llvm/test/Transforms/LoopInterchange/innermost-latch-uses-values-in-middle-header.ll
    M llvm/test/Transforms/LoopInterchange/interchange-flow-dep-outer.ll
    M llvm/test/Transforms/LoopInterchange/interchange-no-deps.ll
    M llvm/test/Transforms/LoopInterchange/interchanged-loop-nest-3.ll
    M llvm/test/Transforms/LoopInterchange/not-interchanged-dependencies-1.ll
    M llvm/test/Transforms/LoopInterchange/not-interchanged-loop-nest-3.ll
    M llvm/test/Transforms/LoopInterchange/not-interchanged-tightly-nested.ll

  Log Message:
  -----------
  [LoopInterchange] Don't rely on ASSERTS build for tests. NFC. (#116780)

A lot of interchange tests unnecessary relied on a build with ASSERTS
enabled. Instead, simply check the IR output for both negative and
positive tests so that we don't rely on debug messages. This increases
test coverage as these tests will now also run with non-assert builds.
For a couple of files keeping some of the debug tests was useful, so
separated out them out and moved them to a similarly named *-remarks.ll
file.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications