Page MenuHomePhabricator

vchuravy (Valentin Churavy)
User

Projects

User does not belong to any projects.

User Details

User Since
Jun 10 2015, 8:17 AM (314 w, 1 d)

Recent Activity

Yesterday

vchuravy added a project to D104480: [Orc] add C-API for transform layer: Unknown Object (Project).
Thu, Jun 17, 11:56 AM · Unknown Object (Project), Restricted Project
vchuravy requested review of D104480: [Orc] add C-API for transform layer.
Thu, Jun 17, 11:55 AM · Unknown Object (Project), Restricted Project

May 14 2021

vchuravy updated subscribers of D102471: New tag for ittapi - fix an error related to cross-compiling ITTAPI in LLVM with mingw.
May 14 2021, 9:07 AM · Restricted Project

May 10 2021

vchuravy added inline comments to D102097: [CompilerRT] Sanitizer compilation on musl systems.
May 10 2021, 10:06 AM · Restricted Project

May 7 2021

vchuravy added inline comments to D102097: [CompilerRT] Sanitizer compilation on musl systems.
May 7 2021, 5:29 PM · Restricted Project
vchuravy updated subscribers of D102097: [CompilerRT] Sanitizer compilation on musl systems.
May 7 2021, 5:28 PM · Restricted Project
vchuravy requested review of D102097: [CompilerRT] Sanitizer compilation on musl systems.
May 7 2021, 5:23 PM · Restricted Project

Feb 16 2021

vchuravy added a project to D96283: [DAGCombine] Do not remove masking argument to FP16_TO_FP for some targets: Unknown Object (Project).
Feb 16 2021, 7:27 PM · Unknown Object (Project), Restricted Project

Jan 19 2021

vchuravy accepted D94980: [PowerPC] Do not emit HW loop with half precision operations.

That does seem to indeed fix all the remaining issues with Float16 that the Julia testsuite covers :)

Jan 19 2021, 7:20 PM · Restricted Project
vchuravy abandoned D94828: [PowerPC] Disable CTR loops containing floating point conversion on half-precision.

Abandoned in favour of D94980

Jan 19 2021, 7:19 PM · Unknown Object (Project), Restricted Project

Jan 15 2021

vchuravy added reviewers for D94813: [MergeICmps] Don't merge icmps derived from pointers with addressspaces: courbet, MaskRay, jyknight.
Jan 15 2021, 9:37 PM · Restricted Project
vchuravy added a reviewer for D94828: [PowerPC] Disable CTR loops containing floating point conversion on half-precision: Restricted Project.
Jan 15 2021, 2:32 PM · Unknown Object (Project), Restricted Project
vchuravy requested review of D94828: [PowerPC] Disable CTR loops containing floating point conversion on half-precision.
Jan 15 2021, 2:32 PM · Unknown Object (Project), Restricted Project
vchuravy retitled D94813: [MergeICmps] Don't merge icmps derived from pointers with addressspaces from [MergeICmps] Don't merge icmps derived frm pointers with addressspaces to [MergeICmps] Don't merge icmps derived from pointers with addressspaces.
Jan 15 2021, 2:27 PM · Restricted Project
vchuravy accepted D94058: [PowerPC] Sign extend comparison operand for signed atomic comparisons.

Good to go from the Julia side.

Jan 15 2021, 11:26 AM · Restricted Project
vchuravy added a reviewer for D94813: [MergeICmps] Don't merge icmps derived from pointers with addressspaces: Restricted Project.
Jan 15 2021, 11:12 AM · Restricted Project
vchuravy requested review of D94813: [MergeICmps] Don't merge icmps derived from pointers with addressspaces.
Jan 15 2021, 11:12 AM · Restricted Project

Jan 5 2021

vchuravy updated subscribers of D94058: [PowerPC] Sign extend comparison operand for signed atomic comparisons.
Jan 5 2021, 7:57 AM · Restricted Project

Dec 16 2020

vchuravy added a comment to D92906: [PowerPC] Restore stack ptr from frame ptr with setjmp.

This fixed a whole set of bugs for us but it also seems to have caused a regression.
https://github.com/JuliaLang/julia/issues/38859

Dec 16 2020, 5:30 PM · Restricted Project

Dec 9 2020

vchuravy accepted D92906: [PowerPC] Restore stack ptr from frame ptr with setjmp.

Looks good from the Julia side, this does indeed fix the upstream bug. Thanks @nemanjai!

