This broke several of the bots (eg). It seems any of the __has_cpp_attribute checks that have a namespace qualified attributes will need to be guarded by a if defined(__cplusplus) if we are including the header in a C file.

Wed, Aug 21, 11:05 AM

sfertile updated the diff for D66032: [AIX][MC][NFC] Explicitly set containing csects on XCOFF Symbols.

We were stretching the definition of an NFC patch by creating but not using the TOC-base on AIX. I've striped this down to the NFC part and will post a separate patch for the TOC base change that includes a test change to reflect the TOC-base being created.

Wed, Aug 21, 8:44 AM · Restricted Project

sfertile commandeered D66032: [AIX][MC][NFC] Explicitly set containing csects on XCOFF Symbols.

Commandeering this.

Wed, Aug 21, 8:38 AM · Restricted Project

Aug 20 2019

sfertile committed rG9467734a1c56: Fix assert in XCOFFObjectWriter related to program code csects. (authored by sfertile).

Fix assert in XCOFFObjectWriter related to program code csects.

Aug 20 2019, 4:25 PM

sfertile committed rL369471: Fix assert in XCOFFObjectWriter related to program code csects..

Fix assert in XCOFFObjectWriter related to program code csects.

Aug 20 2019, 4:23 PM

sfertile committed rG89463fcfc715: Remove assert with tautological compare from XCOFFObjectWriter. (authored by sfertile).

Remove assert with tautological compare from XCOFFObjectWriter.

Aug 20 2019, 3:24 PM

sfertile committed rL369462: Remove assert with tautological compare from XCOFFObjectWriter..

Remove assert with tautological compare from XCOFFObjectWriter.

Aug 20 2019, 3:24 PM

sfertile committed rG1e46d4cec535: Adds support for writing the .bss section for XCOFF object files. (authored by sfertile).

Adds support for writing the .bss section for XCOFF object files.

Aug 20 2019, 3:07 PM

sfertile committed rL369454: Adds support for writing the .bss section for XCOFF object files..

Adds support for writing the .bss section for XCOFF object files.

Aug 20 2019, 3:06 PM

sfertile closed D65159: [PowerPC][XCOFF] Adds support for writing the .bss section for object files..

Aug 20 2019, 3:06 PM · Restricted Project

sfertile added a comment to D65159: [PowerPC][XCOFF] Adds support for writing the .bss section for object files..

Related to the 32-bit vs 64-bit naming and assertions: I can appreciate being defensive about this, but I think trying to disable being able to create the various types and having an assertion (or error) in every function is overkill. Too much defensiveness is just clutter. I've added an earlier fatal_error in executePostlayoutBinding which blocks the ObjectWriter from doing anything interesting. When we are ready to proceed with 64-bit support we can remove that error and either rename the types (and guard all the appropriate functions), or modify the types for both 64-bit and 32-bit support, whichever is appropriate for the way we intend to add 64-bit support.

Aug 20 2019, 9:30 AM · Restricted Project

sfertile updated the diff for D65159: [PowerPC][XCOFF] Adds support for writing the .bss section for object files..

Addressed latest round of review comments.
Replaced 'CSect' in code with 'Csect' and changed errors/comments to use 'CSECT'.
Added a second fatal error related to emiting 64-bit objects, in executePostBindingLayout.

Aug 20 2019, 9:24 AM · Restricted Project

sfertile added inline comments to D65240: [XCOFF][AIX] Generate symbol table entries with llvm-readobj .

Aug 20 2019, 7:33 AM · Restricted Project

Aug 19 2019

sfertile updated the diff for D65159: [PowerPC][XCOFF] Adds support for writing the .bss section for object files..

Addressed most review comments other then those related to 32-bit vs 64-bit.

Aug 19 2019, 1:06 PM · Restricted Project

sfertile added inline comments to D65159: [PowerPC][XCOFF] Adds support for writing the .bss section for object files..

Aug 19 2019, 12:58 PM · Restricted Project

Aug 16 2019

sfertile added inline comments to D65159: [PowerPC][XCOFF] Adds support for writing the .bss section for object files..

