Page MenuHomePhabricator

rovka (Diana Picus)
User

Projects

User does not belong to any projects.

User Details

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

Recent Activity

Tue, Oct 6

rovka updated subscribers of D87085: Add flang out of tree buildbot.

Hi Sameeran,

Tue, Oct 6, 2:59 AM · Restricted Project

Sep 28 2020

rovka committed rZORG073b104115cd: Add flang out of tree buildbot (authored by rovka).
Add flang out of tree buildbot
Sep 28 2020, 2:07 AM
rovka closed D87085: Add flang out of tree buildbot.
Sep 28 2020, 2:07 AM · Restricted Project

Sep 24 2020

rovka updated the diff for D87085: Add flang out of tree buildbot.

Hi Galina, those are all good comments, thanks! I think I fixed them up, does it look ok now?

Sep 24 2020, 5:11 AM · Restricted Project

Sep 21 2020

rovka added a comment to D87085: Add flang out of tree buildbot.

Ping :)

Sep 21 2020, 4:03 AM · Restricted Project

Sep 14 2020

rovka updated the diff for D87085: Add flang out of tree buildbot.

Hi Galina, thanks for having a look!

Sep 14 2020, 1:36 AM · Restricted Project

Sep 9 2020

rovka added a comment to D87083: [cmake] Use absolute paths for modules search.

Great, thanks! :)

Sep 9 2020, 5:00 AM · Restricted Project, Restricted Project
rovka committed rGd4b88ac1658d: [cmake] Use absolute paths for modules search (authored by rovka).
[cmake] Use absolute paths for modules search
Sep 9 2020, 5:00 AM
rovka closed D87083: [cmake] Use absolute paths for modules search.
Sep 9 2020, 5:00 AM · Restricted Project, Restricted Project

Sep 7 2020

rovka added a comment to D87083: [cmake] Use absolute paths for modules search.

Hi Pete,

Sep 7 2020, 4:16 AM · Restricted Project, Restricted Project

Sep 4 2020

rovka added a comment to D87083: [cmake] Use absolute paths for modules search.

Thanks for trying this out!

Sep 4 2020, 2:05 AM · Restricted Project, Restricted Project

Sep 3 2020

rovka requested review of D87085: Add flang out of tree buildbot.
Sep 3 2020, 5:13 AM · Restricted Project
rovka requested review of D87083: [cmake] Use absolute paths for modules search.
Sep 3 2020, 4:58 AM · Restricted Project, Restricted Project

Sep 2 2020

rovka committed rZORG4720197cd417: Add flang to clang-cmake-aarch64-full (authored by rovka).
Add flang to clang-cmake-aarch64-full
Sep 2 2020, 1:44 AM
rovka closed D86869: [zorg] Add flang to clang-cmake-aarch64-full.
Sep 2 2020, 1:44 AM

Sep 1 2020

rovka committed rZORG98990a0a6b3d: Fix typos in builders descriptions (authored by rovka).
Fix typos in builders descriptions
Sep 1 2020, 3:59 AM
rovka closed D86868: Fix typos in builder descriptions.
Sep 1 2020, 3:59 AM

Aug 31 2020

rovka requested review of D86869: [zorg] Add flang to clang-cmake-aarch64-full.
Aug 31 2020, 3:48 AM
rovka requested review of D86868: Fix typos in builder descriptions.
Aug 31 2020, 3:39 AM

Jul 16 2020

rovka added a comment to D83361: [LLVM] Add libatomic load/store functions to TargetLibraryInfo.

Thanks for the update. This looks fine to me as is, but I'll defer the final LGTM to someone that knows this area a bit better.

Jul 16 2020, 1:37 AM · Restricted Project

Jul 10 2020

rovka added a comment to D83361: [LLVM] Add libatomic load/store functions to TargetLibraryInfo.

Hi again,

Jul 10 2020, 4:18 AM · Restricted Project