Dec 9 2020, 10:35 AM · Restricted Project
vchuravy updated subscribers of D92906: [PowerPC] Restore stack ptr from frame ptr with setjmp.
Dec 9 2020, 5:00 AM · Restricted Project

Dec 7 2020

vchuravy updated the diff for D75072: [SCEV] Record NI types in add exprs.

Rebase commit onto main

Dec 7 2020, 6:04 PM · Unknown Object (Project), Restricted Project
vchuravy commandeered D75072: [SCEV] Record NI types in add exprs.
Dec 7 2020, 5:38 PM · Unknown Object (Project), Restricted Project
vchuravy added a comment to D50010: [VNCoercion] Disallow coercion between different ni addrspaces.

Thanks! I rebased the PR and landed it.

Dec 7 2020, 5:21 PM · Unknown Object (Project), Restricted Project
vchuravy committed rG700cf7dcc927: [VNCoercion] Disallow coercion between different ni addrspaces (authored by vchuravy).
[VNCoercion] Disallow coercion between different ni addrspaces
Dec 7 2020, 5:20 PM
vchuravy closed D50010: [VNCoercion] Disallow coercion between different ni addrspaces.
Dec 7 2020, 5:20 PM · Unknown Object (Project), Restricted Project

Dec 6 2020

vchuravy added a comment to D50010: [VNCoercion] Disallow coercion between different ni addrspaces.

Bump for review. We (Julia) have been carrying this one for a while.

Dec 6 2020, 10:58 AM · Unknown Object (Project), Restricted Project

Nov 27 2020

vchuravy updated subscribers of D86837: Allow specifying AA info when constructing certain libcalls.
Nov 27 2020, 5:13 PM · Restricted Project

Nov 23 2020

vchuravy added a comment to D75072: [SCEV] Record NI types in add exprs.

