Page MenuHomePhabricator

sfertile (Sean Fertile)
User

Projects

User does not belong to any projects.

User Details

User Since
Oct 24 2016, 8:15 AM (151 w, 3 d)

Recent Activity

Thu, Sep 12

sfertile added inline comments to D66724: [AIX]Emit function descriptor csect in assembly.
Thu, Sep 12, 10:17 AM · Restricted Project
sfertile added inline comments to D67008: implement parsing relocation information for 32-bit xcoff objectfile.
Thu, Sep 12, 9:06 AM · Restricted Project
sfertile added inline comments to D66724: [AIX]Emit function descriptor csect in assembly.
Thu, Sep 12, 8:37 AM · Restricted Project

Wed, Sep 11

sfertile added inline comments to D67008: implement parsing relocation information for 32-bit xcoff objectfile.
Wed, Sep 11, 8:15 PM · Restricted Project
sfertile added inline comments to D66724: [AIX]Emit function descriptor csect in assembly.
Wed, Sep 11, 7:53 AM · Restricted Project
sfertile added a comment to D66724: [AIX]Emit function descriptor csect in assembly.
Wed, Sep 11, 7:40 AM · Restricted Project

Mon, Sep 9

sfertile added inline comments to D67008: implement parsing relocation information for 32-bit xcoff objectfile.
Mon, Sep 9, 10:00 AM · Restricted Project

Sat, Sep 7

sfertile added inline comments to D66724: [AIX]Emit function descriptor csect in assembly.
Sat, Sep 7, 8:37 AM · Restricted Project
sfertile added inline comments to D66724: [AIX]Emit function descriptor csect in assembly.
Sat, Sep 7, 8:19 AM · Restricted Project

Fri, Sep 6

sfertile added inline comments to D66969: Output XCOFF object text section.
Fri, Sep 6, 1:07 PM · Restricted Project
sfertile committed rGeaf34a983c71: [PowerPC][XCOFF] Remove basic test. [NFC] (authored by sfertile).
[PowerPC][XCOFF] Remove basic test. [NFC]
Fri, Sep 6, 12:59 PM
sfertile committed rL371247: [PowerPC][XCOFF] Remove basic test. [NFC].
[PowerPC][XCOFF] Remove basic test. [NFC]
Fri, Sep 6, 12:59 PM
sfertile added inline comments to D66969: Output XCOFF object text section.
Fri, Sep 6, 12:52 PM · Restricted Project
sfertile committed rG74966aca3559: [PowerPC][XCOFF] Verify symbol table in xcoff object files. [NFC] (authored by sfertile).
[PowerPC][XCOFF] Verify symbol table in xcoff object files. [NFC]
Fri, Sep 6, 11:56 AM
sfertile committed rL371237: [PowerPC][XCOFF] Verify symbol table in xcoff object files. [NFC].
[PowerPC][XCOFF] Verify symbol table in xcoff object files. [NFC]
Fri, Sep 6, 11:55 AM
sfertile closed D66944: [PowerPC][XCOFF] Verify symbol table in xcoff object files. [NFC].
Fri, Sep 6, 11:54 AM · Restricted Project

Wed, Sep 4

sfertile added inline comments to D67125: [PowerPC][AIX] Adds support for writing the data section in object files.
Wed, Sep 4, 6:34 PM · Restricted Project

Tue, Sep 3

sfertile added inline comments to D66969: Output XCOFF object text section.
Tue, Sep 3, 9:06 AM · Restricted Project
sfertile added inline comments to D66944: [PowerPC][XCOFF] Verify symbol table in xcoff object files. [NFC].
Tue, Sep 3, 8:03 AM · Restricted Project
sfertile updated the diff for D66944: [PowerPC][XCOFF] Verify symbol table in xcoff object files. [NFC].

Used numeric substitution block for symbol table indexes.

Tue, Sep 3, 7:51 AM · Restricted Project
sfertile added inline comments to D66944: [PowerPC][XCOFF] Verify symbol table in xcoff object files. [NFC].
Tue, Sep 3, 7:31 AM · Restricted Project

Thu, Aug 29

sfertile created D66944: [PowerPC][XCOFF] Verify symbol table in xcoff object files. [NFC].
Thu, Aug 29, 7:27 AM · Restricted Project

Tue, Aug 27

sfertile added inline comments to D66724: [AIX]Emit function descriptor csect in assembly.
Tue, Aug 27, 6:53 AM · Restricted Project

Mon, Aug 26

sfertile added inline comments to D66724: [AIX]Emit function descriptor csect in assembly.
Mon, Aug 26, 7:28 AM · Restricted Project
sfertile accepted D66097: Handle local commons for XCOFF object file writing.

LGTM.

Mon, Aug 26, 6:37 AM · Restricted Project

Fri, Aug 23

