Page MenuHomePhabricator

rovka (Diana Picus)
User

Projects

User does not belong to any projects.

User Details

User Since
Apr 21 2016, 3:27 AM (137 w, 3 d)

Recent Activity

Thu, Dec 6

rovka committed rL348472: [ARM GlobalISel] Nothing is legal for Thumb.
[ARM GlobalISel] Nothing is legal for Thumb
Thu, Dec 6, 1:31 AM
rovka committed rL348466: Fixup r348306: Require x86 for test.
Fixup r348306: Require x86 for test
Thu, Dec 6, 12:57 AM
rovka committed rLLD348466: Fixup r348306: Require x86 for test.
Fixup r348306: Require x86 for test
Thu, Dec 6, 12:57 AM

Wed, Dec 5

rovka committed rL348347: [ARM GlobalISel] Implement call lowering for Thumb2.
[ARM GlobalISel] Implement call lowering for Thumb2
Wed, Dec 5, 2:38 AM

Mon, Nov 26

rovka committed rL347545: [ARM GlobalISel] Support G_CTLZ and G_CTLZ_ZERO_UNDEF.
[ARM GlobalISel] Support G_CTLZ and G_CTLZ_ZERO_UNDEF
Mon, Nov 26, 3:10 AM
rovka committed rL347544: Fix typo in comment. NFC.
Fix typo in comment. NFC
Mon, Nov 26, 3:10 AM

Thu, Nov 22

rovka committed rL347447: [ARM GlobalISel] Add test for BFC. NFCI.
[ARM GlobalISel] Add test for BFC. NFCI
Thu, Nov 22, 1:57 AM

Wed, Nov 14

rovka committed rLLD346842: Fix r346747 and r346796.
Fix r346747 and r346796
Wed, Nov 14, 2:02 AM
rovka committed rL346842: Fix r346747 and r346796.
Fix r346747 and r346796
Wed, Nov 14, 2:00 AM

Feb 15 2018

rovka abandoned D40074: [GISel] Canonicalize constants to RHS for commutative operations.

This doesn't seem to be what we want to do in the long run.

Feb 15 2018, 4:24 AM

Feb 14 2018

rovka accepted D43195: [CMake] Add -fno-experimental-isel for testing.

Seems to pass now.

Feb 14 2018, 9:09 PM

Feb 13 2018

rovka added a comment to D43195: [CMake] Add -fno-experimental-isel for testing.

I'm not familiar enough with the CMake infrastructure to know if this does the trick for the openmp tests, but as I said on the bug report, I'm ok with this general direction.

Feb 13 2018, 6:36 AM
rovka added inline comments to D43119: [zorg] Add AArch32 global-isel bots.
Feb 13 2018, 5:58 AM

Feb 12 2018

rovka added inline comments to D43090: GlobalISel: IRTranslate llvm.fmuladd.* intrinsic.
Feb 12 2018, 1:36 AM
rovka added inline comments to D43119: [zorg] Add AArch32 global-isel bots.
Feb 12 2018, 1:19 AM

Feb 9 2018

rovka added inline comments to D43119: [zorg] Add AArch32 global-isel bots.
Feb 9 2018, 7:02 AM
rovka added inline comments to D43117: [zorg] Cleanup unnecessary options for ARM and AArch64 bots.
Feb 9 2018, 6:51 AM

Jan 31 2018

rovka committed rL323878: Fix formatting for r323876. NFC.
Fix formatting for r323876. NFC
Jan 31 2018, 7:18 AM
rovka added a comment to D42251: [globalisel][legalizer] Adapt LegalizerInfo to support inter-type dependencies and other things..

I've been playing a bit with this for ARM (r323876) and it's been pretty straightforward to update. I have a few observations that I thought I could share, but feel free to ignore them depending on how you intend to evolve this in the future.

Jan 31 2018, 7:14 AM
rovka committed rL323876: [ARM GlobalISel] Modernize LegalizerInfo. NFCI.
[ARM GlobalISel] Modernize LegalizerInfo. NFCI
Jan 31 2018, 6:57 AM

Jan 30 2018

rovka committed rL323732: [ARM GlobalISel] Add inst selector tests for G_SITOFP and G_UITOFP.
[ARM GlobalISel] Add inst selector tests for G_SITOFP and G_UITOFP
Jan 30 2018, 1:17 AM
rovka committed rL323731: [ARM GlobalISel] Map G_SITOFP and G_UITOFP.
[ARM GlobalISel] Map G_SITOFP and G_UITOFP
Jan 30 2018, 1:17 AM
rovka committed rL323730: [ARM GlobalISel] Legalize G_SITOFP and G_UITOFP.
[ARM GlobalISel] Legalize G_SITOFP and G_UITOFP
Jan 30 2018, 1:17 AM

