Page MenuHomePhabricator
Feed Advanced Search

Thu, Apr 2

lkail added reviewers for D77357: [PowerPC][UpdateTestChecks] Remove the extra # when scrubbing loop comments: RKSimon, dmgreen.
Thu, Apr 2, 11:17 PM · Restricted Project, Restricted Project

Wed, Apr 1

lkail added a reviewer for D73290: [PowerPC] Add clang -msvr4-struct-return for 32-bit ELF: Restricted Project.
Wed, Apr 1, 9:14 PM · Restricted Project

Tue, Mar 31

lkail abandoned D76982: [PowerPC] Update Alignment of ReuseLoadInfo in PPCTargetLowering::LowerFP_TO_INTForReuse.

Should be fixed by 998118c3d3bd6a394c1da35f7570cce1a3145ea3, which does more checks on alignment.

Tue, Mar 31, 8:23 PM · Restricted Project
lkail committed rG8eb40e41f6ec: [PowerPC] Don't generate ST_VSR_SCAL_INT if power8-vector is disabled (authored by lkail).
[PowerPC] Don't generate ST_VSR_SCAL_INT if power8-vector is disabled
Tue, Mar 31, 7:51 PM
lkail closed D76773: [PowerPC] Don't generate ST_VSR_SCAL_INT if power8-vector is disabled, fix PR45297.
Tue, Mar 31, 7:50 PM · Restricted Project

Mon, Mar 30

lkail set the repository for D76982: [PowerPC] Update Alignment of ReuseLoadInfo in PPCTargetLowering::LowerFP_TO_INTForReuse to rG LLVM Github Monorepo.
Mon, Mar 30, 4:24 PM · Restricted Project
lkail updated the diff for D76982: [PowerPC] Update Alignment of ReuseLoadInfo in PPCTargetLowering::LowerFP_TO_INTForReuse.

Rebased.

Mon, Mar 30, 4:24 PM · Restricted Project

Sat, Mar 28

lkail created D76982: [PowerPC] Update Alignment of ReuseLoadInfo in PPCTargetLowering::LowerFP_TO_INTForReuse.
Sat, Mar 28, 3:46 AM · Restricted Project

Thu, Mar 26

lkail committed rG26b46b67d806: [PowerPC] Fix test for PR45297 to adapt build without asserts. NFC. (authored by lkail).
[PowerPC] Fix test for PR45297 to adapt build without asserts. NFC.
Thu, Mar 26, 10:48 PM
lkail updated the diff for D76773: [PowerPC] Don't generate ST_VSR_SCAL_INT if power8-vector is disabled, fix PR45297.
Thu, Mar 26, 10:48 PM · Restricted Project
lkail updated the diff for D76773: [PowerPC] Don't generate ST_VSR_SCAL_INT if power8-vector is disabled, fix PR45297.

Address @sfertile 's comment.

Thu, Mar 26, 10:16 PM · Restricted Project
lkail committed rG351b19231554: [PowerPC] Enhance test for PR45297. NFC. (authored by lkail).
[PowerPC] Enhance test for PR45297. NFC.
Thu, Mar 26, 9:47 PM

Wed, Mar 25

lkail retitled D76773: [PowerPC] Don't generate ST_VSR_SCAL_INT if power8-vector is disabled, fix PR45297 from [PowerPC] Fix PR45297 to [PowerPC] Don't generate ST_VSR_SCAL_INT if power8-vector is disabled, fix PR45297.
Wed, Mar 25, 6:58 PM · Restricted Project
lkail updated the summary of D76773: [PowerPC] Don't generate ST_VSR_SCAL_INT if power8-vector is disabled, fix PR45297.
Wed, Mar 25, 7:32 AM · Restricted Project
lkail created D76773: [PowerPC] Don't generate ST_VSR_SCAL_INT if power8-vector is disabled, fix PR45297.
Wed, Mar 25, 7:32 AM · Restricted Project

Tue, Mar 24