sfertile added a comment to D66609: Allow Compiler.h to be included in C files and fix fallthrough warnings.

Can we please either commit this, or revert r369414 otherwise? Several of the sanitizer buildbots have been red since Tuesday due to r369414.

Fri, Aug 23, 7:47 AM · Restricted Project

Thu, Aug 22

sfertile committed rG5f85a7b1cfe9: [PowerPC] Add combined ELF ABI and 32/64 bit queries to the subtarget. [NFC] (authored by sfertile).
[PowerPC] Add combined ELF ABI and 32/64 bit queries to the subtarget. [NFC]
Thu, Aug 22, 8:14 AM
sfertile committed rG18fd1b0b494a: [PowerPC][XCOFF][MC] Explicitly set containing csect on symbols. [NFC] (authored by sfertile).
[PowerPC][XCOFF][MC] Explicitly set containing csect on symbols. [NFC]
Thu, Aug 22, 8:14 AM
sfertile committed rL369658: [PowerPC] Add combined ELF ABI and 32/64 bit queries to the subtarget. [NFC].
[PowerPC] Add combined ELF ABI and 32/64 bit queries to the subtarget. [NFC]
Thu, Aug 22, 8:14 AM
sfertile closed D65814: [PowerPC] Add combined ELF ABI and 32/64 bit queries to the Subtarget [NFC].
Thu, Aug 22, 8:14 AM · Restricted Project
sfertile committed rL369657: [PowerPC][XCOFF][MC] Explicitly set containing csect on symbols. [NFC].
[PowerPC][XCOFF][MC] Explicitly set containing csect on symbols. [NFC]
Thu, Aug 22, 8:14 AM
sfertile closed D66032: [AIX][MC][NFC] Explicitly set containing csects on XCOFF Symbols.
Thu, Aug 22, 8:14 AM · Restricted Project

Wed, Aug 21

sfertile added a comment to rL369549: Fix -Wimplicit-fallthrough warnings in regcomp.c.

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

Jul 23 2019

sfertile created D65159: [PowerPC][XCOFF] Adds support for writing the .bss section for object files..
Jul 23 2019, 11:08 AM · Restricted Project

Jul 22 2019

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

Committed in https://reviews.llvm.org/rG942537d9fa50

Jul 22 2019, 12:23 PM · Restricted Project
sfertile committed rG942537d9fa50: Stubs out TLOF for AIX and add support for common vars in assembly output. (authored by sfertile).
Stubs out TLOF for AIX and add support for common vars in assembly output.
Jul 22 2019, 12:19 PM
sfertile committed rL366727: Stubs out TLOF for AIX and add support for common vars in assembly output..
Stubs out TLOF for AIX and add support for common vars in assembly output.
Jul 22 2019, 12:15 PM
sfertile committed rG324d33dd4ed6: [PowerPC] Fix comment on MO_PLT Target Operand Flag. [NFC] (authored by sfertile).
[PowerPC] Fix comment on MO_PLT Target Operand Flag. [NFC]
Jul 22 2019, 11:49 AM
sfertile committed rG8034daca5f47: [Object][XCOFF] Remove extra includes from XCOFF related files. [NFC] (authored by sfertile).
[Object][XCOFF] Remove extra includes from XCOFF related files. [NFC]
Jul 22 2019, 11:49 AM
sfertile committed rL366724: [PowerPC] Fix comment on MO_PLT Target Operand Flag. [NFC].
[PowerPC] Fix comment on MO_PLT Target Operand Flag. [NFC]
Jul 22 2019, 11:49 AM
sfertile committed rL366723: [Object][XCOFF] Remove extra includes from XCOFF related files. [NFC].
[Object][XCOFF] Remove extra includes from XCOFF related files. [NFC]
Jul 22 2019, 11:49 AM
sfertile closed D60885: [Object][XCOFF] Remove extra includes from XCOFF related files. [NFC].
Jul 22 2019, 11:48 AM · Restricted Project
sfertile accepted D64814: [NFC][PowerPC]Change ADDIStocHA to ADDIStocHA8 to follow 64-bit naming convention.

LGTM.

Jul 22 2019, 8:05 AM · Restricted Project

Jul 16 2019

sfertile added inline comments to D64814: [NFC][PowerPC]Change ADDIStocHA to ADDIStocHA8 to follow 64-bit naming convention.
Jul 16 2019, 2:12 PM · Restricted Project
sfertile updated subscribers of D64814: [NFC][PowerPC]Change ADDIStocHA to ADDIStocHA8 to follow 64-bit naming convention.
Jul 16 2019, 2:03 PM · Restricted Project
sfertile added inline comments to D63547: [AIX]Global Address Lowering.
Jul 16 2019, 2:03 PM · Restricted Project

Jul 12 2019

sfertile added inline comments to D63547: [AIX]Global Address Lowering.
Jul 12 2019, 12:07 PM · Restricted Project