Jan 29 2018

rovka committed rL323728: [ARM GlobalISel] Add inst selector tests for G_FPTOSI and G_FPTOUI.
[ARM GlobalISel] Add inst selector tests for G_FPTOSI and G_FPTOUI
Jan 29 2018, 11:58 PM
rovka committed rL323727: [ARM GlobalISel] Map G_FPTOSI and G_FPTOUI.
[ARM GlobalISel] Map G_FPTOSI and G_FPTOUI
Jan 29 2018, 11:58 PM
rovka committed rL323726: [ARM GlobalISel] Legalize G_FPTOSI and G_FPTOUI.
[ARM GlobalISel] Legalize G_FPTOSI and G_FPTOUI
Jan 29 2018, 11:58 PM

Jan 26 2018

rovka accepted D42565: [ARM][GISel] PR35965 Constrain RegClasses of nested instructions built from Dst Pattern.

LGTM, thanks!

Jan 26 2018, 3:31 AM

Jan 23 2018

rovka accepted D42040: [PATCH 5/6] [zorg] Add linaro-armv8-01-arm-* slaves.

Ok, LGTM.

Jan 23 2018, 4:46 AM

Jan 18 2018

rovka added inline comments to D42244: [globalisel] Introduce LegalityQuery to better encapsulate the legalizer decisions. NFC..
Jan 18 2018, 1:11 PM

Jan 17 2018

rovka committed rL322667: [ARM GlobalISel] Rename local variable. NFC.
[ARM GlobalISel] Rename local variable. NFC
Jan 17 2018, 7:27 AM
rovka committed rL322665: [ARM GlobalISel] Add instselect tests for G_FPEXT and G_FPTRUNC.
[ARM GlobalISel] Add instselect tests for G_FPEXT and G_FPTRUNC
Jan 17 2018, 7:02 AM
rovka committed rL322657: [ARM GlobalISel] Map G_FPEXT and G_FPTRUNC to FPR.
[ARM GlobalISel] Map G_FPEXT and G_FPTRUNC to FPR
Jan 17 2018, 6:15 AM
rovka committed rL322651: [ARM GlobalISel] Legalize G_FPEXT and G_FPTRUNC.
[ARM GlobalISel] Legalize G_FPEXT and G_FPTRUNC
Jan 17 2018, 5:35 AM

Jan 15 2018

rovka accepted D42041: [PATCH 6/6] [zorg] Don't use -DCMAKE_C_COMPILER*/-DCMAKE_CXX_COMPILER*.

LGTM.

Jan 15 2018, 6:10 AM
rovka added inline comments to D42040: [PATCH 5/6] [zorg] Add linaro-armv8-01-arm-* slaves.
Jan 15 2018, 5:50 AM
rovka accepted D42039: [PATCH 4/6] [zorg] Rename linaro-d05-01-* slaves to linaro-armv8-01-*.

LGTM, but I think you should commit the BOTO change separately.

Jan 15 2018, 5:37 AM
rovka accepted D42038: [PATCH 3/6] [zorg] Remove clang-cmake-thumbv7-a15 bot.

LGTM.

Jan 15 2018, 5:21 AM
rovka accepted D42037: [PATCH 2/6] [zorg] Use slave's "jobs" setting for ARM 32-bit bots..

LGTM, but I have a couple of nitpicks:

  • Why is this PATCH 2/6? It doesn't seem related to either 1/6 or 3/6 and can be committed independently
  • The summary just repeats the title. A better summary would be something along the lines of "Don't force the number of jobs on the ARM builders. This makes it possible to fall back on the number of jobs configured for the slave instead."
Jan 15 2018, 5:20 AM

Jan 12 2018

rovka committed rL322368: [ARM GlobalISel] Add inst selector tests for G_FMA.
[ARM GlobalISel] Add inst selector tests for G_FMA
Jan 12 2018, 4:45 AM
rovka committed rL322367: [ARM GlobalISel] Map G_FMA to FPR.
[ARM GlobalISel] Map G_FMA to FPR
Jan 12 2018, 4:07 AM
rovka committed rL322366: [ARM GlobalISel] Legalize G_FMA.
[ARM GlobalISel] Legalize G_FMA
Jan 12 2018, 3:32 AM

Jan 10 2018