Jul 8 2020

rovka added a comment to D83361: [LLVM] Add libatomic load/store functions to TargetLibraryInfo.

Don't you also have to set as Available/Unavailable when initializing the TLI?

Jul 8 2020, 1:47 AM · Restricted Project

Jun 25 2020

rovka added a comment to D82064: [ARM64] Add QEMU testing environment setup guide for SVE testing.

Hi Omair,

Jun 25 2020, 4:42 AM · Restricted Project

Jun 15 2020

rovka abandoned D81830: [compiler-rt] Fix intermittent failure with instrprof-gcov-multithread_fork.test on 32-bit arm.
Jun 15 2020, 9:13 AM
rovka added a comment to D81830: [compiler-rt] Fix intermittent failure with instrprof-gcov-multithread_fork.test on 32-bit arm.

@calixte: Does this affect the intention of the test in any way?

Jun 15 2020, 2:41 AM
rovka added a comment to D81650: [compiler-rt] Fix intermittent failure with instrprof-gcov-multithread_fork.test on 32-bit arm.

Um, I tried to upload a diff here but it didn't work, so I just created a new revision: https://reviews.llvm.org/D81830
Hope that's ok (if not feel free to re-upload here).

Jun 15 2020, 2:41 AM · Restricted Project
rovka created D81830: [compiler-rt] Fix intermittent failure with instrprof-gcov-multithread_fork.test on 32-bit arm.
Jun 15 2020, 2:41 AM
rovka added a comment to D81650: [compiler-rt] Fix intermittent failure with instrprof-gcov-multithread_fork.test on 32-bit arm.

Hi! This doesn't seem to be enough. I suspect this is because the exception is thrown from one of the other threads rather than the main thread. Maybe f also needs a try-catch block? I'm going to give it a few runs and report back. EDIT: Actually I think I'll move the try-catch closer to the root of the problem, in launcher. I'll post if that worked.

Jun 15 2020, 2:08 AM · Restricted Project

Jun 11 2020

rovka added a comment to D78477: [profile] Don't crash when forking in several threads.

Ping!
https://bugs.llvm.org/show_bug.cgi?id=46092

Jun 11 2020, 3:01 AM · Restricted Project, Restricted Project, Restricted Project

Jun 4 2020

rovka added a comment to D78477: [profile] Don't crash when forking in several threads.

Hi! I think this commit is causing some instability on the 10.0.1 branch on 32-bit arm: https://bugs.llvm.org/show_bug.cgi?id=46092
Can you have a look?

Jun 4 2020, 11:00 AM · Restricted Project, Restricted Project, Restricted Project

Jun 2 2020

rovka added inline comments to D80919: [CodeGen] Fix warnings in getCopyFromPartsVector.
Jun 2 2020, 5:31 AM · Restricted Project

May 4 2020

rovka added a comment to D79284: [llvm-objdump][ARM] Print inline relocations when dumping ARM data.

Hi, I like that we're looping in only one place now, but I have 2 nitpicks:

  1. The formatting changes are pretty distracting, I think you should commit them separately.
  2. The commit message should explain what the problem is and why this change fixes it. Just referencing a bug number, where there haven't even been any discussions, is not very helpful.

Thanks!

May 4 2020, 3:42 AM · Restricted Project

Jan 29 2020

rovka accepted D73459: [ARM] Add documentation for -march= and -mfpu= command line options.

Thanks, LGTM!

Jan 29 2020, 12:41 AM · Restricted Project

Jan 28 2020

rovka added a comment to D73459: [ARM] Add documentation for -march= and -mfpu= command line options.

Just a few nits.

Jan 28 2020, 12:52 AM · Restricted Project

Dec 3 2019

