[all-commits] [llvm/llvm-project] e05c10: [DebugInfo] Re-engineer a test to be stricter, add...

[Jeremy Morse via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: e05c10380ce7c18d1a232dcc5baa7c10d8bd2bf6
      https://github.com/llvm/llvm-project/commit/e05c10380ce7c18d1a232dcc5baa7c10d8bd2bf6
  Author: Jeremy Morse <jeremy.morse at sony.com>
  Date:   2021-02-08 (Mon, 08 Feb 2021)

  Changed paths:
    M llvm/test/DebugInfo/Generic/missing-abstract-variable.ll

  Log Message:
  -----------
  [DebugInfo] Re-engineer a test to be stricter, add XFails

In the LLVM-IR for this test, the inlined argument "b" in the "a" function
is optimized out on certain architectures, not on others. This hasn't been
reported as a test failure since 93faeecd8fa and ff2073a51 because we would
create a variable that looks like this:

  DW_TAG_formal_parameter
    DW_AT_abstract_origin

With no further information (and no location). With D95617 however, we
stop emitting such variables.

Prior to landing D95617: make this test stricter by checking that the
variable mentioned above has a location. We have to accept that on certain
architectures this goes missing, so add those to the XFail list.

I've run a few experiments, and right now it looks likely only powerpc64
still drops the variable location.