rovka committed rL322170: [ARM GlobalISel] Add inst selector tests for G_FNEG s32 and s64.
[ARM GlobalISel] Add inst selector tests for G_FNEG s32 and s64
Jan 10 2018, 3:14 AM
rovka committed rL322169: [ARM GlobalISel] Map G_FNEG to the FPR bank.
[ARM GlobalISel] Map G_FNEG to the FPR bank
Jan 10 2018, 3:14 AM
rovka committed rL322168: [ARM GlobalISel] Legalize G_FNEG for s32 and s64.
[ARM GlobalISel] Legalize G_FNEG for s32 and s64
Jan 10 2018, 2:46 AM
rovka committed rL322164: [ARM GlobalISel] Legalize s32/s64 G_FCONSTANT.
[ARM GlobalISel] Legalize s32/s64 G_FCONSTANT
Jan 10 2018, 2:03 AM
rovka committed rL322162: [ARM GlobalISel] Legalize G_CONSTANT for scalars > 32 bits.
[ARM GlobalISel] Legalize G_CONSTANT for scalars > 32 bits
Jan 10 2018, 1:33 AM

Jan 4 2018

rovka committed rL321797: [ARM GlobalISel] Select G_PHI.
[ARM GlobalISel] Select G_PHI
Jan 4 2018, 5:10 AM
rovka committed rL321796: [ARM GlobalISel] Add RegBankSelect tests for G_PHI.
[ARM GlobalISel] Add RegBankSelect tests for G_PHI
Jan 4 2018, 5:10 AM
rovka committed rL321795: [ARM GlobalISel] Legalize scalar G_PHI.
[ARM GlobalISel] Legalize scalar G_PHI
Jan 4 2018, 5:10 AM
rovka committed rL321793: [ARM GlobalISel] Fix selection of pointer constants.
[ARM GlobalISel] Fix selection of pointer constants
Jan 4 2018, 2:56 AM

Dec 22 2017

rovka committed rL321356: [ARM GlobalISel] Support G_INTTOPTR and G_PTRTOINT for s32.
[ARM GlobalISel] Support G_INTTOPTR and G_PTRTOINT for s32
Dec 22 2017, 5:06 AM
rovka committed rL321354: [ARM GlobalISel] Support pointer constants.
[ARM GlobalISel] Support pointer constants
Dec 22 2017, 3:10 AM

Dec 20 2017

rovka committed rL321165: [ARM GlobalISel] Fix assertion in RegBankSelect.
[ARM GlobalISel] Fix assertion in RegBankSelect
Dec 20 2017, 3:28 AM

Dec 18 2017

rovka committed rL320980: [ARM GlobalISel] Fix G_(UN)MERGE_VALUES handling after r319524.
[ARM GlobalISel] Fix G_(UN)MERGE_VALUES handling after r319524
Dec 18 2017, 5:23 AM

Dec 11 2017

rovka committed rL320367: [zorg] Cleanup CC/CXX settings for Linaro Arm/AArch64 bots.
[zorg] Cleanup CC/CXX settings for Linaro Arm/AArch64 bots
Dec 11 2017, 6:32 AM
rovka closed D40608: [PATCH 2/3] [zorg] Cleanup CC/CXX settings for Linaro Arm/AArch64 bots by committing rL320367: [zorg] Cleanup CC/CXX settings for Linaro Arm/AArch64 bots.
Dec 11 2017, 6:32 AM
rovka committed rL320364: [zorg] Add linaro-d05-* builders.
[zorg] Add linaro-d05-* builders
Dec 11 2017, 6:12 AM
rovka closed D40607: [zorg] Add linaro-d05-* builders by committing rL320364: [zorg] Add linaro-d05-* builders.
Dec 11 2017, 6:12 AM
rovka committed rL320362: [ARM GlobalISel] Add test for a MOVTi16 pattern. NFC.
[ARM GlobalISel] Add test for a MOVTi16 pattern. NFC
Dec 11 2017, 5:29 AM
rovka committed rL320352: [ARM GlobalISel] Add tests for PKHBT and PKHTB.
[ARM GlobalISel] Add tests for PKHBT and PKHTB
Dec 11 2017, 3:45 AM
rovka accepted D40789: [GlobalISel] Disable GISel for big endian.

LGTM until we can handle big endian. Sorry about the delay.

Dec 11 2017, 2:07 AM

Dec 4 2017

rovka accepted D40607: [zorg] Add linaro-d05-* builders.

@rovka , slaves linaro-d05-01-* will run in different containers on the same machine (linaro-d05-01). In this context there is no need to repurpose a slave.
I plan to add a second machine linaro-d05-02 that would host the same set of slaves/builders for redundancy.

Dec 4 2017, 1:13 AM

