Page MenuHomePhabricator
People lei

lei (Lei Huang)
User

Projects

User does not belong to any projects.

User Details

User Since
Oct 24 2016, 11:46 AM (242 w, 3 d)

Recent Activity
View All

Today

lei added inline comments to D102875: [PowerPC] Add PowerPC compare and multiply related builtins and instrinsics for XL compatibility.
Thu, Jun 17, 11:49 AM · Restricted Project, Restricted Project
lei added inline comments to D102875: [PowerPC] Add PowerPC compare and multiply related builtins and instrinsics for XL compatibility.
Thu, Jun 17, 11:47 AM · Restricted Project, Restricted Project
lei added inline comments to D102875: [PowerPC] Add PowerPC compare and multiply related builtins and instrinsics for XL compatibility.
Thu, Jun 17, 7:08 AM · Restricted Project, Restricted Project

Mon, Jun 14

lei accepted D104125: [PowerPC] Moving defineXLCompatMacros() definition.

LGTM
Thanks.

Mon, Jun 14, 10:21 AM · Restricted Project
lei added a comment to D103979: [lld-macho] Have dead-stripping work with literal sections.

The changes in the following commits have been caused failures in our bot, https://lab.llvm.org/buildbot/#/workers/50, since last Friday:

Mon, Jun 14, 9:04 AM · Restricted Project, Restricted Project

May 13 2021

lei committed rG9469ff15b779: [PowerPC] Add clang option -m[no-]prefixed (authored by lei).
[PowerPC] Add clang option -m[no-]prefixed
May 13 2021, 10:02 AM
lei closed D102191: [PowerPC] Add clang option -m[no-]prefixed.
May 13 2021, 10:02 AM · Restricted Project

May 10 2021

lei added a reviewer for D102191: [PowerPC] Add clang option -m[no-]prefixed: amyk.
May 10 2021, 1:38 PM · Restricted Project
lei requested review of D102191: [PowerPC] Add clang option -m[no-]prefixed.
May 10 2021, 1:23 PM · Restricted Project

May 6 2021

lei added inline comments to D101959: [libc++/abi/unwind][AIX] Add scripts and cache file for building libc++/libc++abi/libunwind.
May 6 2021, 1:54 PM · Restricted Project

Apr 8 2021

lei accepted D100038: [Sanitizer] Fix ccache not found and unbound variable errors.

LGTM

Apr 8 2021, 6:54 AM · Restricted Project

Mar 23 2021

lei accepted D99193: [PowerPC] Add mprivileged option.

LGTM

Mar 23 2021, 10:27 AM · Restricted Project, Restricted Project
lei accepted D99185: [PowerPC] Change option to mrop-protect.

LGTM.

Mar 23 2021, 10:19 AM · Restricted Project, Restricted Project

Mar 9 2021

lei accepted D97391: [NFC][PowerPC] Add additional load/store test cases.

This looks to be a NFC to me. Thx for adding the additional tests coverage.

Mar 9 2021, 3:07 PM · Restricted Project
lei accepted D98042: [PowerPC] Implement patterns for PC-Rel zextload/extload byte loads.
Mar 9 2021, 12:40 PM · Restricted Project
lei accepted D93300: [PowerPC] Exploit paddi instruction on Power 10 for constant materialization.
Mar 9 2021, 7:06 AM · Restricted Project
lei added inline comments to D93300: [PowerPC] Exploit paddi instruction on Power 10 for constant materialization.
Mar 9 2021, 6:47 AM · Restricted Project

Mar 8 2021

lei committed rG535a4192a9d1: [AIX][TLS] Generate 64-bit general-dynamic access code sequence (authored by lei).
[AIX][TLS] Generate 64-bit general-dynamic access code sequence
Mar 8 2021, 2:41 PM
lei closed D98078: [AIX][TLS] Generate 64-bit general-dynamic access code sequence.
Mar 8 2021, 2:41 PM · Restricted Project
lei added a comment to D89986: [AIX] do not emit visibility attribute into IR when there is -mignore-xcoff-visibility.
Mar 8 2021, 11:40 AM · Restricted Project
lei committed rGb0f0115308e4: [AIX][TLS] Generate 32-bit general-dynamic access code sequence (authored by nemanjai).
[AIX][TLS] Generate 32-bit general-dynamic access code sequence
Mar 8 2021, 7:31 AM
lei closed D97948: [AIX][TLS] Generate 32-bit general-dynamic access code sequence.
Mar 8 2021, 7:31 AM · Restricted Project