lkail committed rG70f9f4dd9d19: [PowerPC] Pre-commit reduced test case for PR45297. NFC. (authored by lkail).
[PowerPC] Pre-commit reduced test case for PR45297. NFC.
Tue, Mar 24, 11:57 PM

Sun, Mar 15

lkail added a reviewer for D76207: [PowerPC] implement target hook isProfitableToHoist: Restricted Project.
Sun, Mar 15, 7:52 PM · Restricted Project
lkail added a reviewer for D76042: [PowerPC] Add the Uses of implicit register for the BCLRn instruction: hfinkel.
Sun, Mar 15, 7:20 PM · Restricted Project

Sat, Mar 7

Herald updated subscribers of D75388: Expand interleaved memory access pass to identify certain shuffle_vector and transform it into target specific intrinsics..
Sat, Mar 7, 3:15 AM · Restricted Project

Feb 18 2020

Herald added a project to D41689: [SCEVAA] Don't crash on pointers with no dominance relationship.: Restricted Project.
Feb 18 2020, 2:50 AM · Restricted Project

Feb 17 2020

lkail accepted D70771: [PowerPC] Replace the PPCISD:: SExtVElems with ISD::SIGN_EXTEND_INREG to leverage the combine rules .

LGTM. Please wait a couple of days to land in case other reviewers have other concerns.

Feb 17 2020, 9:46 PM · Restricted Project, Restricted Project
lkail added a comment to D70771: [PowerPC] Replace the PPCISD:: SExtVElems with ISD::SIGN_EXTEND_INREG to leverage the combine rules .

We are combing the instructions into sext_inreg instead of Power specific node SExtVElems, and then, it is selected as hw instruction which is defined in the pattern td.

CMIIW, SExtVElems works on types that fit in a vector register, like v16i8. And according to codes SExtVElems generates, it intends to sext lower bits in-place, which suggests SIGN_EXTEND_INREG.

Feb 17 2020, 7:33 PM · Restricted Project, Restricted Project

Feb 13 2020

lkail accepted D72685: [PowerPC] Exploit VSX rounding instrs for rint.

I believe the conversion of SNaN to QNaN is expected here. Note that the (current) C standard does not mention support signaling NaNs at all, and does not really ever mention them. This is planned to be fixed with the upcoming C2x version, which explicitly states that "rint" is supposed to implement the IEEE-754 "roundToIntegerExact" function. And that function, like most general functions defined by IEEE-754, is indeed defined to return a QNaN when the input is a SNaN.

Thanks for the information.

Feb 13 2020, 2:12 AM · Restricted Project

Feb 12 2020

lkail added a comment to D72685: [PowerPC] Exploit VSX rounding instrs for rint.

Is that specified by any doc/spec? Thanks.

Feb 12 2020, 7:49 PM · Restricted Project
lkail added a reviewer for D72685: [PowerPC] Exploit VSX rounding instrs for rint: uweigand.
Feb 12 2020, 1:11 AM · Restricted Project

Feb 10 2020

lkail committed rGd4c7ade1280f: [NFC] Fix typo. (authored by lkail).
[NFC] Fix typo.
Feb 10 2020, 10:00 PM
lkail added a comment to D72685: [PowerPC] Exploit VSX rounding instrs for rint.

Correct me if I'm wrong. rint(x) returns x if x is an NaN. However, intruction like XSRDPIC may turn SNAN to QNAN. Does it matter?

Feb 10 2020, 9:13 PM · Restricted Project

Jan 6 2020

lkail accepted D72125: [NFC] Move InPQueue into normal argument in releaseNode.

LGTM.

Jan 6 2020, 9:46 PM · Restricted Project
lkail abandoned D72311: [NFC] Add tests for `vshasigmaw`.

Bot works.

Jan 6 2020, 7:11 PM · Restricted Project
lkail created D72311: [NFC] Add tests for `vshasigmaw`.
Jan 6 2020, 7:10 PM · Restricted Project
lkail set the repository for D72250: [NFC][PowerPC] Refactor the tryAndWithMask() to rG LLVM Github Monorepo.
Jan 6 2020, 7:10 PM · Restricted Project, Restricted Project

