Page MenuHomePhabricator

shenhan (Han Shen)
User

Projects

User does not belong to any projects.

User Details

User Since
Jan 5 2016, 3:55 PM (240 w, 2 d)

Recent Activity

Mon, Jul 20

shenhan added inline comments to D83885: [Propeller]: Use a descriptive temporary symbol name for the end of the basic block..
Mon, Jul 20, 10:07 AM · Restricted Project

May 5 2020

shenhan added inline comments to D68049: Propeller: Clang options for basic block sections .
May 5 2020, 5:17 PM · Restricted Project, Restricted Project

Feb 18 2020

shenhan added a comment to D73497: lld: Propeller framework part I.

Hi, would you to take another look? Thanks, -Han

Feb 18 2020, 10:07 AM · Restricted Project

Feb 7 2020

shenhan updated the diff for D73497: lld: Propeller framework part I.

Thanks Rui. Uploaded the patch: 1) removed PropellerConfig 2). Removed PropellerCommon.h from patch (and moved it into llvm realm).

Feb 7 2020, 5:26 PM · Restricted Project

Feb 6 2020

shenhan updated the diff for D73497: lld: Propeller framework part I.
Feb 6 2020, 3:02 PM · Restricted Project

Feb 4 2020

shenhan updated the diff for D73497: lld: Propeller framework part I.
Feb 4 2020, 4:22 PM · Restricted Project
shenhan updated the diff for D73497: lld: Propeller framework part I.

Fixed all variable names to comply with lld naming convention.

Feb 4 2020, 10:30 AM · Restricted Project

Jan 31 2020

shenhan added inline comments to D73497: lld: Propeller framework part I.
Jan 31 2020, 5:11 PM · Restricted Project
shenhan updated the diff for D73497: lld: Propeller framework part I.

Hi Markray, thanks for the review.

Jan 31 2020, 5:11 PM · Restricted Project

Jan 30 2020

shenhan updated the diff for D73497: lld: Propeller framework part I.
Jan 30 2020, 2:55 PM · Restricted Project

Jan 27 2020

shenhan retitled D73497: lld: Propeller framework part I from Propeller to lld: Propeller framework part I.
Jan 27 2020, 12:24 PM · Restricted Project
shenhan created D73497: lld: Propeller framework part I.
Jan 27 2020, 12:24 PM · Restricted Project

Jan 17 2020

shenhan added a comment to D68062: Propeller lld framework for basicblock sections.

Hello, over the past 2 months since our first draft of Propeller, we've made huge changes/improvements and this describes what happens -http://lists.llvm.org/pipermail/llvm-dev/2020-January/138426.html

Jan 17 2020, 4:55 PM · Restricted Project

Jan 16 2020

shenhan updated the diff for D68062: Propeller lld framework for basicblock sections.
Jan 16 2020, 4:31 PM · Restricted Project

Nov 21 2019

shenhan updated the diff for D68062: Propeller lld framework for basicblock sections.

Added lld/ELF/Propeller/README.md. Moved some of file comments into README.md.

Nov 21 2019, 5:32 PM · Restricted Project

Nov 12 2019

shenhan updated the diff for D68062: Propeller lld framework for basicblock sections.

Hi, we've refactored propeller and cleaned up dependencies and interfaces. Here are the change summaries:

Nov 12 2019, 12:19 PM · Restricted Project

Oct 8 2019

shenhan added inline comments to D68062: Propeller lld framework for basicblock sections.
Oct 8 2019, 12:12 PM · Restricted Project
shenhan updated the diff for D68062: Propeller lld framework for basicblock sections.
Oct 8 2019, 12:12 PM · Restricted Project

Oct 4 2019

shenhan added inline comments to D68062: Propeller lld framework for basicblock sections.
Oct 4 2019, 11:29 AM · Restricted Project
shenhan updated the diff for D68062: Propeller lld framework for basicblock sections.
Oct 4 2019, 11:29 AM · Restricted Project

Oct 3 2019

shenhan added inline comments to D68062: Propeller lld framework for basicblock sections.
Oct 3 2019, 5:02 PM · Restricted Project
shenhan updated the diff for D68062: Propeller lld framework for basicblock sections.
Oct 3 2019, 5:02 PM · Restricted Project