rovka committed rG057626b43938: Fixup 6d18e53: xfail TestShowLocationDwarf5.py properly (authored by rovka).
Fixup 6d18e53: xfail TestShowLocationDwarf5.py properly
Dec 3 2019, 2:59 AM
rovka committed rG6d18e5366c9a: Mark some tests as xfail on AArch64 Linux (authored by rovka).
Mark some tests as xfail on AArch64 Linux
Dec 3 2019, 2:00 AM

Nov 19 2019

rovka committed rGbb7c8e984f87: Mark PR44037 tests as XFAIL on AArch64 Linux dwo (authored by rovka).
Mark PR44037 tests as XFAIL on AArch64 Linux dwo
Nov 19 2019, 1:56 AM
rovka closed D70381: Mark PR44037 tests as XFAIL on AArch64 Linux dwo.
Nov 19 2019, 1:55 AM · Restricted Project
rovka updated subscribers of D70381: Mark PR44037 tests as XFAIL on AArch64 Linux dwo.

Looks fine to me, but I am not running aarch64 tests. You might want to check of @omjavaid is running them and whether he's ok with that. Otherwise, you can just manage the decorators yourself without any special reviews.

Nov 19 2019, 12:57 AM · Restricted Project

Nov 18 2019

rovka updated the diff for D70381: Mark PR44037 tests as XFAIL on AArch64 Linux dwo.
Nov 18 2019, 3:41 AM · Restricted Project
rovka created D70381: Mark PR44037 tests as XFAIL on AArch64 Linux dwo.
Nov 18 2019, 2:56 AM · Restricted Project

Nov 15 2019

rovka committed rG5f0c3bad2f03: Fix TestFormatters.py stepping too far (authored by rovka).
Fix TestFormatters.py stepping too far
Nov 15 2019, 5:26 AM
rovka closed D70303: Fix TestFormatters.py stepping too far.
Nov 15 2019, 5:26 AM · Restricted Project
rovka accepted D66871: [SVE] MVT scalable size queries.

I don't see anything else wrong with this. LGTM if you rename the LE predicate.

Nov 15 2019, 5:16 AM · Restricted Project
rovka updated the diff for D70303: Fix TestFormatters.py stepping too far.

Addressed @labath's comment (test run in progress). Thanks for having a look!

Nov 15 2019, 4:19 AM · Restricted Project
rovka created D70303: Fix TestFormatters.py stepping too far.
Nov 15 2019, 3:59 AM · Restricted Project

Nov 14 2019

rovka committed rGe03a06b348ba: Fix typos in docs. NFC (authored by rovka).
Fix typos in docs. NFC
Nov 14 2019, 3:26 AM

Nov 13 2019

rovka added a comment to D66871: [SVE] MVT scalable size queries.

Regarding the change to return const, I'm not convinced that's a good idea (we actually have a clang-tidy check that warns about that). I think it would be better to either name those temporaries or use std::make_tuple instead of std::tie (whichever you prefer).

Nov 13 2019, 3:47 AM · Restricted Project

Nov 12 2019

rovka committed rG7f1dcc8952e6: [InstCombine] Skip scalable vectors in combineLoadToOperationType (authored by rovka).
[InstCombine] Skip scalable vectors in combineLoadToOperationType
Nov 12 2019, 3:29 AM
rovka closed D70075: [InstCombine] Skip scalable vectors in combineLoadToOperationType.
Nov 12 2019, 3:29 AM · Restricted Project
rovka updated the diff for D70075: [InstCombine] Skip scalable vectors in combineLoadToOperationType.

Update test checks. Also brush it up a bit so it fits better with the rest of the tests in the file.

Nov 12 2019, 1:33 AM · Restricted Project
rovka added inline comments to D70075: [InstCombine] Skip scalable vectors in combineLoadToOperationType.
Nov 12 2019, 1:32 AM · Restricted Project

Nov 11 2019

rovka created D70075: [InstCombine] Skip scalable vectors in combineLoadToOperationType.
Nov 11 2019, 5:02 AM · Restricted Project

Nov 5 2019