Aug 16 2019, 1:41 PM · Restricted Project

sfertile updated the diff for D65159: [PowerPC][XCOFF] Adds support for writing the .bss section for object files..

Changed MCSymbolXCOFF StorageClass member to be an optional<StorageClass> and added asserts checking for redefineing the storage class and accessing an unset storage class.
Minor comment changes.

Aug 16 2019, 1:41 PM · Restricted Project

Aug 14 2019

sfertile added inline comments to D65159: [PowerPC][XCOFF] Adds support for writing the .bss section for object files..

Aug 14 2019, 1:44 PM · Restricted Project

sfertile updated the diff for D65159: [PowerPC][XCOFF] Adds support for writing the .bss section for object files..

Rebased and addressed numerous review comments.

Aug 14 2019, 1:44 PM · Restricted Project

Aug 13 2019

sfertile added inline comments to D65159: [PowerPC][XCOFF] Adds support for writing the .bss section for object files..

Aug 13 2019, 8:37 AM · Restricted Project

sfertile accepted D66032: [AIX][MC][NFC] Explicitly set containing csects on XCOFF Symbols.

LGTM. (After either responding to or addressing Jason comment)

Aug 13 2019, 7:07 AM · Restricted Project

Aug 12 2019

sfertile updated the diff for D65159: [PowerPC][XCOFF] Adds support for writing the .bss section for object files..

Rebased to reflect changes made in https://reviews.llvm.org/rL368584

Aug 12 2019, 9:49 AM · Restricted Project

sfertile added inline comments to D65240: [XCOFF][AIX] Generate symbol table entries with llvm-readobj .

Aug 12 2019, 8:48 AM · Restricted Project

sfertile committed rG29141da75ed9: [XCOFF] Use a single symbolic constant for the size of an embeded name. [NFC] (authored by sfertile).

[XCOFF] Use a single symbolic constant for the size of an embeded name. [NFC]

Aug 12 2019, 8:29 AM

sfertile committed rL368584: [XCOFF] Use a single symbolic constant for the size of an embeded name. [NFC].

[XCOFF] Use a single symbolic constant for the size of an embeded name. [NFC]

Aug 12 2019, 8:26 AM

sfertile added inline comments to D65240: [XCOFF][AIX] Generate symbol table entries with llvm-readobj .

Aug 12 2019, 7:19 AM · Restricted Project

Aug 9 2019

sfertile updated the diff for D65159: [PowerPC][XCOFF] Adds support for writing the .bss section for object files..

Rebased.

Aug 9 2019, 10:11 AM · Restricted Project

sfertile added inline comments to D65240: [XCOFF][AIX] Generate symbol table entries with llvm-readobj .

Aug 9 2019, 6:36 AM · Restricted Project

Aug 8 2019

sfertile accepted D64825: Enable assembly output of local commons for AIX.

LGTM.

Aug 8 2019, 7:02 AM · Restricted Project

Aug 7 2019

sfertile updated the diff for D65159: [PowerPC][XCOFF] Adds support for writing the .bss section for object files..

Rebased.

Aug 7 2019, 11:05 AM · Restricted Project

sfertile added inline comments to D64825: Enable assembly output of local commons for AIX.

Aug 7 2019, 11:00 AM · Restricted Project

Aug 6 2019

sfertile added inline comments to D65240: [XCOFF][AIX] Generate symbol table entries with llvm-readobj .

Aug 6 2019, 1:49 PM · Restricted Project

sfertile added inline comments to D64424: [AIX] Implement LR prolog/epilog save/restore .

Aug 6 2019, 10:28 AM · Restricted Project

sfertile accepted D65755: [ELF][PPC] Don't relax ifunc toc-indirect accesses to toc-relative.

LGTM.

Aug 6 2019, 9:48 AM · Restricted Project

sfertile created D65814: [PowerPC] Add combined ELF ABI and 32/64 bit queries to the Subtarget [NFC].

Aug 6 2019, 9:19 AM · Restricted Project