Mar 5 2021

lei requested review of D98078: [AIX][TLS] Generate 64-bit general-dynamic access code sequence.
Mar 5 2021, 2:19 PM · Restricted Project
lei updated the diff for D97948: [AIX][TLS] Generate 32-bit general-dynamic access code sequence.

Remove autogen check msg from testcase.

Mar 5 2021, 11:00 AM · Restricted Project
lei updated the diff for D97948: [AIX][TLS] Generate 32-bit general-dynamic access code sequence.

Add checks for TOC entries

Mar 5 2021, 10:57 AM · Restricted Project
lei updated the diff for D97948: [AIX][TLS] Generate 32-bit general-dynamic access code sequence.

Remove unnecessary code documentation.
Update test to use IR generated via -O2

Mar 5 2021, 7:10 AM · Restricted Project

Mar 4 2021

lei updated the diff for D97948: [AIX][TLS] Generate 32-bit general-dynamic access code sequence.

fix typo

Mar 4 2021, 6:18 PM · Restricted Project
lei updated the diff for D97948: [AIX][TLS] Generate 32-bit general-dynamic access code sequence.

Remove non tls related tests.
Split up linkage coverage between the 3 test files.
Improve on documentation of code.

Mar 4 2021, 1:27 PM · Restricted Project
lei updated the diff for D97948: [AIX][TLS] Generate 32-bit general-dynamic access code sequence.

Clean up test cases.

Mar 4 2021, 9:16 AM · Restricted Project
lei requested review of D97948: [AIX][TLS] Generate 32-bit general-dynamic access code sequence.
Mar 4 2021, 7:09 AM · Restricted Project

Feb 22 2021

lei added inline comments to D96184: [AIX][TLS] Generate TLS variables in assembly files.
Feb 22 2021, 1:25 PM · Unknown Object (Project), Restricted Project

Jan 28 2021

lei accepted D95634: [PowerPC][Power10] Fix XXSPLI32DX not correctly exploiting specific cases.

LGTM

Jan 28 2021, 10:47 AM · Unknown Object (Project), Restricted Project, Restricted Project

Dec 17 2020

lei accepted D89195: [PowerPC][Power10] Exploit store rightmost vector element instructions..

LGTM

Dec 17 2020, 1:22 PM · Unknown Object (Project), Restricted Project
lei added inline comments to D92879: [PowerPC] Materialize 34 bit constants with pli on Power 10..
Dec 17 2020, 8:58 AM · Restricted Project

Dec 16 2020

lei added a comment to D92089: [PowerPC] Materialize i64 constants by enumerated patterns..

Minor nit to be addressed upon commit. Thx!

Dec 16 2020, 5:27 AM · Restricted Project

Nov 25 2020

lei added a comment to rGe16c0a9a6897: clang+lld: Improve clang+ld.darwinnew.lld interaction, pass -demangle.

Thank-you!

Nov 25 2020, 8:28 AM

Nov 24 2020

lei added a comment to rGe16c0a9a6897: clang+lld: Improve clang+ld.darwinnew.lld interaction, pass -demangle.
In rGe16c0a9a689719f379e49d0a05cb58774cce4adb#961517, @thakis wrote:

I'm guessing your bot sets CLANG_DEFAULT_LINKER to lld?

Nov 24 2020, 12:50 PM
lei added a comment to rGe16c0a9a6897: clang+lld: Improve clang+ld.darwinnew.lld interaction, pass -demangle.

This seem to be only failing for our bot that uses the lld linker. Does not affect bots using the ld linker.

Nov 24 2020, 12:49 PM
lei added a comment to rGe16c0a9a6897: clang+lld: Improve clang+ld.darwinnew.lld interaction, pass -demangle.

Testscase is expecting

Nov 24 2020, 11:35 AM
lei added a comment to rGe16c0a9a6897: clang+lld: Improve clang+ld.darwinnew.lld interaction, pass -demangle.

This patch broke our rhel bot: clang-ppc64le-rhel
The subsequent patch to fix symlink name did not fix the original failure seen:

Nov 24 2020, 11:15 AM

Nov 18 2020

lei added a comment to rG4cadb66b490e: [AST] Update matchers to be traverse-aware.