Oct 2 2019

shenhan updated the diff for D68062: Propeller lld framework for basicblock sections.
Oct 2 2019, 6:18 PM · Restricted Project

Oct 1 2019

shenhan added inline comments to D68062: Propeller lld framework for basicblock sections.
Oct 1 2019, 5:13 PM · Restricted Project
shenhan updated the diff for D68062: Propeller lld framework for basicblock sections.
Oct 1 2019, 5:10 PM · Restricted Project

Sep 30 2019

shenhan added inline comments to D68062: Propeller lld framework for basicblock sections.
Sep 30 2019, 5:28 PM · Restricted Project
shenhan updated the diff for D68062: Propeller lld framework for basicblock sections.
Sep 30 2019, 5:26 PM · Restricted Project

Sep 27 2019

shenhan added inline comments to D68062: Propeller lld framework for basicblock sections.
Sep 27 2019, 5:27 PM · Restricted Project
shenhan updated the diff for D68062: Propeller lld framework for basicblock sections.

Updated header file description. Also changed local variable names to lower case camel.

Sep 27 2019, 5:21 PM · Restricted Project

Sep 26 2019

shenhan updated the diff for D68062: Propeller lld framework for basicblock sections.
Sep 26 2019, 5:52 PM · Restricted Project
shenhan added a comment to D68062: Propeller lld framework for basicblock sections.

Let me start from very high-level review comments before getting into the details of the actual code:

  • First and foremost, can you add a file comments to each file to explain what you are doing in the file? It doesn't have to be detailed, but giving an overview at the beginning of a file is generally very useful.
Sep 26 2019, 5:06 PM · Restricted Project
shenhan updated the diff for D68062: Propeller lld framework for basicblock sections.

Addressed maskray's comments.

Sep 26 2019, 4:58 PM · Restricted Project

Sep 25 2019

shenhan updated subscribers of D68062: Propeller lld framework for basicblock sections.
Sep 25 2019, 5:08 PM · Restricted Project
shenhan created D68062: Propeller lld framework for basicblock sections.
Sep 25 2019, 5:02 PM · Restricted Project

Jun 6 2018

shenhan committed rL334155: ClangTidy fix - 'clang::Sema::checkAllowedCUDAInitializer' has a definition….
ClangTidy fix - 'clang::Sema::checkAllowedCUDAInitializer' has a definition…
Jun 6 2018, 6:00 PM
shenhan committed rC334155: ClangTidy fix - 'clang::Sema::checkAllowedCUDAInitializer' has a definition….
ClangTidy fix - 'clang::Sema::checkAllowedCUDAInitializer' has a definition…
Jun 6 2018, 6:00 PM
shenhan committed rLLD334135: Correct aligment computation for shared object symbols..
Correct aligment computation for shared object symbols.
Jun 6 2018, 2:48 PM
shenhan committed rL334135: Correct aligment computation for shared object symbols..
Correct aligment computation for shared object symbols.
Jun 6 2018, 2:48 PM
shenhan closed D47602: Correct aligment computation for shared object symbols.
Jun 6 2018, 2:48 PM
shenhan committed rL334114: Fix the test case that places intermediate in source directory..
Fix the test case that places intermediate in source directory.
Jun 6 2018, 12:00 PM
shenhan closed D47839: Fix the test case that places intermediate in source directory.
Jun 6 2018, 11:59 AM
shenhan created D47839: Fix the test case that places intermediate in source directory.
Jun 6 2018, 11:45 AM
shenhan added a comment to D47602: Correct aligment computation for shared object symbols.

@ruiu whatś your opinion on this?

Jun 6 2018, 9:41 AM

Jun 5 2018

shenhan added inline comments to D47602: Correct aligment computation for shared object symbols.
Jun 5 2018, 3:17 PM
shenhan updated the diff for D47602: Correct aligment computation for shared object symbols.
Jun 5 2018, 3:16 PM
shenhan added a comment to D47602: Correct aligment computation for shared object symbols.

Re-write the getAlignment function, keep the original logic but with the 3 additional tests:

Jun 5 2018, 1:56 PM
shenhan updated the diff for D47602: Correct aligment computation for shared object symbols.
Jun 5 2018, 1:56 PM
shenhan added a comment to D47602: Correct aligment computation for shared object symbols.