Jan 2 2020

lkail added inline comments to D72069: [NFC] Add explicit instantiation to releaseNode.
Jan 2 2020, 3:11 AM · Restricted Project
lkail added a comment to D72069: [NFC] Add explicit instantiation to releaseNode.

Can you add more info about compilation errors and compiler version?

Jan 2 2020, 12:44 AM · Restricted Project

Dec 30 2019

lkail accepted D70758: [PowerPC][NFC] Rename record instructions to use _rec suffix instead of o.

LGTM. Given some verbosity can really help us to understand code quickly.

Dec 30 2019, 6:22 PM · Restricted Project
lkail committed rGcd2a73a9f01b: [MCP] Add stats for backward copy propagation. NFC. (authored by lkail).
[MCP] Add stats for backward copy propagation. NFC.
Dec 30 2019, 12:52 AM

Dec 29 2019

lkail accepted D71940: [PowerPC] Legalize saturating vector add/sub.

LGTM with one nit.

Dec 29 2019, 6:28 PM · Restricted Project

Dec 26 2019

lkail added a reviewer for D71886: [NFC] Add test case for load-insert-store pattern in InstCombine: shchenz.
Dec 26 2019, 11:02 PM · Restricted Project
lkail added inline comments to D71828: [InstCombine] Convert vector store to scalar store if only one element updated.
Dec 26 2019, 12:06 AM · Restricted Project

Dec 23 2019

lkail added a comment to D71833: [PowerPC] if value type is changed after folding rlwinm, stop folding.

I would suggest to retitle it to be more descriptive about code behavior, like 'Add check' blah blah. Describe what bugs it can fix in summary field.

Dec 23 2019, 6:49 PM · Restricted Project

Dec 22 2019

lkail committed rG9681dc9627b1: [PowerPC] Exploit `vrl(b|h|w|d)` to perform vector rotation (authored by lkail).
[PowerPC] Exploit `vrl(b|h|w|d)` to perform vector rotation
Dec 22 2019, 7:09 PM
lkail closed D71324: [PowerPC] Exploit `vrl(b|h|w|d)` to perform vector rotation.
Dec 22 2019, 7:09 PM · Restricted Project

Dec 18 2019

lkail added inline comments to D71589: [PowerPC] Adding a match pattern to recognize the and mask with RLWINM8.
Dec 18 2019, 10:34 PM · Restricted Project
lkail added inline comments to D71589: [PowerPC] Adding a match pattern to recognize the and mask with RLWINM8.
Dec 18 2019, 9:35 PM · Restricted Project
lkail added a comment to D71589: [PowerPC] Adding a match pattern to recognize the and mask with RLWINM8.
Dec 18 2019, 9:26 PM · Restricted Project

Dec 17 2019

lkail updated the diff for D71324: [PowerPC] Exploit `vrl(b|h|w|d)` to perform vector rotation.
Dec 17 2019, 5:29 PM · Restricted Project

Dec 16 2019

lkail updated the diff for D71324: [PowerPC] Exploit `vrl(b|h|w|d)` to perform vector rotation.

Add indentation in RUN lines.

Dec 16 2019, 10:53 PM · Restricted Project
lkail added inline comments to D71324: [PowerPC] Exploit `vrl(b|h|w|d)` to perform vector rotation.
Dec 16 2019, 10:18 PM · Restricted Project

Dec 12 2019

lkail added a comment to D71391: [PowerPC] Modify the hasSideEffects of some VSX instructions from 1 to 0.
In D71391#1781219, @Jim wrote:

Hi @ZhangKang
You may follow @lkail 's suggestion is fine.
I am not a PowerPC guy. Just give some suggestions.

Dec 12 2019, 1:39 AM · Restricted Project
lkail added a comment to D71391: [PowerPC] Modify the hasSideEffects of some VSX instructions from 1 to 0.

Hi @Jim we currently prefer to set this flag at a fine grained granularity, it's the purpose of sending a series of patches to perform such cleanups.

Dec 12 2019, 1:04 AM · Restricted Project

Dec 11 2019

