[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