Page MenuHomePhabricator

aguinet (Adrien Guinet)
User

Projects

User does not belong to any projects.

User Details

User Since
Jun 23 2016, 2:29 PM (232 w, 1 d)

Recent Activity

Thu, Dec 3

aguinet accepted D92180: Speedup some unicode rendering.
Thu, Dec 3, 6:37 AM · Restricted Project

Wed, Dec 2

aguinet accepted D92180: Speedup some unicode rendering.
Wed, Dec 2, 6:55 AM · Restricted Project

Sun, Nov 29

aguinet updated the diff for D89490: Introduce __attribute__((darwin_abi)).

Relax checks in CodeGenCXX/darwinabi-returnthis.cpp Clang test (to adapt to new attributes), and removes some useless brackets in if statements.

Sun, Nov 29, 1:23 AM · Restricted Project, Restricted Project
aguinet added inline comments to D89490: Introduce __attribute__((darwin_abi)).
Sun, Nov 29, 12:56 AM · Restricted Project, Restricted Project
aguinet updated the diff for D89490: Introduce __attribute__((darwin_abi)).
Sun, Nov 29, 12:56 AM · Restricted Project, Restricted Project

Sat, Nov 28

aguinet added inline comments to D92180: Speedup some unicode rendering.
Sat, Nov 28, 11:08 AM · Restricted Project

Oct 16 2020

aguinet updated the diff for D89490: Introduce __attribute__((darwin_abi)).

One missing formatting case...

Oct 16 2020, 2:21 AM · Restricted Project, Restricted Project
aguinet updated the diff for D89490: Introduce __attribute__((darwin_abi)).

Add some clang-format tags, and restore back the "original" formatting for these cases

Oct 16 2020, 2:18 AM · Restricted Project, Restricted Project

Oct 15 2020

aguinet added inline comments to D89490: Introduce __attribute__((darwin_abi)).
Oct 15 2020, 11:38 PM · Restricted Project, Restricted Project
aguinet updated the diff for D89490: Introduce __attribute__((darwin_abi)).

Fix one clang-tidy warning

Oct 15 2020, 11:30 PM · Restricted Project, Restricted Project
aguinet added inline comments to D89490: Introduce __attribute__((darwin_abi)).
Oct 15 2020, 11:22 PM · Restricted Project, Restricted Project
aguinet updated the diff for D89490: Introduce __attribute__((darwin_abi)).

Update format + llvm tests

Oct 15 2020, 2:04 PM · Restricted Project, Restricted Project
aguinet added a comment to D89490: Introduce __attribute__((darwin_abi)).

I see that the llvm side of the patch lacks tests?

Oct 15 2020, 1:29 PM · Restricted Project, Restricted Project
aguinet requested review of D89490: Introduce __attribute__((darwin_abi)).
Oct 15 2020, 12:09 PM · Restricted Project, Restricted Project

Aug 25 2020

aguinet added a comment to D85740: Universal MachO: support LLVM IR objects.

Thanks for the merge and reviews @alexshap (and @smeenai) !

Aug 25 2020, 11:23 PM · Restricted Project
aguinet updated the diff for D85740: Universal MachO: support LLVM IR objects.

This patch fixes one bug that was present: running llvm-lipo -archs on an LLVM IR object would crash, because we didn't check for that case in printBinaryArchs. This new patch fixes it, with an associated test.

Aug 25 2020, 7:36 AM · Restricted Project
aguinet added a comment to D85740: Universal MachO: support LLVM IR objects.

@alexshap note that I don't have RW access to the llvm repository, so I can't merge this myself

Aug 25 2020, 12:19 AM · Restricted Project
aguinet updated the diff for D85740: Universal MachO: support LLVM IR objects.
Aug 25 2020, 12:15 AM · Restricted Project
aguinet added a comment to D85740: Universal MachO: support LLVM IR objects.

Thanks for the new reviews! Fixed in the incoming diff