lkail added a comment to D71390: [PowerPC] Modify the hasSideEffects of MTLR and MFLR from 1 to 0.
Dec 11 2019, 11:12 PM · Restricted Project
lkail accepted D71391: [PowerPC] Modify the hasSideEffects of some VSX instructions from 1 to 0.

LGTM.

Dec 11 2019, 10:43 PM · Restricted Project
lkail requested changes to D70771: [PowerPC] Replace the PPCISD:: SExtVElems with ISD::SIGN_EXTEND_INREG to leverage the combine rules .

Can you provide tests to demonstrate how ISD::SIGN_EXTEND_INREG works on PowerPC?

Dec 11 2019, 1:48 AM · Restricted Project, Restricted Project
lkail updated the diff for D71324: [PowerPC] Exploit `vrl(b|h|w|d)` to perform vector rotation.
Dec 11 2019, 1:25 AM · Restricted Project

Dec 10 2019

lkail created D71324: [PowerPC] Exploit `vrl(b|h|w|d)` to perform vector rotation.
Dec 10 2019, 7:49 PM · Restricted Project

Dec 9 2019

lkail added inline comments to D71002: [PowerPC] Exploitate the Vector Integer Average Instructions.
Dec 9 2019, 11:57 PM · Restricted Project
lkail added a comment to rG884351547da2: [PowerPC] Fix MI peephole optimization for splats.

@lkail When committing code for others, please follow the format described in Developer Policy. Thanks.
https://llvm.org/docs/DeveloperPolicy.html#attribution-of-changes

https://llvm.org/docs/DeveloperPolicy.html#commit-messages

Attribution of Changes should be in a separate line, after the end of the body, as simple as “Patch by John Doe.”. This is how we officially handle attribution, and there are automated processes that rely on this format.
Dec 9 2019, 4:07 PM
lkail added a comment to rG884351547da2: [PowerPC] Fix MI peephole optimization for splats.
Dec 9 2019, 4:07 PM

Dec 6 2019

lkail added a comment to D69497: [PowerPC] Fix MI peephole optimization for splats.

@vddvss Landed with https://reviews.llvm.org/rG884351547da27e76e14bef5fe20c3e3cb0e89acd, thanks for fixing it.

Dec 6 2019, 11:10 PM · Restricted Project
lkail committed rG884351547da2: [PowerPC] Fix MI peephole optimization for splats (authored by lkail).
[PowerPC] Fix MI peephole optimization for splats
Dec 6 2019, 11:02 PM
lkail closed D69497: [PowerPC] Fix MI peephole optimization for splats.
Dec 6 2019, 11:02 PM · Restricted Project

Dec 5 2019

lkail added a comment to D69497: [PowerPC] Fix MI peephole optimization for splats.

Hi @vddvss do you need help landing this patch?

Dec 5 2019, 9:19 PM · Restricted Project

Dec 4 2019

