On Fri, Dec 4, 2020 at 5:35 AM Rainer Orth <r...@cebitec.uni-bielefeld.de> 
wrote:

> On AIX 7.2, there are changes like
>
> -PASS: g++.dg/gomp/tls-5.C  -std=c++2a  scan-assembler-symbol-section symbol 
> ^_?ir$ (found ir) has section ^\\.tbss|\\[TL\\] (found _tls5.tls_[TL],4)
> +PASS: g++.dg/gomp/tls-5.C  -std=c++2a  scan-assembler-symbol-section symbol 
> ^_?ir$ (found ir) has section ^\\.tbss|\\[TL\\] (found _tls5.tls_[TL])
>
> i.e. the ",4" after (?) the section name is now stripped.  I believe
> this is benign: David?

The ",4" is the symbol alignment.  It is not necessary for the purpose
of the tests.

Thanks for looking further into this problem.  As I mentioned in my
earlier reply to the patch itself, I believe that this new feature and
infrastructure change should have been tested and fixed on
non-Linux/ELF/x86 architectures, not left as an exercise for the
maintainers of other targets.  A patch that introduces regressions in
the testsuite should be fixed or reverted and should be the
responsibility of the author -- whether the change is to the compiler
or to the testsuite.

Thanks, David

Reply via email to