Aug 25 2020, 12:15 AM · Restricted Project
aguinet updated the diff for D85740: Universal MachO: support LLVM IR objects.

Fresh air!

Aug 25 2020, 12:12 AM · Restricted Project

Aug 24 2020

aguinet updated the diff for D85740: Universal MachO: support LLVM IR objects.

Just a small variable naming cleanup

Aug 24 2020, 5:12 AM · Restricted Project
aguinet updated the diff for D85740: Universal MachO: support LLVM IR objects.

@alexshap there we go, it should fix all your remarks. Please let my know if there are still some issues.

Aug 24 2020, 5:08 AM · Restricted Project
aguinet added a comment to D85740: Universal MachO: support LLVM IR objects.

Thanks @alexshap for your review and comments! Answers are inline.

Aug 24 2020, 4:13 AM · Restricted Project

Aug 22 2020

aguinet updated the diff for D85740: Universal MachO: support LLVM IR objects.

Cleanup/refactoring

Aug 22 2020, 10:02 AM · Restricted Project

Aug 17 2020

aguinet updated the diff for D85740: Universal MachO: support LLVM IR objects.

Clang-tidy + format remarks

Aug 17 2020, 6:26 AM · Restricted Project
aguinet updated the diff for D85740: Universal MachO: support LLVM IR objects.

Fix fatal error message of MachOUniversalBinary::ObjectForArch::getAsIRObject

Aug 17 2020, 4:39 AM · Restricted Project
aguinet updated the diff for D85740: Universal MachO: support LLVM IR objects.

Here is a new patch that includes:

Aug 17 2020, 4:18 AM · Restricted Project

Aug 16 2020

aguinet updated the diff for D67258: Out-of-tree machine IR passes.
  • Update to monorepo and master
  • Add forgotten RegisterMIRPasses.h
  • Compile and test even in static mode
Aug 16 2020, 2:58 AM · Restricted Project

Aug 14 2020

aguinet added a comment to D85740: Universal MachO: support LLVM IR objects.

