[llvm] [DWP] Fix default for continue-on-cu-index-overflow (PR #75540)

David Blaikie via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 18 11:21:34 PST 2023


================
@@ -3,6 +3,14 @@ RUN: llvm-dwarfdump -v %t | FileCheck --check-prefixes=CHECK,NOTYP %s
 RUN: llvm-objdump -h %t | FileCheck --check-prefix=NOTYPOBJ %s
 RUN: llvm-dwp %p/../Inputs/simple/types/a.dwo %p/../Inputs/simple/types/b.dwo -o - \
 RUN:   | llvm-dwarfdump -v - | FileCheck --check-prefixes=CHECK,TYPES %s
+RUN: llvm-dwp %p/../Inputs/simple/notypes/a.dwo %p/../Inputs/simple/notypes/b.dwo -o %t     --continue-on-cu-index-overflow
+RUN: llvm-dwp %p/../Inputs/simple/notypes/a.dwo %p/../Inputs/simple/notypes/b.dwo -o %t     --continue-on-cu-index-overflow=continue
+RUN: llvm-dwp %p/../Inputs/simple/notypes/a.dwo %p/../Inputs/simple/notypes/b.dwo -o %t     --continue-on-cu-index-overflow=soft-stop
+RUN: not llvm-dwp %p/../Inputs/simple/notypes/a.dwo %p/../Inputs/simple/notypes/b.dwo -o %t --continue-on-cu-index-overflow=foobar
+RUN: llvm-dwp %p/../Inputs/simple/notypes/a.dwo %p/../Inputs/simple/notypes/b.dwo -o %t     -continue-on-cu-index-overflow
+RUN: llvm-dwp %p/../Inputs/simple/notypes/a.dwo %p/../Inputs/simple/notypes/b.dwo -o %t     -continue-on-cu-index-overflow=continue
+RUN: llvm-dwp %p/../Inputs/simple/notypes/a.dwo %p/../Inputs/simple/notypes/b.dwo -o %t     -continue-on-cu-index-overflow=soft-stop
+RUN: not llvm-dwp %p/../Inputs/simple/notypes/a.dwo %p/../Inputs/simple/notypes/b.dwo -o %t -continue-on-cu-index-overflow=foobar
----------------
dwblaikie wrote:

does this check the behavior? Looks like it checks if the `llvm-dwp` invocation succeeds or fails, but nothing about what it does. So it's unclear/untested that, for instance, `--continue-on-cu-index-overflow` does the `continue` thing, and not something else successful?

Could you add these checks wherever the `--continue-on-cu-index-overflow` behavior is tested, and test them as different phrasings of the same test (eg: there's some test that checks the continue behavior, and it could have two RUN lines, one with `--continue-on-cu-index-overflow` and one with `--continue-on-cu-index-overflow=continue` for instance)

https://github.com/llvm/llvm-project/pull/75540


More information about the llvm-commits mailing list