Actually it;s the previous patch that causes the regression: 246b428fb3b5d58685064966bcea6592f8839b7e

Nov 18 2020, 11:11 AM

Nov 17 2020

lei added a comment to rG4cadb66b490e: [AST] Update matchers to be traverse-aware.

This patch broke the clang-ppc64le* buildbots: http://lab.llvm.org:8011/#/builders/105/builds/1907
Pleases upload fix or pull.

Nov 17 2020, 2:19 PM

Nov 9 2020

lei added inline comments to D90799: [PowerPC] Add paired vector load and store builtins and intrinsics.
Nov 9 2020, 6:51 AM · Unknown Object (Project), Restricted Project, Restricted Project

Nov 2 2020

lei accepted D90645: [zorg][PowerPC] Don't checkout compiler-rt for runtime build.

thx for the fix.

Nov 2 2020, 1:50 PM

Oct 23 2020

lei accepted D89345: [PowerPC] Add intrinsics for MMA.
Oct 23 2020, 7:34 AM · Unknown Object (Project), Restricted Project
lei accepted D87705: [PowerPC] Implement Set Boolean Condition Instructions.

LGTM. Thanks for the update.
Please address the additional comments on commit.

Oct 23 2020, 6:19 AM · Unknown Object (Project), Restricted Project

Oct 22 2020

lei added inline comments to D89195: [PowerPC][Power10] Exploit store rightmost vector element instructions..
Oct 22 2020, 7:31 AM · Unknown Object (Project), Restricted Project
lei added a comment to D87705: [PowerPC] Implement Set Boolean Condition Instructions.

For the p10 specific tests, I think just use the default CHECK vs CHECK-P10.
Also, please ensure we are testing on both BE and LE.

Oct 22 2020, 6:16 AM · Unknown Object (Project), Restricted Project

Sep 18 2020

lei accepted D84359: [PowerPC] Add vector pair load/store instructions and vector pair register class.

LGTM. Thx!

Sep 18 2020, 12:39 PM · Unknown Object (Project), Restricted Project
lei added a comment to D87804: [PowerPC][Power10] Implement Vector signed/unsigned __int128 overloads for the comparison builtins.

please fix the clang format issues.

Sep 18 2020, 7:05 AM · Restricted Project, Unknown Object (Project), Restricted Project
lei accepted D84968: [PowerPC] Legalize v256i1 and v512i1 and implement load and store of these types.

Just some minor comments. Please address them prior to commit.

Sep 18 2020, 6:26 AM · Restricted Project, Unknown Object (Project), Restricted Project

Sep 17 2020

lei accepted D84847: [PowerPC] Add accumulator register class and instructions.

LGTM

Sep 17 2020, 3:13 PM · Unknown Object (Project), Restricted Project
lei added inline comments to D84359: [PowerPC] Add vector pair load/store instructions and vector pair register class.
Sep 17 2020, 2:52 PM · Unknown Object (Project), Restricted Project

Sep 11 2020

lei accepted D86252: [PowerPC] Add Set Boolean Condition Instruction Definitions and MC Tests.

LGTM

Sep 11 2020, 10:54 AM · Restricted Project

Aug 26 2020

lei accepted D82609: [PowerPC] Implement Vector Multiply High/Divide Extended Builtins in LLVM/Clang.

LGTM
Thx!

Aug 26 2020, 1:32 PM · Unknown Object (Project), Restricted Project, Restricted Project

Jul 29 2020

lei committed rG7aaa85627b59: [PowerPC] Add options to control paired vector memops support (authored by bsaleil).
[PowerPC] Add options to control paired vector memops support
Jul 29 2020, 12:01 PM
lei closed D83722: [PowerPC] Add options to control paired vector memops support.
Jul 29 2020, 12:01 PM · Unknown Object (Project), Restricted Project, Restricted Project

Jul 27 2020

lei added a comment to D83338: [PowerPC][Power10] Implemented Vector Shift Builtins.

Please address the auto generated clang-format issues for the added code in this patch.

Jul 27 2020, 6:38 AM · Unknown Object (Project), Restricted Project, Restricted Project
lei added inline comments to D82502: [PowerPC] Implement Load VSX Vector and Sign Extend and Zero Extend.
Jul 27 2020, 6:18 AM · Unknown Object (Project), Restricted Project, Restricted Project

Jul 23 2020