Dec 1 2017

rovka added a reviewer for D40609: [PATCH 1/6] [zorg] Remove obsolete workarounds for Arm/AArch64 libcxx bots: mclow.lists.
Dec 1 2017, 3:07 AM
rovka accepted D40608: [PATCH 2/3] [zorg] Cleanup CC/CXX settings for Linaro Arm/AArch64 bots.

LGTM, thanks.

Dec 1 2017, 3:04 AM
rovka added a comment to D40607: [zorg] Add linaro-d05-* builders.

This looks fine, but I think it would be better if the slave names weren't tied to what is currently running on them, but rather to the machine configurations. E.g. if there's no difference in the environment of linaro-d05-01-quick and linaro-d05-global-isel, they should just be linaro-d05-01 and linaro-d05-02. But for instance if the machines running the libcxx bots need something special, then they can be linaro-d05-libcxx-01 and linaro-d05-libcxx-02. That way, if we want to repurpose a slave to run a different builder, it can keep its name without looking awkward, and it's also easy for us to tell which machines have different environments. What do you think?

Dec 1 2017, 2:45 AM

Nov 30 2017

rovka committed rL319428: [ARM GlobalISel] Bail out for byval.
[ARM GlobalISel] Bail out for byval
Nov 30 2017, 4:24 AM
rovka added a comment to D40074: [GISel] Canonicalize constants to RHS for commutative operations.

FWIW, if we did end up introducing canonicalization, we'd prefer it to be a separate pass (so that it can be skipped) and we'd like it to be possible for targets to opt out of (at least some) canonicalizations. We've had a few occasions where we have to repeatedly de-canonicalize in DAGISel.

Nov 30 2017, 4:18 AM

Nov 29 2017

rovka committed rL319313: [ARM GlobalISel] Fix selecting G_BRCOND.
[ARM GlobalISel] Fix selecting G_BRCOND
Nov 29 2017, 6:20 AM
rovka updated subscribers of D40532: [globalisel][tablegen] Fix PR35375 by sign-extending the table value to match getConstantVRegVal().

I think GIM_CheckLiteralInt might have the same issue but I don't think there's any existing rules that would detect the problem. As far as I know X86 is the only user and only uses 0 and 1. We should probably change it to match GIM_CheckConstantInt if only for consistency. Do you agree?

Nov 29 2017, 3:58 AM

Nov 28 2017

rovka accepted D40532: [globalisel][tablegen] Fix PR35375 by sign-extending the table value to match getConstantVRegVal().

I don't know if this fixes every edge case that we currently have with and/or immediates in DAGISel, but it works for the PKHBT and doesn't seem to break anything else, so in the interest of simplicity it's probably a good route to take. You should add a summary when you commit though.

Nov 28 2017, 3:37 AM

Nov 23 2017

rovka committed rL318915: [ARM GlobalISel] Support G_FDIV for s32 and s64.
[ARM GlobalISel] Support G_FDIV for s32 and s64
Nov 23 2017, 5:27 AM
rovka committed rL318910: [ARM GlobalISel] Support G_FMUL for s32 and s64.
[ARM GlobalISel] Support G_FMUL for s32 and s64
Nov 23 2017, 4:44 AM

Nov 21 2017

rovka committed rL318747: [ARM GlobalISel] Add comment for r318398. NFC..
[ARM GlobalISel] Add comment for r318398. NFC.
Nov 21 2017, 2:17 AM

Nov 20 2017

rovka committed rL318643: [ARM GlobalISel] Add test for RSBri. NFC.
[ARM GlobalISel] Add test for RSBri. NFC
Nov 20 2017, 3:05 AM
rovka committed rL318642: [ARM GlobalISel] Clean up binary operator tests. NFC.
[ARM GlobalISel] Clean up binary operator tests. NFC
Nov 20 2017, 2:36 AM

Nov 16 2017

rovka committed rL318398: [ARM GlobalISel] Add tests for BIC. NFC.
[ARM GlobalISel] Add tests for BIC. NFC
Nov 16 2017, 5:33 AM
rovka committed rL318393: [ARM GlobalISel] Add tests for REVSH patterns. NFC.
[ARM GlobalISel] Add tests for REVSH patterns. NFC
Nov 16 2017, 4:30 AM
rovka added a comment to D40074: [GISel] Canonicalize constants to RHS for commutative operations.

If this is the first canonicalization (of potentially many to come) in the IRTranslator, would it make sense to try and document the canonical representation(s) somewhere and/or somehow?

Even better would be a pass that verified canonical order, that could be inserted between other passes in debug mode to verify that optimisations preserve canonical-ness

