diff mbox series

Skip testsuite/g++.old-deja/g++.pt/const2.C on vxworks_kernel

Message ID orczz1h5za.fsf@lxoliva.fsfla.org
State New
Headers show
Series Skip testsuite/g++.old-deja/g++.pt/const2.C on vxworks_kernel | expand

Commit Message

Alexandre Oliva Dec. 22, 2020, 9:43 p.m. UTC
Linking in vxworks kernel-mode is partial linking, so missing symbols
are not detected.

Regstrapped on x86_64-linux-gnu, and tested with -x-arm-wrs-vxworks7r2.
Ok to install?


from Jerome Lambourg <lambourg@adacore.com>
for  gcc/testsuite/ChangeLog

        * g++.old-deja/g++.pt/const2.C: Skip on vxworks kernel.
---
 gcc/testsuite/g++.old-deja/g++.pt/const2.C |    1 +
 1 file changed, 1 insertion(+)

Comments

Mike Stump Dec. 29, 2020, 6:38 p.m. UTC | #1
On Dec 22, 2020, at 1:43 PM, Alexandre Oliva <oliva@adacore.com> wrote:
> 
> Linking in vxworks kernel-mode is partial linking, so missing symbols
> are not detected.
> 
> Regstrapped on x86_64-linux-gnu, and tested with -x-arm-wrs-vxworks7r2.
> Ok to install?

Ok.

Generally nicer to bunch all like ones ("partial link" for example) together.
diff mbox series

Patch

diff --git a/gcc/testsuite/g++.old-deja/g++.pt/const2.C b/gcc/testsuite/g++.old-deja/g++.pt/const2.C
index 3ca4a532b1987..7340ff4e982ce 100644
--- a/gcc/testsuite/g++.old-deja/g++.pt/const2.C
+++ b/gcc/testsuite/g++.old-deja/g++.pt/const2.C
@@ -1,4 +1,5 @@ 
 // { dg-do link }
+// { dg-skip-if "VxWorks dkms use partial link" { vxworks_kernel } }
 // This test should get a linker error for the reference to A<int>::i.
 // { dg-prune-output "ld: symbol" }
 // { dg-message "i" "" { target *-*-* } 0 }