lei accepted D84382: [PowerPC][Power10] Cleanup p10vector clang test.

LGTM
thx!

Jul 23 2020, 5:34 AM · Restricted Project
lei accepted D84384: [PowerPC][Power10] Add Vector Extract Double Instruction Definitions and MC tests.

LGTM

Jul 23 2020, 5:31 AM · Restricted Project
lei added inline comments to D82502: [PowerPC] Implement Load VSX Vector and Sign Extend and Zero Extend.
Jul 23 2020, 5:23 AM · Unknown Object (Project), Restricted Project, Restricted Project
lei added a comment to D84197: [PowerPC][Power10] Vector String Isolate instruction definitions and MC Tests.

Why are you removing MC tests for here?

Jul 23 2020, 5:19 AM · Unknown Object (Project), Restricted Project, Restricted Project
lei accepted D82467: [PowerPC][Power10] Implement Truncate and Store VSX Vector Builtins.

LGTM,
Please add tests for BE in llvm/test/CodeGen/PowerPC/builtins-ppc-p10vsx.ll prior to committing.

Jul 23 2020, 4:55 AM · Unknown Object (Project), Restricted Project, Restricted Project

Jul 10 2020

lei accepted D83516: [PowerPC][Power10] 128-bit Binary Integer Operation instruction definitions and MC Tests.

LGTM
Please address the nits on commit.

Jul 10 2020, 1:00 PM · Unknown Object (Project), Restricted Project, Restricted Project
lei committed rG90b1a710aede: [PowerPC] Enable default support of quad precision operations (authored by lei).
[PowerPC] Enable default support of quad precision operations
Jul 10 2020, 11:28 AM
lei closed D83437: [PowerPC] Enable default support of quad precision operations.
Jul 10 2020, 11:27 AM · Restricted Project

Jul 9 2020

lei accepted D83497: [PowerPC][Power10] Fix VINS* (vector insert byte/half/word) instructions to have i32 arguments..

LGTM

Jul 9 2020, 1:53 PM · Restricted Project, Unknown Object (Project), Restricted Project
lei added inline comments to D83500: [PowerPC][Power10] Implement custom codegen for the vec_replace_elt and vec_replace_unaligned builtins..
Jul 9 2020, 12:40 PM · Restricted Project, Unknown Object (Project), Restricted Project
lei added a reviewer for D83497: [PowerPC][Power10] Fix VINS* (vector insert byte/half/word) instructions to have i32 arguments.: rzurob.
Jul 9 2020, 12:10 PM · Restricted Project, Unknown Object (Project), Restricted Project
lei accepted D83364: [PowerPC][Power10] Implement Instruction definition and MC Tests for Load and Store VSX Vector with Zero or Sign Extend.

LGTM thx.

Jul 9 2020, 11:15 AM · Unknown Object (Project), Restricted Project, Restricted Project
lei removed a reviewer for D83437: [PowerPC] Enable default support of quad precision operations: • qshanz.
Jul 9 2020, 6:24 AM · Restricted Project

Jul 8 2020

lei added a reviewer for D83437: [PowerPC] Enable default support of quad precision operations: • qshanz.
Jul 8 2020, 3:21 PM · Restricted Project
lei added reviewers for D83437: [PowerPC] Enable default support of quad precision operations: nemanjai, Restricted Project.
Jul 8 2020, 3:20 PM · Restricted Project
Herald added a project to D83437: [PowerPC] Enable default support of quad precision operations: Restricted Project.
Jul 8 2020, 3:19 PM · Restricted Project
lei added inline comments to D82609: [PowerPC] Implement Vector Multiply High/Divide Extended Builtins in LLVM/Clang.
Jul 8 2020, 12:24 PM · Unknown Object (Project), Restricted Project, Restricted Project
lei added a comment to D82510: [PowerPC][Power10] Implement low-order Vector Multiply, Modulus and Divide Instructions.

This LGTM, just wondering why you have not included testing for BE.

Jul 8 2020, 11:38 AM · Unknown Object (Project), Restricted Project
lei added inline comments to D83364: [PowerPC][Power10] Implement Instruction definition and MC Tests for Load and Store VSX Vector with Zero or Sign Extend.
Jul 8 2020, 9:12 AM · Unknown Object (Project), Restricted Project, Restricted Project

Jul 7 2020