@reames I think you are the right person to review this. Any objections?
We have been carrying this on the Julia side. I can rebase this on master if that would help (LLVM 11 version is available here https://github.com/JuliaLang/julia/blob/master/deps/patches/llvm-11-D75072-SCEV-add-type.patch)

Nov 23 2020, 5:41 PM · Unknown Object (Project), Restricted Project
vchuravy added a reviewer for D73156: [clang] Build clang-shlib on mingw: mstorsjo.
Nov 23 2020, 5:17 PM · Restricted Project
vchuravy added a comment to D88630: [clang/CMake] Respect LLVM_TOOLS_INSTALL_DIR.

Well, I've tested it and it doesn't seem to break anything for us (Gentoo) but I don't really understand why you'd have a different install dir for LLVM and Clang.

Nov 23 2020, 5:12 PM · Restricted Project
vchuravy committed rG4bc88a0e9a2e: Enable support for floating-point division reductions (authored by yuyichao).
Enable support for floating-point division reductions
Nov 23 2020, 5:06 PM
vchuravy closed D34078: Enable support for floating-point division reductions.
Nov 23 2020, 5:06 PM · Restricted Project, Unknown Object (Project)
vchuravy abandoned D89211: [RTDYLD] add debug statements for resolveX86Relocation.
Nov 23 2020, 4:06 PM · Restricted Project

Nov 6 2020

vchuravy abandoned D44650: Fix build of llvm-cfi-verify on mingw32.

In favor of https://reviews.llvm.org/D90969

Nov 6 2020, 1:53 PM · Unknown Object (Project), Restricted Project

Nov 3 2020

vchuravy accepted D90722: [RTDYLD] support absolute relocations where needed.

Looks good from my side. @lhames should sign off though.

Nov 3 2020, 3:01 PM · Restricted Project
vchuravy abandoned D89241: [RTDYLD] be more defensive about relocations with empty symbol names.
Nov 3 2020, 2:51 PM · Restricted Project

Oct 19 2020

vchuravy added a comment to rGc068e9c8c123: [Support][CommandLine] Delete unused llvm::cl::ParseEnvrironmentOptions.

It is questionable for a downstream project to parse and interpret LLVM internal options directly. It should at least to have a -mllvm or --plugin-opt= prefix.

Oct 19 2020, 6:33 AM

Oct 18 2020

vchuravy added a comment to rGc068e9c8c123: [Support][CommandLine] Delete unused llvm::cl::ParseEnvrironmentOptions.

Yeah, I basically copy-and-pasted the original implementation, since we are indeed parsing this from the environment. https://github.com/JuliaLang/julia/pull/38092/commits/72c2136dfad7a535dba563c0f37f3c70da5558a2

Oct 18 2020, 10:05 PM
vchuravy added a comment to rGc068e9c8c123: [Support][CommandLine] Delete unused llvm::cl::ParseEnvrironmentOptions.

@MaskRay why was it necessary to delete this? It is used by downstream projects (like Julia). I agree that it isn't too much hassle to implement, but every tool that uses LLVM as a library will have to write a version themselves.

Oct 18 2020, 6:52 PM

Oct 12 2020

vchuravy updated the diff for D89241: [RTDYLD] be more defensive about relocations with empty symbol names.

clang-format

Oct 12 2020, 7:58 AM · Restricted Project
vchuravy requested review of D89241: [RTDYLD] be more defensive about relocations with empty symbol names.
Oct 12 2020, 7:55 AM · Restricted Project
vchuravy added a comment to D89211: [RTDYLD] add debug statements for resolveX86Relocation.

I think this can be very noisy as the number of relocations can be very large.

Oct 12 2020, 7:00 AM · Restricted Project

Oct 11 2020

vchuravy updated the diff for D89211: [RTDYLD] add debug statements for resolveX86Relocation.

Fix typo

Oct 11 2020, 11:23 AM · Restricted Project
vchuravy requested review of D89211: [RTDYLD] add debug statements for resolveX86Relocation.
Oct 11 2020, 11:21 AM · Restricted Project

Oct 8 2020

vchuravy accepted D88846: [mlir] Improve LLVM shlib support.
Oct 8 2020, 11:52 AM · Restricted Project

Jul 27 2020

vchuravy abandoned D66035: [WebAssembly] WIP: Add support for reference types.

I am interested in continuing this work and have a patch in progress based on the current available one here. Should I post the new patch here or under a new bug?

Jul 27 2020, 7:30 AM · Restricted Project, Restricted Project

Jul 17 2020

vchuravy updated subscribers of D84031: Clear NewGEPBases after finish using them in CodeGenPrep pass.
Jul 17 2020, 8:54 AM · Restricted Project

Jul 14 2020

vchuravy added a reviewer for D83800: Support PC relative relocation on AArch64 and PPC64 in RelocationResolver: nemanjai.
Jul 14 2020, 11:50 AM · Restricted Project

May 4 2020

vchuravy abandoned D78771: Add dependency edges between generated headers and users.

Abandoned in favour of @stephenneuendorffer recent list of changes.

May 4 2020, 6:54 AM · Restricted Project, Restricted Project
vchuravy accepted D79067: [MLIR] Fix libMLIR.so and LLVM_LINK_LLVM_DYLIB.
May 4 2020, 6:54 AM · Restricted Project
vchuravy accepted D79241: [cmake] Ensure that LINK_LIBS are dependencies for object library targets.
May 4 2020, 6:54 AM · Restricted Project

Apr 29 2020

vchuravy added a comment to D78773: Adjust libMLIR building to more closely follow libClang.

@vchuravy, What's your plan with this patch? I think this should land and I've started building on top of it.

Apr 29 2020, 7:29 PM · Restricted Project, Restricted Project
vchuravy added a comment to D44485: [MC] Always emit relocations for same-section function references.

Right the issue for is that LLVM6 used to accept and compile it without issue:

Apr 29 2020, 4:46 PM · Restricted Project
Herald added a reviewer for D73702: [WIP][mlir] enable conversion materialization for 1-1 type conversions: herhut.
Apr 29 2020, 6:56 AM · Restricted Project

Apr 24 2020

vchuravy added a comment to D78773: Adjust libMLIR building to more closely follow libClang.

Does this get us closer to fixing LLVM_LINK_LLVM_DYLIB?

Apr 24 2020, 10:49 AM · Restricted Project, Restricted Project
vchuravy added a comment to D78771: Add dependency edges between generated headers and users.

Ultimately, the solution may be to just generate all the headers before building everything else, which means that all mlir libraries would have a dependence on all IncGen targets. That isn't great, but I think it's better than the fix proposed in this review.

Apr 24 2020, 10:48 AM · Restricted Project, Restricted Project
vchuravy added a comment to D78773: Adjust libMLIR building to more closely follow libClang.

With ninja all dependencies edges are fine, but with Makefiles it indeed still complains about missing edges to the generated headers.

Apr 24 2020, 9:09 AM · Restricted Project, Restricted Project
vchuravy updated the diff for D78773: Adjust libMLIR building to more closely follow libClang.

small fixes and use MLIR_STATIC_LIBS

Apr 24 2020, 9:09 AM · Restricted Project, Restricted Project

Apr 23 2020

vchuravy added a reviewer for D78773: Adjust libMLIR building to more closely follow libClang: rriddle.
Apr 23 2020, 4:52 PM · Restricted Project, Restricted Project
vchuravy added inline comments to D78771: Add dependency edges between generated headers and users.
Apr 23 2020, 4:52 PM · Restricted Project, Restricted Project
vchuravy added reviewers for D78773: Adjust libMLIR building to more closely follow libClang: mehdi_amini, stephenneuendorffer.
Apr 23 2020, 4:52 PM · Restricted Project, Restricted Project
vchuravy added a reviewer for D78771: Add dependency edges between generated headers and users: stephenneuendorffer.
Apr 23 2020, 4:52 PM · Restricted Project, Restricted Project
vchuravy created D78773: Adjust libMLIR building to more closely follow libClang.
Apr 23 2020, 4:52 PM · Restricted Project, Restricted Project
Herald added a reviewer for D78771: Add dependency edges between generated headers and users: herhut.
Apr 23 2020, 4:52 PM · Restricted Project, Restricted Project

Apr 15 2020

vchuravy added a comment to D78196: [PowerPC] Clear the set of symbols that need to be updated in MCTargetStreamer.

We do reuse the constructed pass pipeline more than once, but that is a supported configuration see -compile-twice in LLC (as an example https://reviews.llvm.org/D17712)

Apr 15 2020, 10:57 AM · Restricted Project
vchuravy accepted D78196: [PowerPC] Clear the set of symbols that need to be updated in MCTargetStreamer.

I am unable to reproduce the issue with this patch applied. 👍

Apr 15 2020, 7:05 AM · Restricted Project

Mar 31 2020

vchuravy added a comment to D77133: [mlir] Add an out-of-tree dialect example.

This is fantastic, thanks.

Mar 31 2020, 1:11 PM · Restricted Project
vchuravy committed rGcba15d1b5c54: Add manually overriding of the darwin SDK version (authored by vchuravy).
Add manually overriding of the darwin SDK version
Mar 31 2020, 10:02 AM
vchuravy closed D77026: Add manually overriding of the darwin SDK version.
Mar 31 2020, 10:01 AM · Restricted Project

Mar 29 2020

vchuravy updated the diff for D77026: Add manually overriding of the darwin SDK version.

Allow for manually specifying the darwin SDK version

Mar 29 2020, 8:52 PM · Restricted Project
vchuravy added inline comments to D77026: Add manually overriding of the darwin SDK version.
Mar 29 2020, 8:52 PM · Restricted Project
vchuravy retitled D77026: Add manually overriding of the darwin SDK version from Add caching to darwin SDK version lookup to Add manually overriding of the darwin SDK version.
Mar 29 2020, 8:52 PM · Restricted Project
vchuravy updated subscribers of D77026: Add manually overriding of the darwin SDK version.
Mar 29 2020, 6:44 PM · Restricted Project
vchuravy created D77026: Add manually overriding of the darwin SDK version.
Mar 29 2020, 6:43 PM · Restricted Project
vchuravy updated subscribers of D73156: [clang] Build clang-shlib on mingw.
Mar 29 2020, 6:43 PM · Restricted Project

Feb 21 2020

vchuravy added inline comments to D73130: [MLIR] Add support for libMLIR.so.
Feb 21 2020, 3:25 PM · Restricted Project
vchuravy added a comment to D74867: [MLIR] add out of tree version of mlir-opt.

This is good to have as a test, but I assume that eventually we would want to have the equivalent of LLVM_LINK_LLVM_DYLIB.

Feb 21 2020, 3:16 PM · Restricted Project
vchuravy added a comment to D74864: [MLIR] Move from using target_link_libraries to LINK_LIBS for llvm libraries..

@mehdi_amini and I had a conversation on discord about libMLIR.so and I think we allayed all fears that this would lock us into being stuck with a libMLIR.so if we want to eventually go down the path of including it as part of libMLIR.so
So I think we are good to go ahead with this.

Feb 21 2020, 3:16 PM · Restricted Project

Feb 20 2020

vchuravy added a comment to D74864: [MLIR] Move from using target_link_libraries to LINK_LIBS for llvm libraries..

I asked in another revision, but I don't think I got an answer: what is the story for the circular dependency if we were to use MLIR in LLVM in the future? I'd be worried about adding barriers into this and I have some memory of a previous discussion that keeping a single .so would be somehow required?

Feb 20 2020, 8:21 AM · Restricted Project

Feb 19 2020

vchuravy added inline comments to D74864: [MLIR] Move from using target_link_libraries to LINK_LIBS for llvm libraries..
Feb 19 2020, 6:34 PM · Restricted Project

Feb 8 2020

vchuravy updated the diff for D73130: [MLIR] Add support for libMLIR.so.

Properly implement shared library similar to clang-shlib

Feb 8 2020, 7:29 PM · Restricted Project
vchuravy retitled D73130: [MLIR] Add support for libMLIR.so from [mlir] Strawman PR for shared library support to [mlir] Shared library support.
Feb 8 2020, 7:29 PM · Restricted Project

Feb 2 2020

vchuravy added a comment to D68292: [CMake] Disable building all Darwin libraries (except builtins) for macOS i386 when the SDK is >= 10.15..

This broke cross-compilation from Linux for us, since we don't have xcodebuild/xcrun . We currently build with

Feb 2 2020, 8:21 AM · Restricted Project, Restricted Project

Jan 28 2020

vchuravy added a comment to D72822: [mlir] Replace AbstractOperation::classof with a ClassID instance..

I built a shared library (D73130) and run into the case that my HasParent<FuncOp> trait failed to verify.

Jan 28 2020, 2:25 PM · Restricted Project

Jan 23 2020

vchuravy added a comment to D44485: [MC] Always emit relocations for same-section function references.

Just as a note, Julia currently reverts this patch. IIRC it caused issue with gcc/mingw32 which we use for linking.
My notes from back then are fairly limited and another team member did the git bisect.

Jan 23 2020, 5:35 AM · Restricted Project

Jan 21 2020

vchuravy added a comment to D73156: [clang] Build clang-shlib on mingw.

Some additional context. We [Julia] is cross-compiling LLVM from Linux to mingw32
as part of an automatic buildsystem for our binary dependencies. The corresponding PR is https://github.com/JuliaPackaging/Yggdrasil/pull/417

Jan 21 2020, 7:13 PM · Restricted Project
vchuravy added a reviewer for D73156: [clang] Build clang-shlib on mingw: tstellar.
Jan 21 2020, 7:13 PM · Restricted Project
vchuravy created D73156: [clang] Build clang-shlib on mingw.
Jan 21 2020, 7:13 PM · Restricted Project
vchuravy added inline comments to D73130: [MLIR] Add support for libMLIR.so.
Jan 21 2020, 12:23 PM · Restricted Project
vchuravy created D73130: [MLIR] Add support for libMLIR.so.
Jan 21 2020, 12:23 PM · Restricted Project
vchuravy added a reviewer for D73130: [MLIR] Add support for libMLIR.so: stephenneuendorffer.
Jan 21 2020, 12:23 PM · Restricted Project

Jan 17 2020

vchuravy added a comment to D72586: [MLIR] Move from add_llvm_library to add_llvm_component_library.

Are there other LLVM subproject who inject themselves into the global llvm shared library? Should we have a libMLIR.so instead?

Jan 17 2020, 5:51 AM · Restricted Project

Jan 14 2020

Herald added a project to D44485: [MC] Always emit relocations for same-section function references: Restricted Project.
Jan 14 2020, 1:42 AM · Restricted Project
vchuravy added a project to D44650: Fix build of llvm-cfi-verify on mingw32: Unknown Object (Project).
Jan 14 2020, 1:33 AM · Unknown Object (Project), Restricted Project
vchuravy added a project to D34078: Enable support for floating-point division reductions: Unknown Object (Project).
Jan 14 2020, 1:33 AM · Restricted Project, Unknown Object (Project)
vchuravy added a comment to D27629: Fix unwind info relocation with large code model on AArch64.

Is there anything else to do? We (Julia) have been carrying this patch for a while.

Jan 14 2020, 1:33 AM · Restricted Project, Restricted Project
vchuravy updated the diff for D27629: Fix unwind info relocation with large code model on AArch64.

move test and rebase

Jan 14 2020, 1:23 AM · Restricted Project, Restricted Project
vchuravy updated the summary of D27629: Fix unwind info relocation with large code model on AArch64.
Jan 14 2020, 1:23 AM · Restricted Project, Restricted Project
vchuravy added a project to D27629: Fix unwind info relocation with large code model on AArch64: Restricted Project.
Jan 14 2020, 1:23 AM · Restricted Project, Restricted Project