rovka accepted D69644: [globalisel][docs] Add KnownBits Analysis documentation.

LGTM

Nov 5 2019, 12:49 AM · Restricted Project

Nov 4 2019

rovka accepted D69545: [globalisel][docs] Rework GMIR documentation and add an early GenericOpcode reference.

The text itself LGTM.

Nov 4 2019, 1:27 AM · Restricted Project
rovka added inline comments to D69644: [globalisel][docs] Add KnownBits Analysis documentation.
Nov 4 2019, 1:09 AM · Restricted Project
rovka accepted D69734: [globalisel] Rename G_GEP to G_PTR_ADD.

LGTM.

Nov 4 2019, 1:09 AM · Restricted Project

Oct 31 2019

rovka added a comment to D69545: [globalisel][docs] Rework GMIR documentation and add an early GenericOpcode reference.

Just a few more typos and stuff.

Oct 31 2019, 2:58 AM · Restricted Project

Oct 29 2019

rovka added inline comments to D66871: [SVE] MVT scalable size queries.
Oct 29 2019, 3:37 AM · Restricted Project
rovka added inline comments to D69545: [globalisel][docs] Rework GMIR documentation and add an early GenericOpcode reference.
Oct 29 2019, 3:09 AM · Restricted Project
rovka accepted D69457: [globalisel][docs] Rewrite the IRTranslator documentation.

LGTM.

Oct 29 2019, 2:23 AM · Restricted Project
rovka accepted D69456: [globalisel][docs] Rewrite the pipeline overview.

Thanks, looks great!

Oct 29 2019, 2:13 AM · Restricted Project

Oct 28 2019

rovka added inline comments to D69457: [globalisel][docs] Rewrite the IRTranslator documentation.
Oct 28 2019, 2:30 AM · Restricted Project
rovka added a comment to D69456: [globalisel][docs] Rewrite the pipeline overview.

Thanks for writing this up! I just have a few suggestions and one small bug to point out (I can't figure out how to comment on a png file, so I'll write it here).

Oct 28 2019, 2:14 AM · Restricted Project

Oct 18 2019

rovka accepted D69064: [ARM] Add dependency on GlobalISel for unit tests to fix shared libs build.

Thanks, LGTM then!

Oct 18 2019, 9:06 AM · Restricted Project

Oct 17 2019

rovka added a comment to D69064: [ARM] Add dependency on GlobalISel for unit tests to fix shared libs build.

Could you be a little more specific? I just ran a build with BUILD_SHARED_LIBS=ON and it seemed to work.

Oct 17 2019, 1:57 AM · Restricted Project

Oct 2 2019

rovka accepted D53137: Scalable vector core instruction support + size queries.

This looks good to me, maybe wait a while to see if anyone else has any further comments.

Oct 2 2019, 3:06 AM · Restricted Project, Restricted Project

Oct 1 2019

rovka committed rGc2c377ea584d: Fixup r373278: Move test to X86 directory (authored by rovka).
Fixup r373278: Move test to X86 directory
Oct 1 2019, 2:27 AM
rovka committed rL373314: Fixup r373278: Move test to X86 directory.
Fixup r373278: Move test to X86 directory
Oct 1 2019, 2:27 AM

Sep 25 2019

rovka accepted D67732: Fix arm build failure with libgcc.

I'm not very familiar with this area, but it seems like a sensible fix.

Sep 25 2019, 1:25 AM · Restricted Project

Sep 24 2019

rovka added a comment to D53137: Scalable vector core instruction support + size queries.

I suspect 'ScalableSize' is the wrong term now; 'TypeSize' may be better. Thoughts?

Sep 24 2019, 2:26 AM · Restricted Project, Restricted Project

Sep 16 2019

rovka added inline comments to D67549: [IntrinsicEmitter] Add overloaded types for SVE intrinsics (Subdivide2 & Subdivide4).
Sep 16 2019, 9:53 AM · Restricted Project
rovka accepted D66524: [SVE][Inline-Asm] Add constraints for SVE predicate registers.