Mind take another look?

Jun 5 2018, 9:24 AM

Jun 1 2018

shenhan added inline comments to D47602: Correct aligment computation for shared object symbols.
Jun 1 2018, 5:33 PM
shenhan updated the diff for D47602: Correct aligment computation for shared object symbols.
Jun 1 2018, 5:32 PM
shenhan added inline comments to D47602: Correct aligment computation for shared object symbols.
Jun 1 2018, 2:25 PM
shenhan updated the diff for D47602: Correct aligment computation for shared object symbols.
Jun 1 2018, 2:25 PM
shenhan updated the diff for D47602: Correct aligment computation for shared object symbols.
Jun 1 2018, 2:14 PM
shenhan added inline comments to D47602: Correct aligment computation for shared object symbols.
Jun 1 2018, 2:09 PM
shenhan updated the diff for D47602: Correct aligment computation for shared object symbols.
Jun 1 2018, 2:09 PM
shenhan updated the diff for D47602: Correct aligment computation for shared object symbols.

@pcc thanks for the review. Updated the patch -

  1. if getAlignment sees a ABS symbols, returns 0
  2. initialize Ret to UINT64_MAX
  3. in "addCopyRelSymbol", errors out if alignment is 0.
  4. unit test case that tests for proper handling of ABS symbols.
Jun 1 2018, 1:44 PM
shenhan added inline comments to D47602: Correct aligment computation for shared object symbols.
Jun 1 2018, 10:36 AM

May 31 2018

shenhan added inline comments to D47602: Correct aligment computation for shared object symbols.
May 31 2018, 4:41 PM
shenhan added a comment to D47602: Correct aligment computation for shared object symbols.

Thanks @ruiu Updated the patch, also attached a test case.

May 31 2018, 3:50 PM
shenhan updated the summary of D47602: Correct aligment computation for shared object symbols.
May 31 2018, 3:49 PM
shenhan updated the diff for D47602: Correct aligment computation for shared object symbols.
May 31 2018, 3:43 PM
shenhan added a comment to D47602: Correct aligment computation for shared object symbols.

@pcc do you agree with initializing Ret to uint64-1?

May 31 2018, 2:35 PM
shenhan added a comment to D47602: Correct aligment computation for shared object symbols.

@pcc

May 31 2018, 2:02 PM
shenhan added inline comments to D47602: Correct aligment computation for shared object symbols.
May 31 2018, 1:51 PM
shenhan added inline comments to D47602: Correct aligment computation for shared object symbols.
May 31 2018, 1:32 PM
shenhan added a comment to D47602: Correct aligment computation for shared object symbols.

Thanks Rui. I just updated the patch a little bit, that - st_value is only valid for alignment when symbol shndx is set to SHN_COMMON.

May 31 2018, 1:25 PM
shenhan updated the diff for D47602: Correct aligment computation for shared object symbols.
May 31 2018, 1:22 PM
shenhan updated the summary of D47602: Correct aligment computation for shared object symbols.
May 31 2018, 1:21 PM
shenhan added a reviewer for D47602: Correct aligment computation for shared object symbols: ruiu.
May 31 2018, 12:32 PM
shenhan created D47602: Correct aligment computation for shared object symbols.
May 31 2018, 12:31 PM

May 25 2018

shenhan added a comment to D47396: [LLD] Place .nv_fatbin section at the beginning of the executable..

+Han Shen <shenhan@google.com>

Han, did you try that idea before?

May 25 2018, 4:45 PM

May 17 2018

shenhan committed rL332688: Mitigate relocation overflow [part 2 of 2].
Mitigate relocation overflow [part 2 of 2]
May 17 2018, 8:05 PM
shenhan committed rLLD332688: Mitigate relocation overflow [part 2 of 2].
Mitigate relocation overflow [part 2 of 2]
May 17 2018, 8:05 PM
shenhan added a comment to D46200: Mitigate relocation overflow [part 2 of 2].

Thanks. (Comments revised and DefaoutOutSec -> Default)

May 17 2018, 2:11 PM
shenhan updated the diff for D46200: Mitigate relocation overflow [part 2 of 2].
May 17 2018, 2:10 PM
shenhan added inline comments to D46200: Mitigate relocation overflow [part 2 of 2].
May 17 2018, 9:30 AM
shenhan updated the diff for D46200: Mitigate relocation overflow [part 2 of 2].
May 17 2018, 9:30 AM

