[all-commits] [llvm/llvm-project] 3c9bcf: [Clang][Coroutine][DebugInfo] Relax test ordering ...

[Jeremy Morse via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 3c9bcf0e3549d89b31e19e67a5a16babdee2c72d
      https://github.com/llvm/llvm-project/commit/3c9bcf0e3549d89b31e19e67a5a16babdee2c72d
  Author: Jeremy Morse <jeremy.morse at sony.com>
  Date:   2021-04-26 (Mon, 26 Apr 2021)

  Changed paths:
    M clang/test/CodeGenCoroutines/coro-dwarf.cpp

  Log Message:
  -----------
  [Clang][Coroutine][DebugInfo] Relax test ordering requirement

The test added in D97533 (and modified by this patch) has some overly
strict printed metadata ordering requirements, specifically the
interleaving of DILocalVariable nodes and DILocation nodes. Slight changes
in metadata emission can easily break this unfortunately.

This patch stops after clang codegen rather than allowing the coro splitter
to run,  and reduces the need for ordering: it picks out the
DILocalVariable nodes being sought, in any order (CHECK-DAG), and doesn't
examine any DILocations. The implicit CHECK-NOT is what's important: the
test seeks to ensure a duplicate set of DILocalVariables aren't emitted in
the same scope.

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