Page MenuHomePhabricator

pzheng (Pengxuan Zheng)
User

Projects

User does not belong to any projects.

User Details

User Since
Dec 7 2016, 1:53 PM (198 w, 3 d)

Recent Activity

Mon, Sep 21

pzheng committed rGe5fea37f1a2a: [Hexagon] Make HexagonVLCR compatibile with New PM (authored by pzheng).
[Hexagon] Make HexagonVLCR compatibile with New PM
Mon, Sep 21, 1:45 PM
pzheng closed D86955: [Hexagon] Make HexagonVLCR compatibile with New PM.
Mon, Sep 21, 1:45 PM · Restricted Project

Sun, Sep 20

pzheng committed rGdeb00cf0b5ab: [Polly][NewPM] Port Simplify to the new pass manager (authored by pzheng).
[Polly][NewPM] Port Simplify to the new pass manager
Sun, Sep 20, 7:18 PM
pzheng closed D87328: [Polly][NewPM] Port Simplify to the new pass manager.
Sun, Sep 20, 7:18 PM · Restricted Project
pzheng added a comment to D87328: [Polly][NewPM] Port Simplify to the new pass manager.

LGTM

A single test test file would be enough to check that the new pass manager pipeline is working. Checking all tests with both pass manager doubles the execution time for check-polly without improving test coverage. There are not too many tests for the simplify pass, but please consider testing it only once with the scalability tests, as these take the most time.

Sun, Sep 20, 7:05 PM · Restricted Project

Sat, Sep 19

pzheng updated the diff for D87328: [Polly][NewPM] Port Simplify to the new pass manager.

minor update to a test

Sat, Sep 19, 1:05 PM · Restricted Project
pzheng updated the diff for D87328: [Polly][NewPM] Port Simplify to the new pass manager.

Thanks for reviewing the patch, @Meinersbur!

Sat, Sep 19, 12:59 PM · Restricted Project

Fri, Sep 18

pzheng accepted D86955: [Hexagon] Make HexagonVLCR compatibile with New PM.
Fri, Sep 18, 5:02 PM · Restricted Project
pzheng added a comment to D80465: [RISCV-V] Provide muldi3 builtins for riscv.