May 16 2018

shenhan added a comment to D46200: Mitigate relocation overflow [part 2 of 2].

Hi George, mind take a look? (This is a follow up of D45788.)

May 16 2018, 11:16 AM
shenhan edited reviewers for D46200: Mitigate relocation overflow [part 2 of 2], added: grimar; removed: espindola.
May 16 2018, 11:15 AM
shenhan updated the diff for D46200: Mitigate relocation overflow [part 2 of 2].
May 16 2018, 11:14 AM

May 15 2018

shenhan committed rL332374: [lld] Mitigate relocation overflow [part 1 of 2]..
[lld] Mitigate relocation overflow [part 1 of 2].
May 15 2018, 10:09 AM
shenhan committed rLLD332374: [lld] Mitigate relocation overflow [part 1 of 2]..
[lld] Mitigate relocation overflow [part 1 of 2].
May 15 2018, 10:09 AM
shenhan closed D45788: Mitigate relocation overflow [part 1 of 2].
May 15 2018, 10:09 AM

May 14 2018

shenhan added a comment to D46764: Fix test cases that check addresses that are not always invariable.

Thanks for the review. Fixed as suggested in D45788.

May 14 2018, 9:08 PM
shenhan updated the diff for D45788: Mitigate relocation overflow [part 1 of 2].

Merge D46764 (Fix test cases that check addresses that are not always invariable) into this.

May 14 2018, 9:03 PM

May 11 2018

shenhan created D46764: Fix test cases that check addresses that are not always invariable.
May 11 2018, 11:31 AM
shenhan added a comment to D45788: Mitigate relocation overflow [part 1 of 2].

Sorry for breaking the builders. Should double check again after my fix for the broken 4 cases.

May 11 2018, 7:25 AM

May 10 2018

shenhan added a comment to D45788: Mitigate relocation overflow [part 1 of 2].

Some clarification why my test result is different from George's.

May 10 2018, 3:43 PM
shenhan committed rLLD332054: Fix test cases failure caused by revision 332038..
Fix test cases failure caused by revision 332038.
May 10 2018, 3:36 PM
shenhan committed rL332054: Fix test cases failure caused by revision 332038..
Fix test cases failure caused by revision 332038.
May 10 2018, 3:36 PM
shenhan created D46730: Fix test cases failure by revision 332038 .
May 10 2018, 3:30 PM
shenhan added a comment to D45788: Mitigate relocation overflow [part 1 of 2].

Thanks. Landed as https://llvm.org/svn/llvm-project/lld/trunk@332038

May 10 2018, 1:58 PM
shenhan updated the diff for D45788: Mitigate relocation overflow [part 1 of 2].

Synced to newest and re-ran tests, make sure everything passes.

May 10 2018, 1:51 PM
shenhan committed rL332038: Mitigate relocation overflow [part 1 of 2]..
Mitigate relocation overflow [part 1 of 2].
May 10 2018, 1:48 PM
shenhan committed rLLD332038: Mitigate relocation overflow [part 1 of 2]..
Mitigate relocation overflow [part 1 of 2].
May 10 2018, 1:48 PM

May 8 2018

shenhan added a comment to D45788: Mitigate relocation overflow [part 1 of 2].

Thanks for the review.

May 8 2018, 10:19 AM
shenhan updated the diff for D45788: Mitigate relocation overflow [part 1 of 2].
May 8 2018, 10:19 AM
shenhan updated the diff for D45788: Mitigate relocation overflow [part 1 of 2].

Updated according to review feedback. Also synced to upstream HEAD and re-run all tests. All passed now.

May 8 2018, 10:15 AM

May 7 2018

shenhan updated the diff for D45788: Mitigate relocation overflow [part 1 of 2].

Revised linkerscript/out-of-order.s and linkerscript/no-space.s to honor layout constraint (also added notes). Also addressed previous 2 minor comments.

May 7 2018, 10:51 AM

May 4 2018

shenhan added inline comments to D45788: Mitigate relocation overflow [part 1 of 2].
May 4 2018, 1:40 PM