lei committed rG62ba48b45f65: [PowerPC] Implement Vector Replace Builtins in LLVM (authored by biplmish).
[PowerPC] Implement Vector Replace Builtins in LLVM
Jul 7 2020, 10:24 AM
lei closed D83308: [Power10] Implement Vector Replace Builtins in LLVM.
Jul 7 2020, 10:24 AM · Restricted Project
lei added inline comments to D82502: [PowerPC] Implement Load VSX Vector and Sign Extend and Zero Extend.
Jul 7 2020, 6:45 AM · Unknown Object (Project), Restricted Project, Restricted Project

Jul 6 2020

lei committed rG0c6b6e28e70c: [PowerPC] Implement Vector Splat Immediate Builtins in Clang (authored by biplmish).
[PowerPC] Implement Vector Splat Immediate Builtins in Clang
Jul 6 2020, 6:31 PM
lei closed D82520: [Power10] Implement Vector Splat Immediate Builtins in LLVM/Clang.
Jul 6 2020, 6:31 PM · Restricted Project, Restricted Project
lei accepted D83245: [PowerPC][Power10] Exploit the xxsplti32dx instruction when lowering VECTOR_SHUFFLE..

LGTM

Jul 6 2020, 1:51 PM · Restricted Project
lei accepted D82520: [Power10] Implement Vector Splat Immediate Builtins in LLVM/Clang.

LGTM

Jul 6 2020, 1:06 PM · Restricted Project, Restricted Project
lei added inline comments to D83245: [PowerPC][Power10] Exploit the xxsplti32dx instruction when lowering VECTOR_SHUFFLE..
Jul 6 2020, 12:40 PM · Restricted Project
lei added inline comments to D83245: [PowerPC][Power10] Exploit the xxsplti32dx instruction when lowering VECTOR_SHUFFLE..
Jul 6 2020, 11:44 AM · Restricted Project
lei added inline comments to D82520: [Power10] Implement Vector Splat Immediate Builtins in LLVM/Clang.
Jul 6 2020, 11:16 AM · Restricted Project, Restricted Project
lei added inline comments to D82520: [Power10] Implement Vector Splat Immediate Builtins in LLVM/Clang.
Jul 6 2020, 10:25 AM · Restricted Project, Restricted Project
lei added inline comments to D82359: [Power10] Implement Vector Replace Builtins in LLVM/Clang.
Jul 6 2020, 9:38 AM · Restricted Project
lei added inline comments to D82365: [Power10] Implement Vector Insert Builtins in LLVM/Clang.
Jul 6 2020, 6:42 AM · Restricted Project, Restricted Project
lei added inline comments to D82359: [Power10] Implement Vector Replace Builtins in LLVM/Clang.
Jul 6 2020, 6:38 AM · Restricted Project

Jul 3 2020

lei committed rGe359ab1eca57: [PowerPC][NFC] Fix indentation (authored by lei).
[PowerPC][NFC] Fix indentation
Jul 3 2020, 3:14 PM
lei added inline comments to D82359: [Power10] Implement Vector Replace Builtins in LLVM/Clang.
Jul 3 2020, 3:14 PM · Restricted Project
lei requested changes to D82359: [Power10] Implement Vector Replace Builtins in LLVM/Clang.

This patch doesn't contain full context. Please regenerate with -U999999.

Jul 3 2020, 2:09 PM · Restricted Project
lei added inline comments to D82365: [Power10] Implement Vector Insert Builtins in LLVM/Clang.
Jul 3 2020, 2:09 PM · Restricted Project, Restricted Project
lei committed rG0939e04e4176: [PowerPC] Implement Vector Insert Builtins in LLVM/Clang (authored by biplmish).
[PowerPC] Implement Vector Insert Builtins in LLVM/Clang
Jul 3 2020, 1:38 PM
lei closed D82365: [Power10] Implement Vector Insert Builtins in LLVM/Clang.
Jul 3 2020, 1:37 PM · Restricted Project, Restricted Project
lei added a comment to D82365: [Power10] Implement Vector Insert Builtins in LLVM/Clang.

I will commit this for Biplob since he does't have commit access yet.

Jul 3 2020, 10:44 AM · Restricted Project, Restricted Project
lei added inline comments to D82365: [Power10] Implement Vector Insert Builtins in LLVM/Clang.
Jul 3 2020, 6:26 AM · Restricted Project, Restricted Project