I noticed that this patch (along with D86036) caused a compiler-rt test to fail for riscv64 when M extension is enabled.
compiler-rt/test/builtins/Unit/muldi3_test.c:11: undefined reference to `muldi3'
Basically, with this patch,
muldi3 is no longer defined anymore for riscv64 when M extension is enabled. Any suggestion on how this should be fixed?

That is fixed by my patch in D86457. (It hasn't yet been approved after I addressed some initial review comments, but I think it's now in good shape to land).

Fri, Sep 18, 4:06 PM · Restricted Project
pzheng added a comment to D86955: [Hexagon] Make HexagonVLCR compatibile with New PM.

Is there any existing test case for this pass with the legacy pass manager? If so, you might want to add one with the new pass manager too.

All the existing testcases use opt to run the pass. The pass is not run integrated in the main list of passes for any -Ox level.

That's what I meant. For each legacy pass manager test "RUN: opt -hexagon-vlcr ...", you can add a corresponding new pass manager test "RUN: opt -passes=hexagon-vlcr". This way, we will have the same test coverage with both the legacy and new pass managers.

Hi pzheng, I could not figure out a way to register a target specific pass in opt? I can see that llvm/lib/Passes/PassRegistry.def is the registry for target independent passes. Can you point me to an example/API which can help me in registering this pass so that I can use it -passes option in opt?

I am not sure if there is a better place to register the pass other than PassRegistry.def. Maybe someone who knows can comment on this.

Hi pzheng, testing target specific passes using opt+NPM is not currently supported and the work is still work-in-progress. Can we land this without modifying the testcases for now?

Fri, Sep 18, 4:03 PM · Restricted Project
pzheng added a comment to D80465: [RISCV-V] Provide muldi3 builtins for riscv.

I noticed that this patch (along with D86036) caused a compiler-rt test to fail for riscv64 when M extension is enabled.

Fri, Sep 18, 3:58 PM · Restricted Project

Tue, Sep 15

pzheng added a comment to D86955: [Hexagon] Make HexagonVLCR compatibile with New PM.

Is there any existing test case for this pass with the legacy pass manager? If so, you might want to add one with the new pass manager too.

All the existing testcases use opt to run the pass. The pass is not run integrated in the main list of passes for any -Ox level.

That's what I meant. For each legacy pass manager test "RUN: opt -hexagon-vlcr ...", you can add a corresponding new pass manager test "RUN: opt -passes=hexagon-vlcr". This way, we will have the same test coverage with both the legacy and new pass managers.

Hi pzheng, I could not figure out a way to register a target specific pass in opt? I can see that llvm/lib/Passes/PassRegistry.def is the registry for target independent passes. Can you point me to an example/API which can help me in registering this pass so that I can use it -passes option in opt?

Tue, Sep 15, 11:42 AM · Restricted Project

Thu, Sep 10

pzheng added a reviewer for D87328: [Polly][NewPM] Port Simplify to the new pass manager: chrisj.
Thu, Sep 10, 1:28 PM · Restricted Project

Wed, Sep 9

pzheng added a reviewer for D87328: [Polly][NewPM] Port Simplify to the new pass manager: philip.pfaffe.
Wed, Sep 9, 11:25 AM · Restricted Project

Tue, Sep 8

pzheng requested review of D87328: [Polly][NewPM] Port Simplify to the new pass manager.
Tue, Sep 8, 2:16 PM · Restricted Project
pzheng added a comment to D86955: [Hexagon] Make HexagonVLCR compatibile with New PM.

Is there any existing test case for this pass with the legacy pass manager? If so, you might want to add one with the new pass manager too.

All the existing testcases use opt to run the pass. The pass is not run integrated in the main list of passes for any -Ox level.

Tue, Sep 8, 11:24 AM · Restricted Project
pzheng added a comment to D86955: [Hexagon] Make HexagonVLCR compatibile with New PM.

Is there any existing test case for this pass with the legacy pass manager? If so, you might want to add one with the new pass manager too.

Tue, Sep 8, 11:01 AM · Restricted Project

Fri, Sep 4

pzheng added a comment to D87162: [Driver] Allow -specs and -nostartfiles to be forwarded to GCC.

LGTM.

It's probably not used very common, but we do have some baremetal use cases still relying on these flags to be propagated to GCC.

Thanks for the explanation! (Though I think in these cases calling ld or ld.lld directly might be better?)

And apologies about the friction but I hope the current behavior is better than the previous whether many unrelated options can be forwarded and many can cause warnings on GCC side.

Fri, Sep 4, 3:27 PM · Restricted Project
pzheng committed rG2bccd2b4350f: [Driver] Allow -specs and -nostartfiles to be forwarded to GCC (authored by pzheng).
[Driver] Allow -specs and -nostartfiles to be forwarded to GCC
Fri, Sep 4, 3:10 PM
pzheng closed D87162: [Driver] Allow -specs and -nostartfiles to be forwarded to GCC.
Fri, Sep 4, 3:10 PM · Restricted Project
pzheng updated the diff for D87162: [Driver] Allow -specs and -nostartfiles to be forwarded to GCC.

Updated the test case.

Fri, Sep 4, 3:07 PM · Restricted Project
pzheng added a comment to D87162: [Driver] Allow -specs and -nostartfiles to be forwarded to GCC.

Can D83648 be closed now?

Fri, Sep 4, 3:06 PM · Restricted Project
pzheng abandoned D83648: [Driver] Fix integrated_as definition by setting it as a DriverOption.
Fri, Sep 4, 3:01 PM · Restricted Project
pzheng added a reviewer for D87162: [Driver] Allow -specs and -nostartfiles to be forwarded to GCC: efriedma.
Fri, Sep 4, 2:04 PM · Restricted Project
pzheng requested review of D87162: [Driver] Allow -specs and -nostartfiles to be forwarded to GCC.
Fri, Sep 4, 2:03 PM · Restricted Project

Tue, Sep 1

pzheng added a comment to D86955: [Hexagon] Make HexagonVLCR compatibile with New PM.

Is there any test case for this pass? Please also run clang-format.

Tue, Sep 1, 4:51 PM · Restricted Project

Aug 4 2020

pzheng accepted D85067: [RISCV] Enable the use of the old mucounteren name.

LGTM

Aug 4 2020, 7:49 AM · Restricted Project

Aug 3 2020

pzheng added a comment to D85067: [RISCV] Enable the use of the old mucounteren name.

LGTM, thanks for rebasing the patch, @mmxsrup!

Aug 3 2020, 8:18 AM · Restricted Project

Aug 2 2020

pzheng requested changes to D85067: [RISCV] Enable the use of the old mucounteren name.

The mcountinhibit CSR has already been added in d36f2c6a6c4b. Looks like the patch needs to be rebased?

Aug 2 2020, 10:07 AM · Restricted Project

Jul 15 2020

pzheng updated the diff for D83648: [Driver] Fix integrated_as definition by setting it as a DriverOption.

Add target triple used in test

Jul 15 2020, 7:37 PM · Restricted Project
pzheng added a comment to D83648: [Driver] Fix integrated_as definition by setting it as a DriverOption.

Created http://lists.llvm.org/pipermail/cfe-dev/2020-July/066245.html [cfe-dev] Usage of clang::driver::options::DriverOption (-Xarch_ & gcc toolchain)
to ask about the use case.

When I get time, I'll work on a patch fixing the whole class of options instead of just -fintegrated-as. I will use OPT_Link_Group and add some extra options in the new group: GCCLinkOption.

Jul 15 2020, 7:31 PM · Restricted Project

Jul 12 2020

pzheng updated the diff for D83648: [Driver] Fix integrated_as definition by setting it as a DriverOption.

Add a test case.

Jul 12 2020, 7:37 PM · Restricted Project
pzheng added a comment to D83648: [Driver] Fix integrated_as definition by setting it as a DriverOption.

Actually, this patch won't change --help because it just reduced some duplication by extracting the common part (" the integrated assembler") of the help message into the "help" of "OptOutFFlag". Sorry for the confusion.

Jul 12 2020, 6:27 PM · Restricted Project
pzheng added a reviewer for D83648: [Driver] Fix integrated_as definition by setting it as a DriverOption: apazos.
Jul 12 2020, 4:18 PM · Restricted Project
Herald added a project to D83648: [Driver] Fix integrated_as definition by setting it as a DriverOption: Restricted Project.
Jul 12 2020, 4:17 PM · Restricted Project

Jul 1 2020

pzheng committed rGd36f2c6a6c4b: [RISCV] Add mcountinhibit CSR (authored by pzheng).
[RISCV] Add mcountinhibit CSR
Jul 1 2020, 8:38 AM
pzheng closed D82913: [RISCV] Add mcountinhibit CSR.
Jul 1 2020, 8:38 AM · Restricted Project

Jun 30 2020

pzheng created D82913: [RISCV] Add mcountinhibit CSR.
Jun 30 2020, 2:41 PM · Restricted Project

May 22 2020

pzheng committed rG22ed724975d2: [RISCV] Register null target streamer for RISC-V (authored by pzheng).
[RISCV] Register null target streamer for RISC-V
May 22 2020, 9:39 AM
pzheng closed D80352: [RISCV] Register null target streamer for RISC-V.
May 22 2020, 9:39 AM · Restricted Project
pzheng added a comment to D80352: [RISCV] Register null target streamer for RISC-V.

Thanks for reviewing, @HsiangKai.

May 22 2020, 9:38 AM · Restricted Project

May 21 2020

pzheng updated the diff for D80352: [RISCV] Register null target streamer for RISC-V.

Rebase and clang-format

May 21 2020, 8:03 AM · Restricted Project

May 20 2020

pzheng created D80352: [RISCV] Register null target streamer for RISC-V.
May 20 2020, 7:20 PM · Restricted Project

May 5 2020

pzheng committed rG85aff8a4e49d: [RISCV] Update debug scratch register names (authored by pzheng).
[RISCV] Update debug scratch register names
May 5 2020, 9:09 AM
pzheng closed D78764: [RISCV] Update debug scratch register names.
May 5 2020, 9:08 AM · Restricted Project
pzheng updated the diff for D78764: [RISCV] Update debug scratch register names.

Good catch. Thanks for reviewing, @asb.

May 5 2020, 9:08 AM · Restricted Project

May 4 2020

pzheng updated the diff for D78764: [RISCV] Update debug scratch register names.

Address @asb's comments

May 4 2020, 9:05 AM · Restricted Project

Apr 24 2020

pzheng committed rG79702dd349f3: [RISCV] Add instruction definition for dret (authored by pzheng).
[RISCV] Add instruction definition for dret
Apr 24 2020, 1:33 PM
pzheng closed D78583: [RISCV] Add instruction definition for dret.
Apr 24 2020, 1:33 PM · Restricted Project
pzheng updated the diff for D78764: [RISCV] Update debug scratch register names.

Moving debug mode registers to a new test file

Apr 24 2020, 12:26 PM · Restricted Project

Apr 23 2020

pzheng updated the diff for D78764: [RISCV] Update debug scratch register names.

Addressing comments

Apr 23 2020, 8:06 PM · Restricted Project
pzheng created D78764: [RISCV] Update debug scratch register names.
Apr 23 2020, 3:15 PM · Restricted Project
pzheng updated the diff for D78583: [RISCV] Add instruction definition for dret.

Reverted changes in priv-valid.s

Apr 23 2020, 11:52 AM · Restricted Project
pzheng updated the diff for D78583: [RISCV] Add instruction definition for dret.

Addressing comments

Apr 23 2020, 11:23 AM · Restricted Project

Apr 21 2020

pzheng created D78583: [RISCV] Add instruction definition for dret.
Apr 21 2020, 1:33 PM · Restricted Project

Mar 23 2020

pzheng abandoned D76632: [libc++] Do not use futex if LIBCXX_HAS_MUSL_LIBC is ON.

The build actually fails because of "#include <linux/futex.h>". Musl does not have this header file and the build fails with a missing header file error.

Maybe that header is supposed to come from the kernel and not the C library.

Mar 23 2020, 3:48 PM · Restricted Project
pzheng added a comment to D76632: [libc++] Do not use futex if LIBCXX_HAS_MUSL_LIBC is ON.

The build actually fails because of "#include <linux/futex.h>". Musl does not have this header file and the build fails with a missing header file error.

Mar 23 2020, 1:38 PM · Restricted Project
pzheng added reviewers for D76632: [libc++] Do not use futex if LIBCXX_HAS_MUSL_LIBC is ON: __simt__, apazos, bcain, sidneym.
Mar 23 2020, 11:28 AM · Restricted Project
pzheng created D76632: [libc++] Do not use futex if LIBCXX_HAS_MUSL_LIBC is ON.
Mar 23 2020, 11:27 AM · Restricted Project

Feb 19 2020

pzheng added inline comments to D74023: [RISCV] ELF attribute section for RISC-V.
Feb 19 2020, 4:42 PM · Restricted Project, Restricted Project

Feb 18 2020

pzheng added inline comments to D74023: [RISCV] ELF attribute section for RISC-V.
Feb 18 2020, 4:03 PM · Restricted Project, Restricted Project
pzheng added a reviewer for D74023: [RISCV] ELF attribute section for RISC-V: pzheng.
Feb 18 2020, 3:15 PM · Restricted Project, Restricted Project

Jan 13 2020

pzheng added a comment to D62686: [RISCV] Add support for save/restore of callee-saved registers via libcalls.

Fix .cfi_offset signedness error.

Jan 13 2020, 2:48 PM · Restricted Project, Restricted Project

Jan 10 2020

pzheng added a comment to D62686: [RISCV] Add support for save/restore of callee-saved registers via libcalls.

I see the following .cfi_offset directives generated using @shiva0217's test case. Any idea why the offset for ra is 536870908?

Jan 10 2020, 3:48 PM · Restricted Project, Restricted Project

Nov 14 2019

pzheng added inline comments to D62686: [RISCV] Add support for save/restore of callee-saved registers via libcalls.
Nov 14 2019, 1:02 PM · Restricted Project, Restricted Project

Sep 20 2019

pzheng committed rL372431: Request commit access for pzheng.
Request commit access for pzheng
Sep 20 2019, 2:55 PM

Aug 15 2019

pzheng abandoned D66266: [WIP][RISCV] Set MaxAtomicPromoteWidth and MaxAtomicInlineWidth.

Hi Sam,

Aug 15 2019, 12:41 PM · Restricted Project

Aug 14 2019

pzheng created D66266: [WIP][RISCV] Set MaxAtomicPromoteWidth and MaxAtomicInlineWidth.
Aug 14 2019, 5:51 PM · Restricted Project

Jun 26 2019

pzheng committed rG4030631bfa24: [cmake] Allow config.guess to be run with MSYS on Windows (authored by pzheng).
[cmake] Allow config.guess to be run with MSYS on Windows
Jun 26 2019, 3:09 PM
pzheng committed rL364485: [cmake] Allow config.guess to be run with MSYS on Windows.
[cmake] Allow config.guess to be run with MSYS on Windows
Jun 26 2019, 3:08 PM
pzheng closed D63834: [cmake] Allow config.guess to be run with MSYS on Windows.
Jun 26 2019, 3:07 PM · Restricted Project
pzheng added a reviewer for D63834: [cmake] Allow config.guess to be run with MSYS on Windows: compnerd.
Jun 26 2019, 11:39 AM · Restricted Project
pzheng created D63834: [cmake] Allow config.guess to be run with MSYS on Windows.
Jun 26 2019, 11:39 AM · Restricted Project

Jun 5 2019

pzheng committed rGe34d1a4e07b8: [cmake] Remove duplicate TestingSupport library for linking (authored by pzheng).
[cmake] Remove duplicate TestingSupport library for linking
Jun 5 2019, 12:42 PM
pzheng committed rL362637: [cmake] Remove duplicate TestingSupport library for linking.
[cmake] Remove duplicate TestingSupport library for linking
Jun 5 2019, 12:42 PM
pzheng closed D62924: [cmake] Remove duplicate TestingSupport library for linking.
Jun 5 2019, 12:42 PM · Restricted Project
pzheng added inline comments to D62333: Fix unresolved symbols when linking tools/clang/unittests/Tooling/ToolingTests.
Jun 5 2019, 12:05 PM · Restricted Project, Restricted Project
pzheng added reviewers for D62924: [cmake] Remove duplicate TestingSupport library for linking: quantum, thakis, tlively.
Jun 5 2019, 12:01 PM · Restricted Project
pzheng created D62924: [cmake] Remove duplicate TestingSupport library for linking.
Jun 5 2019, 11:56 AM · Restricted Project

Apr 11 2019

pzheng committed rGd8ada681df24: [cmake] Fix dependency issue in TableGen (authored by pzheng).
[cmake] Fix dependency issue in TableGen
Apr 11 2019, 2:08 PM
pzheng committed rL358226: [cmake] Fix dependency issue in TableGen.
[cmake] Fix dependency issue in TableGen
Apr 11 2019, 2:08 PM
pzheng closed D60576: [cmake] Fix dependency issue in TableGen.
Apr 11 2019, 2:08 PM · Restricted Project
pzheng added a comment to D60576: [cmake] Fix dependency issue in TableGen.

Thanks for reviewing, aganea!

Apr 11 2019, 1:55 PM · Restricted Project
pzheng added reviewers for D60576: [cmake] Fix dependency issue in TableGen: aganea, smeenai.
Apr 11 2019, 1:31 PM · Restricted Project
pzheng created D60576: [cmake] Fix dependency issue in TableGen.
Apr 11 2019, 1:28 PM · Restricted Project

Apr 1 2019

pzheng added inline comments to D58951: [compiler-rt][tests] Improve handling with non-default toolchains.
Apr 1 2019, 5:53 PM · Restricted Project, Restricted Project

Feb 7 2019

pzheng committed rL353458: [zorg] Update host compiler for reverse-iteration builder.
[zorg] Update host compiler for reverse-iteration builder
Feb 7 2019, 12:18 PM
pzheng closed D57917: [zorg] Update host compiler for reverse-iteration builder.
Feb 7 2019, 12:18 PM
pzheng added inline comments to D57917: [zorg] Update host compiler for reverse-iteration builder.
Feb 7 2019, 12:00 PM
pzheng updated the diff for D57917: [zorg] Update host compiler for reverse-iteration builder.

[zorg] Update host compiler for reverse-iteration builder

Feb 7 2019, 11:57 AM
pzheng added a reviewer for D57917: [zorg] Update host compiler for reverse-iteration builder: gkistanova.
Feb 7 2019, 11:25 AM
pzheng added reviewers for D57917: [zorg] Update host compiler for reverse-iteration builder: mgrang, jfb, kparzysz, zinob.
Feb 7 2019, 11:13 AM
pzheng created D57917: [zorg] Update host compiler for reverse-iteration builder.
Feb 7 2019, 11:11 AM
pzheng committed rL353441: [zorg] Update host compiler for polly-arm-linux builder.
[zorg] Update host compiler for polly-arm-linux builder
Feb 7 2019, 10:24 AM
pzheng closed D57865: [zorg] Update host compiler for polly-arm-linux builder.
Feb 7 2019, 10:24 AM

Feb 6 2019

pzheng added reviewers for D57865: [zorg] Update host compiler for polly-arm-linux builder: grosser, zinob, kparzysz, jfb, gkistanova.
Feb 6 2019, 4:31 PM
pzheng created D57865: [zorg] Update host compiler for polly-arm-linux builder.
Feb 6 2019, 4:23 PM

Apr 5 2018

pzheng committed rL329364: Update recipient list of AOSP builder.
Update recipient list of AOSP builder
Apr 5 2018, 4:57 PM

Oct 25 2017

pzheng committed rL316603: [zorg] Update status reporting of reverse-iteration builder.
[zorg] Update status reporting of reverse-iteration builder
Oct 25 2017, 12:37 PM
pzheng closed D39258: [zorg] Update status reporting of reverse-iteration builder.
Oct 25 2017, 12:37 PM