lkail committed rGb200c5180e8d: Reland [MachineCopyPropagation] Extend MCP to do trivial copy backward… (authored by lkail).
Reland [MachineCopyPropagation] Extend MCP to do trivial copy backward…
Dec 4 2019, 10:45 PM
lkail added a comment to rG75b3a1c318cc: [MachineCopyPropagation] Extend MCP to do trivial copy backward propagation.
clang-10: /home/buildslave/buildslave/clang-cmake-armv7-selfhost-neon/llvm/llvm/lib/CodeGen/MachineOperand.cpp:118: bool llvm::MachineOperand::isRenamable() const: Assertion `Register::isPhysicalRegister(getReg()) && "isRenamable should only be checked on physical registers"' failed.

It misses check if the register is zero before invoking isRenamable.

Dec 4 2019, 10:14 PM
lkail added a comment to rG75b3a1c318cc: [MachineCopyPropagation] Extend MCP to do trivial copy backward propagation.

It breaks http://lab.llvm.org:8011/builders/clang-cmake-armv7-selfhost/builds/2747, reverted. I'll investigate it.

Dec 4 2019, 9:03 PM
lkail committed rG3882edbe191f: Revert "[MachineCopyPropagation] Extend MCP to do trivial copy backward… (authored by lkail).
Revert "[MachineCopyPropagation] Extend MCP to do trivial copy backward…
Dec 4 2019, 8:53 PM
lkail added a reverting change for rG75b3a1c318cc: [MachineCopyPropagation] Extend MCP to do trivial copy backward propagation: rG3882edbe191f: Revert "[MachineCopyPropagation] Extend MCP to do trivial copy backward….
Dec 4 2019, 8:53 PM
lkail committed rG75b3a1c318cc: [MachineCopyPropagation] Extend MCP to do trivial copy backward propagation (authored by lkail).
[MachineCopyPropagation] Extend MCP to do trivial copy backward propagation
Dec 4 2019, 7:10 PM
lkail closed D67794: [MachineCopyPropagation] Extend MCP to do trivial copy backward propagation.
Dec 4 2019, 7:10 PM · Restricted Project
lkail added a comment to D70758: [PowerPC][NFC] Rename record instructions to use _rec suffix instead of o.

As a newbie of PowerPC backend, I think what @jsji proposed will ease my mind burden when faced with o ended instructions. I have read ISA-3.0 and know

A period (.) as the last character of an instruction
mnemonic means that the instruction *records* sta-
tus information in certain fields of the Condition
Register as a side effect of execution.

I can directly react to the effect of record-form instruction if it ends with _record. If _record is too long, how about _rec, just like X86's _alt ended instructions?

Dec 4 2019, 1:49 AM · Restricted Project

Dec 3 2019

lkail added a comment to D67794: [MachineCopyPropagation] Extend MCP to do trivial copy backward propagation.
Dec 3 2019, 6:36 PM · Restricted Project
lkail added inline comments to D67794: [MachineCopyPropagation] Extend MCP to do trivial copy backward propagation.
Dec 3 2019, 6:36 PM · Restricted Project
lkail added inline comments to D65793: [UpdateTestChecks] Update tests option.
Dec 3 2019, 12:52 AM · Restricted Project
lkail added a comment to D65793: [UpdateTestChecks] Update tests option.
Dec 3 2019, 12:52 AM · Restricted Project

Dec 2 2019

lkail added inline comments to D65793: [UpdateTestChecks] Update tests option.
Dec 2 2019, 11:57 PM · Restricted Project
lkail updated the diff for D67794: [MachineCopyPropagation] Extend MCP to do trivial copy backward propagation.

Rebased and add comment for invalidateRegister.

Dec 2 2019, 6:59 PM · Restricted Project
lkail added inline comments to D67794: [MachineCopyPropagation] Extend MCP to do trivial copy backward propagation.
Dec 2 2019, 6:40 PM · Restricted Project
lkail accepted D70374: [PowerPC] combine rlwinm + rlwinm to rlwinm.

LGTM. Thanks for fixing it. Personally, I would recommend Alive to verify peephole optimizations.

Dec 2 2019, 1:37 AM · Restricted Project

Nov 29 2019

lkail updated the diff for D67794: [MachineCopyPropagation] Extend MCP to do trivial copy backward propagation.
Nov 29 2019, 1:10 AM · Restricted Project
lkail added a comment to D67794: [MachineCopyPropagation] Extend MCP to do trivial copy backward propagation.

The RISC-V test changes look good. (I do not intend to review the optimisation implementation or other backend test changes).

Nov 29 2019, 1:10 AM · Restricted Project
lkail updated the diff for D67794: [MachineCopyPropagation] Extend MCP to do trivial copy backward propagation.

Rebased and fixed broken tests. Ready to be reviewed.

Nov 29 2019, 1:10 AM · Restricted Project

Nov 28 2019

lkail planned changes to D67794: [MachineCopyPropagation] Extend MCP to do trivial copy backward propagation.

Rebased and get broken tests, I'll investigate it and change this patch.

Nov 28 2019, 6:16 AM · Restricted Project

Nov 24 2019

lkail added a comment to rG29f6f9b2b2bf: [PowerPC] combine rlwinm+rlwinm to rlwinm combine x3 = rlwinm x3, 27, 5, 31 x3….

What it broke seems to be tests on Power8.

Nov 24 2019, 6:30 PM

Nov 20 2019

lkail added inline comments to D67258: Out-of-tree machine IR passes.
Nov 20 2019, 11:45 PM · Restricted Project

Nov 19 2019

lkail accepted D69601: [Power9] Implement the vector extend sign instruction pattern match.

LGTM considering patterns added for SIGN_EXTENDED_INREG also follow its atomic semantics.

Nov 19 2019, 7:18 PM · Restricted Project
lkail added a comment to D70223: [DAGCombine] Split vector load-update-store into single element stores.

I plan to move the two changed swap-le tests into another NFC patch, since they're not related to the logic.

I think all test cases change should be reflected so that reviewers can have a look if they are regressions.

Nov 19 2019, 1:32 AM · Restricted Project

Nov 18 2019

lkail updated the diff for D67794: [MachineCopyPropagation] Extend MCP to do trivial copy backward propagation.

Add test case involving RegMask.

Nov 18 2019, 10:48 PM · Restricted Project
lkail added a comment to D70373: [NFC] [PowerPC] Add volatile flag to a swap optimization test.

My concern is store and store volatile are so different in semantics that it might break original test intention.

Nov 18 2019, 7:20 PM · Restricted Project
lkail committed rG6ae7315746f1: Revert "[Bindings][Go] Fix go.test failure due to missing argument" (authored by lkail).
Revert "[Bindings][Go] Fix go.test failure due to missing argument"
Nov 18 2019, 6:07 PM
lkail added a reverting change for rG88235812a71d: [Bindings][Go] Fix go.test failure due to missing argument go.test failed with…: rG6ae7315746f1: Revert "[Bindings][Go] Fix go.test failure due to missing argument".
Nov 18 2019, 6:07 PM
lkail added a comment to rG88235812a71d: [Bindings][Go] Fix go.test failure due to missing argument go.test failed with….

Since d27a16eb392f39f9ee04ff5194b1eff3e189e6f8 reverts previous changes, LLVMDIBuilderCreateTypedef's signature becomes what it was. I'll revert this commit.

Nov 18 2019, 5:31 PM
lkail committed rG88235812a71d: [Bindings][Go] Fix go.test failure due to missing argument go.test failed with… (authored by lkail).
[Bindings][Go] Fix go.test failure due to missing argument go.test failed with…
Nov 18 2019, 5:13 PM
lkail closed D70360: [Bindings][Go] Fix go.test failure due to missing argument.
Nov 18 2019, 5:13 PM · Restricted Project

Nov 17 2019

lkail added a reviewer for D70360: [Bindings][Go] Fix go.test failure due to missing argument: bkramer.
Nov 17 2019, 10:51 PM · Restricted Project
lkail added a reviewer for D70360: [Bindings][Go] Fix go.test failure due to missing argument: CodaFi.
Nov 17 2019, 10:43 PM · Restricted Project

Nov 16 2019

lkail updated the summary of D70360: [Bindings][Go] Fix go.test failure due to missing argument.
Nov 16 2019, 6:14 PM · Restricted Project
lkail created D70360: [Bindings][Go] Fix go.test failure due to missing argument.
Nov 16 2019, 6:06 PM · Restricted Project

Nov 15 2019

lkail updated the diff for D67794: [MachineCopyPropagation] Extend MCP to do trivial copy backward propagation.
Nov 15 2019, 4:48 AM · Restricted Project
lkail updated the diff for D67794: [MachineCopyPropagation] Extend MCP to do trivial copy backward propagation.
Nov 15 2019, 4:19 AM · Restricted Project
lkail updated the diff for D67794: [MachineCopyPropagation] Extend MCP to do trivial copy backward propagation.
Nov 15 2019, 3:59 AM · Restricted Project
lkail updated the diff for D67794: [MachineCopyPropagation] Extend MCP to do trivial copy backward propagation.
Nov 15 2019, 3:41 AM · Restricted Project