I think all the outstanding comments have been addressed. LGTM.

Sep 16 2019, 2:05 AM · Restricted Project

Sep 4 2019

rovka committed rL370867: Request commit access for rovka.
Request commit access for rovka
Sep 4 2019, 3:03 AM

Sep 2 2019

rovka added a comment to D66524: [SVE][Inline-Asm] Add constraints for SVE predicate registers.

Just some drive-by suggestions :)

Sep 2 2019, 2:24 AM · Restricted Project

Aug 30 2019

rovka added a comment to D53137: Scalable vector core instruction support + size queries.

Does anyone like Sander's suggestion to make ScalableSize (or whatever we end up naming it) the return value for all size queries and provide an overloaded cast operator to transparently work with existing code comparing against unsigned values? Or is it preferable to keep the current split?

Aug 30 2019, 1:28 AM · Restricted Project, Restricted Project

Aug 1 2019

rovka added a comment to D53137: Scalable vector core instruction support + size queries.

Several more general comments:

  • Should all the getXSize assert when called on a scalable type? I see that for MVT::getSizeInBits and for Type::getPrimitiveSize, but not for the others. This should also be made clear in the comments for each of them.
  • Great test for the IR, thanks!
  • I don't see any test for the CodeGen stuff though. Is it possible to add one? (If not, maybe add the changes to EVT etc when we can actually test them).
  • Ditto for TableGen (or if that's too difficult/hairy to test, just update the commit message to explain exactly why the change belongs in this patch).
Aug 1 2019, 5:01 AM · Restricted Project, Restricted Project

Jul 31 2019

rovka committed rG7117881fdb2f: [docs] Add cmake to Software requirements (authored by rovka).
[docs] Add cmake to Software requirements
Jul 31 2019, 1:49 AM
rovka committed rL367395: [docs] Add cmake to Software requirements.
[docs] Add cmake to Software requirements
Jul 31 2019, 1:49 AM
rovka closed D65438: [docs] Add cmake to Software requirements.
Jul 31 2019, 1:49 AM · Restricted Project

Jul 30 2019

rovka created D65438: [docs] Add cmake to Software requirements.
Jul 30 2019, 4:49 AM · Restricted Project
rovka added a comment to D65436: [docs] Add note about git version to git-llvm section.

Some people might be interested just in compiling LLVM (e.g. fetching some release sources from releases.llvm.org and building them). We should probably keep the package list here focused on that. If we want to add git at all, we should probably mention it somewhere else. I'm not sure where that would be - I suppose somewhere where it makes sense to worry about the minimum version, e.g. where the git-llvm script is concerned. Do we have any other reason to bind ourselves to a certain minimum version of git?

Jul 30 2019, 4:32 AM · Restricted Project, Restricted Project

Jul 26 2019

rovka accepted D65321: [MIPS GlobalISel] Fix check for void return during lowerCall.

LGTM

Jul 26 2019, 1:09 AM · Restricted Project

Jul 23 2019

rovka added a comment to D53137: Scalable vector core instruction support + size queries.

Hi Diana,

Thanks for the comments.

I think this patch is difficult to review. It covers many different source files with only a small unit test to check the correctness. This isn't very robust against future changes and it makes it hard to know exactly what is and isn't supported.

Yeah, I was worried about that -- this is basically the size queries alone without anything actually using scalable vectors. It demonstrates roughly where changes will be needed, but doesn't actually change the surrounding code to use e.g. getElementCount instead of getNumElements.

I would find it much easier to review with an incremental strategy based on regression tests. For instance, with ToT opt, the attached testcase fails (error: '%r' defined with type '<4 x i1>' but expected '<vscale x 4 x i1>'). I would add a patch to fix that, and maybe other similar, really simple cases. We could then proceed to more complex examples, run some of the passes that come after the vectorizer on them, and progressively fix the places required to make them pass, with focused tests for each hurdle that we run into. It shouldn't be too hard to reduce such snippets from the tests you've already been running.

An incremental approach sounds good; assuming nobody objects, I'll remove most of the code in this patch and just leave the core mechanism behind (in enforcing mode) and add in that test case. We can fill in the other cases as we enable codegen/acle/autovec in separate patches.

Jul 23 2019, 4:21 AM · Restricted Project, Restricted Project
rovka added a comment to D53137: Scalable vector core instruction support + size queries.

Hi Graham,

Jul 23 2019, 2:22 AM · Restricted Project, Restricted Project

Jul 22 2019

rovka added inline comments to D64739: [SVE][Inline-Asm] Add support to specify SVE registers in the clobber list.
Jul 22 2019, 3:45 AM · Restricted Project, Restricted Project

Jul 19 2019

rovka added a comment to D62960: Add SVE opaque built-in types.

FWIW, I think the tests look great. Would be nice if someone more experienced with clang could also have a look though.

Jul 19 2019, 2:41 AM · Restricted Project, Restricted Project

Jul 18 2019

rovka added inline comments to D64739: [SVE][Inline-Asm] Add support to specify SVE registers in the clobber list.
Jul 18 2019, 2:51 AM · Restricted Project, Restricted Project
rovka committed rGf26706fa1c15: Fixup r366333 (require x86 in test) (authored by rovka).
Fixup r366333 (require x86 in test)
Jul 18 2019, 1:32 AM
rovka committed rL366416: Fixup r366333 (require x86 in test).
Fixup r366333 (require x86 in test)
Jul 18 2019, 1:32 AM

Jul 17 2019

rovka committed rG37e403d18c1a: [ARM GlobalISel] Cleanup CallLowering. NFC (authored by rovka).
[ARM GlobalISel] Cleanup CallLowering. NFC
Jul 17 2019, 3:06 AM
rovka committed rL366308: [ARM GlobalISel] Cleanup CallLowering. NFC.
[ARM GlobalISel] Cleanup CallLowering. NFC
Jul 17 2019, 3:05 AM

Jul 15 2019

rovka committed rG0bf0b8ff7c7e: [libFuzzer] Disable fork.test on AArch64 (authored by rovka).
[libFuzzer] Disable fork.test on AArch64
Jul 15 2019, 4:35 AM
rovka committed rL366055: [libFuzzer] Disable fork.test on AArch64.
[libFuzzer] Disable fork.test on AArch64
Jul 15 2019, 4:35 AM

Jul 2 2019

rovka added a comment to D62960: Add SVE opaque built-in types.

This looks much better, thanks! Shouldn't there be more tests, e.g. for mangling and maybe the ASTImporter?

Jul 2 2019, 3:23 AM · Restricted Project, Restricted Project

Jul 1 2019

rovka added a comment to D63905: Fix ASAN error caused by commit r364512.

Thanks, committed in r364778!

Jul 1 2019, 8:16 AM · Restricted Project
rovka committed rG2ba16011c139: Fixup r364512 (authored by rovka).
Fixup r364512
Jul 1 2019, 8:10 AM
rovka committed rL364778: Fixup r364512.
Fixup r364512
Jul 1 2019, 8:08 AM

Jun 28 2019

rovka added a comment to D63905: Fix ASAN error caused by commit r364512.

Looks good to me, please commit

(@rovka - this is at least a short-term fix, I'm approving this to unbreak LLVM's mainline - feel free to refix with other ideas if you have any (also happy to discuss this further with you here, IRC, or elsewhere on the mailing lists)

Jun 28 2019, 3:08 AM · Restricted Project

Jun 27 2019

rovka added a comment to D62960: Add SVE opaque built-in types.

Just a few nits/suggestions.

Jun 27 2019, 4:59 AM · Restricted Project, Restricted Project