sfertile added inline comments to D65755: [ELF][PPC] Don't relax ifunc toc-indirect accesses to toc-relative.

Aug 6 2019, 8:57 AM · Restricted Project

Aug 1 2019

sfertile added a reviewer for D65159: [PowerPC][XCOFF] Adds support for writing the .bss section for object files.: stefanp.

Aug 1 2019, 7:44 AM · Restricted Project

sfertile added a comment to D65159: [PowerPC][XCOFF] Adds support for writing the .bss section for object files..

ping

Aug 1 2019, 7:44 AM · Restricted Project

Jul 30 2019

sfertile added a comment to D64652: [AIX] Add a TargetLoweringObjectFile for XCOFF and add support for common variables..

@hubert.reinterpretcast I've committed a patch that addresses your comments: https://reviews.llvm.org/rL367324.

Jul 30 2019, 9:08 AM · Restricted Project

sfertile committed rG39f35038140d: Address post commit review comments on revision 366727. (authored by sfertile).

Address post commit review comments on revision 366727.

Jul 30 2019, 8:40 AM

sfertile committed rL367324: Address post commit review comments on revision 366727..

Address post commit review comments on revision 366727.

Jul 30 2019, 8:37 AM

Jul 29 2019

sfertile updated subscribers of D65240: [XCOFF][AIX] Generate symbol table entries with llvm-readobj .

Jul 29 2019, 4:01 PM · Restricted Project

sfertile added inline comments to D64825: Enable assembly output of local commons for AIX.

Jul 29 2019, 12:09 PM · Restricted Project

sfertile added inline comments to D63547: [AIX]Global Address Lowering.

Jul 29 2019, 11:05 AM · Restricted Project

sfertile added inline comments to D65240: [XCOFF][AIX] Generate symbol table entries with llvm-readobj .

Jul 29 2019, 8:20 AM · Restricted Project

Jul 26 2019

sfertile accepted D63547: [AIX]Global Address Lowering.

A minor comment, but otherwise LGTM.

Jul 26 2019, 10:56 AM · Restricted Project

sfertile committed rG9df6177d380e: [PowerPC][AIX]Add lowering of MCSymbol MachineOperand. (authored by sfertile).

[PowerPC][AIX]Add lowering of MCSymbol MachineOperand.

Jul 26 2019, 10:26 AM

sfertile committed rL367133: [PowerPC][AIX]Add lowering of MCSymbol MachineOperand..

[PowerPC][AIX]Add lowering of MCSymbol MachineOperand.

Jul 26 2019, 10:25 AM

sfertile closed D63738: [AIX]Enable lowering of MCSymbol MachineOperand.

Jul 26 2019, 10:25 AM · Restricted Project

sfertile committed rG9bd22fec0d7b: [PowerPC] Add getCRSaveOffset to improve readability. [NFC] (authored by sfertile).

[PowerPC] Add getCRSaveOffset to improve readability. [NFC]

Jul 26 2019, 7:03 AM

sfertile committed rL367111: [PowerPC] Add getCRSaveOffset to improve readability. [NFC].

[PowerPC] Add getCRSaveOffset to improve readability. [NFC]

Jul 26 2019, 7:03 AM

Jul 24 2019

sfertile added a comment to D63738: [AIX]Enable lowering of MCSymbol MachineOperand.

We will need to add handling of Text SectionKind in TargetLoweringObjectFileXCOFF::getExplicitSectionGlobal in this because it blocks assembly generation for the test.

Jul 24 2019, 10:13 AM · Restricted Project

sfertile added a comment to D63547: [AIX]Global Address Lowering.

A couple minor comments, I think we are almost there.

Jul 24 2019, 9:48 AM · Restricted Project

sfertile added a comment to D64424: [AIX] Implement LR prolog/epilog save/restore .

I'll have to dig into the AIX ABI docs before I can review this more thoroughly but it looks good. I suggest we split out renaming of the hardcoded '8' to --> getCRSaveOffset() as a separate NFC patch and that we can land immediately.

Jul 24 2019, 8:13 AM · Restricted Project