Nov 16 2017, 3:45 AM

Nov 15 2017

rovka created D40074: [GISel] Canonicalize constants to RHS for commutative operations.
Nov 15 2017, 5:51 AM

Nov 14 2017

rovka committed rL318146: [ARM GlobalISel] Remove C++ code for G_CONSTANT.
[ARM GlobalISel] Remove C++ code for G_CONSTANT
Nov 14 2017, 3:20 AM

Nov 13 2017

rovka added a comment to D39267: [GISel]: Change Legalization from top down to bottom up + DCE.

Hi, I've updated the ARM tests, and you'll need to also commit this together with your patch: https://reviews.llvm.org/differential/diff/122657/

Nov 13 2017, 8:18 AM
rovka committed rL318047: [ARM GlobalISel] Update legalizer test.
[ARM GlobalISel] Update legalizer test
Nov 13 2017, 8:03 AM

Nov 10 2017

rovka added inline comments to D39808: [ARM GlobalISel] Fix call opcode for Thumb .
Nov 10 2017, 10:04 AM

Nov 7 2017

rovka accepted D39747: [globalisel][tablegen] Generate rule coverage and use it to identify untested rules.

Awesome, I could use something like this. LGTM with a few nits.

Nov 7 2017, 10:44 PM
rovka added a comment to D39267: [GISel]: Change Legalization from top down to bottom up + DCE.

Sure, will do.

Nov 7 2017, 8:47 PM
rovka added inline comments to D39592: [ARM|GlobalISel] : Adding legalizer tests for Thumb.
Nov 7 2017, 4:04 AM

Nov 6 2017

rovka added inline comments to D39592: [ARM|GlobalISel] : Adding legalizer tests for Thumb.
Nov 6 2017, 3:05 AM

Nov 3 2017

rovka added a comment to D39592: [ARM|GlobalISel] : Adding legalizer tests for Thumb.

I think it would be simpler to add a RUN line to the existing test (you can rename it to just legalizer.mir if you think the current name would be misleading).

Nov 3 2017, 8:52 AM
rovka added a comment to D39592: [ARM|GlobalISel] : Adding legalizer tests for Thumb.

Hi Javed,

Nov 3 2017, 6:10 AM
rovka committed rL317313: [globalisel][tablegen] Skip src child predicates.
[globalisel][tablegen] Skip src child predicates
Nov 3 2017, 3:30 AM
rovka closed D39554: [globalisel][tablegen] Skip src child predicates by committing rL317313: [globalisel][tablegen] Skip src child predicates.
Nov 3 2017, 3:30 AM
rovka committed rL317312: [ARM GlobalISel] Move the check for Thumb higher up.
[ARM GlobalISel] Move the check for Thumb higher up
Nov 3 2017, 3:30 AM

Nov 2 2017

rovka added a comment to D39554: [globalisel][tablegen] Skip src child predicates.

Sorry about the indirect test, but it's a bit difficult to test directly since this is restricting functionality rather than adding it. Are we interested in having tests for failed imports? I haven't seen any, but I may have missed them...

Nov 2 2017, 8:14 AM
rovka created D39554: [globalisel][tablegen] Skip src child predicates.
Nov 2 2017, 8:09 AM

Nov 1 2017

rovka added a comment to D39415: [ARMISelLowering] Better handling of NEON load/store for sequential memory regions.

I tried to give this a run [1] on top of r317072 on an NVIDIA TK1 (Cortex-A15) and I'm getting some failures in the test-suite (in 42 of the benchmarks) along the lines of [2]. I think we should fix those before worrying about performance numbers...

Nov 1 2017, 6:24 AM

Oct 30 2017

rovka accepted D39261: [GlobalISel|ARM] : Allow legalizing G_FSUB.

LGTM, thanks!

Oct 30 2017, 5:00 AM
rovka committed rL316897: [ARM GlobalISel] Fixup r316572. NFC.
[ARM GlobalISel] Fixup r316572. NFC
Oct 30 2017, 4:58 AM

Oct 26 2017

rovka added inline comments to D39264: [GISel]: Make the combiner check if shifts are legal before creating them.
Oct 26 2017, 2:58 AM

Oct 25 2017

rovka committed rL316573: [ARM GlobalISel] Remove redundant testcases. NFC.
[ARM GlobalISel] Remove redundant testcases. NFC
Oct 25 2017, 5:22 AM
rovka committed rL316572: [ARM GlobalISel] Update test after r316479. NFC.
[ARM GlobalISel] Update test after r316479. NFC
Oct 25 2017, 5:22 AM