[clang-tools-extra] [clang] [llvm] [XCOFF] Add compiler version to an auxiliary symbol table entry (PR #80162)

2024-02-06 Thread Anton Korobeynikov via cfe-commits


@@ -159,14 +159,26 @@ declare i32 @bar(i32)
 ; SYM:  Symbols [
 ; SYM-NEXT:   Symbol {
 ; SYM-NEXT: Index: 0
-; SYM-NEXT: Name: 
+; SYM-NEXT: Name: .file
 ; SYM-NEXT: Value (SymbolTableIndex): 0x0
 ; SYM-NEXT: Section: N_DEBUG
 ; SYM-NEXT: Source Language ID: TB_CPLUSPLUS (0x9)
 ; SYM32-NEXT:   CPU Version ID: TCPU_COM (0x3)
 ; SYM64-NEXT:   CPU Version ID: TCPU_PPC64 (0x2)
 ; SYM-NEXT: StorageClass: C_FILE (0x67)
-; SYM-NEXT: NumberOfAuxEntries: 0
+; SYM-NEXT: NumberOfAuxEntries: 2
+; SYM-NEXT: File Auxiliary Entry {
+; SYM-NEXT:   Index: 1
+; SYM-NEXT:   Name: 
+; SYM-NEXT:   Type: XFT_FN (0x0)
+; SYM64-NEXT: Auxiliary Type: AUX_FILE (0xFC)
+; SYM-NEXT: }
+; SYM-NEXT: File Auxiliary Entry {
+; SYM-NEXT:   Index: 2
+; SYM-NEXT:   Name: LLVM version 18.0.0git

asl wrote:

This should not be here.

https://github.com/llvm/llvm-project/pull/80162
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits


[clang-tools-extra] [clang] [llvm] [XCOFF] Add compiler version to an auxiliary symbol table entry (PR #80162)

2024-02-06 Thread Anton Korobeynikov via cfe-commits

asl wrote:

@nico How about revert?

https://github.com/llvm/llvm-project/pull/80162
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits


[clang-tools-extra] [clang] [llvm] [XCOFF] Add compiler version to an auxiliary symbol table entry (PR #80162)

2024-02-06 Thread Anton Korobeynikov via cfe-commits

asl wrote:

@stephenpeckham Apparently this was merged without waiting for buildkite 
results or ignoring its results. Linux build failed due to obvious spelling 
mistakes. Will you please next time wait until CI to finish?

Thanks!

https://github.com/llvm/llvm-project/pull/80162
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits


[clang-tools-extra] [clang] [llvm] [XCOFF] Add compiler version to an auxiliary symbol table entry (PR #80162)

2024-02-06 Thread Nico Weber via cfe-commits

nico wrote:

This seems to break tests, even after the spelling fix: 
http://45.33.8.238/linux/130117/step_12.txt

Does that look related?

https://github.com/llvm/llvm-project/pull/80162
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits


[clang-tools-extra] [clang] [llvm] [XCOFF] Add compiler version to an auxiliary symbol table entry (PR #80162)

2024-02-06 Thread Jan Patrick Lehr via cfe-commits

jplehr wrote:

This introduced a spelling mistake that broke some builds.
https://lab.llvm.org/buildbot/#/builders/193/builds/46220

https://github.com/llvm/llvm-project/pull/80162
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits