hubert.reinterpretcast added a comment.

Tests require fixing.



================
Comment at: llvm/test/LTO/PowerPC/data-sections-aix.ll:18-19
+
+; CHECK-NOT:              (csect: .data)
+; CHECK:                  00000000 g O .data {{.*}} var
+
----------------
Unfortunately (without the change) this would pass even if the line was
```
00000000 g     O .data (csect: .data)   00000000 var
```



================
Comment at: llvm/test/LTO/PowerPC/data-sections-linux.ll:20
+
+; CHECK-NO-DATA-SECTIONS-NOT: .var
+; CHECK-NO-DATA-SECTIONS:     0000000000000000 g O .bss {{.*}} var
----------------
This is not needed. The whitespace before the `{{.*}}` will not match if 
`.bss.var` is encountered.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D129401/new/

https://reviews.llvm.org/D129401

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to