About that part, like I said in the description, the way I see it is to enhance the ObjectYAML library so that we can generate LLVM bitcode file format (as described in https://llvm.org/docs/BitCodeFormat.html), so that yaml2obj can generate appropriate LLVM IR test objects. That would be used for instance to test the -create flag. Do you see another way to do this?

Can't you just use llvm-as to produce bitcode files and feed those to llvm-lipo?

Aug 14 2020, 7:19 AM · Restricted Project

Aug 12 2020

aguinet updated the diff for D85740: Universal MachO: support LLVM IR objects.
Aug 12 2020, 1:48 AM · Restricted Project
aguinet added a comment to D85740: Universal MachO: support LLVM IR objects.

Thanks for the comments. Answers below:

Aug 12 2020, 1:46 AM · Restricted Project

Aug 11 2020

aguinet updated the summary of D85740: Universal MachO: support LLVM IR objects.
Aug 11 2020, 9:46 AM · Restricted Project
aguinet updated the summary of D85740: Universal MachO: support LLVM IR objects.
Aug 11 2020, 8:46 AM · Restricted Project
aguinet requested review of D85740: Universal MachO: support LLVM IR objects.
Aug 11 2020, 8:42 AM · Restricted Project

Dec 13 2019

aguinet added inline comments to D67258: Out-of-tree machine IR passes.
Dec 13 2019, 1:49 AM · Restricted Project

Nov 26 2019

aguinet updated the diff for D67258: Out-of-tree machine IR passes.

Use BUILDTREE_ONLY, as suggested by @serge-sans-paille .

Nov 26 2019, 12:06 AM · Restricted Project

Nov 25 2019

aguinet added inline comments to D67258: Out-of-tree machine IR passes.
Nov 25 2019, 5:26 AM · Restricted Project
aguinet added inline comments to D67258: Out-of-tree machine IR passes.
Nov 25 2019, 5:26 AM · Restricted Project

Nov 21 2019

aguinet updated the diff for D67258: Out-of-tree machine IR passes.

Copyright added on new cpp files.

Nov 21 2019, 12:16 AM · Restricted Project

Nov 20 2019

aguinet updated the diff for D67258: Out-of-tree machine IR passes.

Cleaning up some commented CMake code...

Nov 20 2019, 11:35 PM · Restricted Project
aguinet updated the diff for D67258: Out-of-tree machine IR passes.

Thanks Serge for the comments !

Nov 20 2019, 11:25 PM · Restricted Project

Sep 5 2019

aguinet added a comment to D67258: Out-of-tree machine IR passes.

As a side remark, I have tested this with an out of tree MIR pass that dump MIR, and it works well. Should I add this test within the test suite? I didn't find such test for the current out-of-tree IR system (but chances are I missed it!).

Sep 5 2019, 11:41 PM · Restricted Project
aguinet created D67258: Out-of-tree machine IR passes.
Sep 5 2019, 11:40 PM · Restricted Project

Mar 21 2018

aguinet added a comment to D42350: Add constructor DWARF calling convention for every supported LLVM CC.

BTW, I don't have any commit rights thought, so I guess someone needs to merge this at some point?

Mar 21 2018, 3:30 PM
aguinet added a comment to D42350: Add constructor DWARF calling convention for every supported LLVM CC.

Thanks everyone for the review and comments!

Mar 21 2018, 3:11 PM

Feb 10 2018

aguinet updated the diff for D42351: Emit DWARF "constructor" calling convention for every supported Clang CC.

Test case added!

Feb 10 2018, 2:30 AM

Jan 31 2018

aguinet added a comment to D42350: Add constructor DWARF calling convention for every supported LLVM CC.

I'll try to do the same with icc by tring the various ABI it supports (https://software.intel.com/en-us/node/522787), and getting the potential value with readelf (all with a script to be able to run this again!)

Jan 31 2018, 11:18 PM

Jan 22 2018

aguinet added a comment to D42350: Add constructor DWARF calling convention for every supported LLVM CC.

By the way, if you can contribute patches that add more 3rd party DWARF extensions to LLVM, that would be awesome to help avoiding clashes in the future.

Jan 22 2018, 11:18 PM
aguinet updated the diff for D42350: Add constructor DWARF calling convention for every supported LLVM CC.
Jan 22 2018, 11:13 PM
aguinet added a comment to D42350: Add constructor DWARF calling convention for every supported LLVM CC.

I suppose the namespace for DW_CCs is large enough that doing this isn't very risky. Have you double-checked that this doesn't overlap with any other vendor extensions?

Jan 22 2018, 9:47 AM

Jan 21 2018

aguinet created D42351: Emit DWARF "constructor" calling convention for every supported Clang CC.
Jan 21 2018, 3:50 AM
aguinet created D42350: Add constructor DWARF calling convention for every supported LLVM CC.
Jan 21 2018, 3:49 AM
aguinet added a comment to D40644: Fix LLVM_ENABLE_PIC under Unix/Linux.

Hello Chill,

Jan 21 2018, 3:42 AM
aguinet updated subscribers of D40644: Fix LLVM_ENABLE_PIC under Unix/Linux.
Jan 21 2018, 3:35 AM

Nov 30 2017

aguinet created D40644: Fix LLVM_ENABLE_PIC under Unix/Linux.
Nov 30 2017, 2:46 AM

Jun 23 2016

aguinet retitled D21664: Fix Thumb text sections' flags under COFF/Windows (2/2) from to Fix Thumb text sections' flags under COFF/Windows (2/2).
Jun 23 2016, 3:14 PM
aguinet retitled D21662: Fix Thumb text sections' flags under COFF/Windows (1/2) from to Fix Thumb text sections' flags under COFF/Windows (1/2).
Jun 23 2016, 3:11 PM