Page MenuHomePhabricator

Meinersbur (Michael Kruse)
User

Projects

User does not belong to any projects.

User Details

User Since
Jun 17 2015, 7:07 AM (195 w, 5 d)

Recent Activity

Yesterday

Meinersbur committed rG89251edefcb4: [CodeGen] LLVM OpenMP Backend. (authored by Meinersbur).
[CodeGen] LLVM OpenMP Backend.
Mon, Mar 18, 8:18 PM
Meinersbur added a comment to D58736: [System Model] Introduce a target system model.

Thank you for pushing this forward and sorry for the delay.

Mon, Mar 18, 3:50 PM · Restricted Project

Thu, Mar 14

Meinersbur added inline comments to D57598: [VPLAN] Determine Vector Width programmatically..
Thu, Mar 14, 1:38 PM · Restricted Project
Meinersbur added inline comments to D57598: [VPLAN] Determine Vector Width programmatically..
Thu, Mar 14, 12:28 PM · Restricted Project

Tue, Mar 12

Meinersbur accepted D59263: Moving ManagedMemoryRewritePass when hybrid option is selected.

LGTM

Tue, Mar 12, 9:57 AM · Restricted Project, Restricted Project

Mon, Mar 11

Meinersbur updated the diff for D55952: [libomptarget] Introduce LIBOMPTARGET_ENABLE_DEBUG cmake option..
  • Undo whitespace change
Mon, Mar 11, 4:47 PM · Restricted Project
Meinersbur added a comment to D55952: [libomptarget] Introduce LIBOMPTARGET_ENABLE_DEBUG cmake option..

Sorry for the delay. I missed your reply during the holidays.

Mon, Mar 11, 4:45 PM · Restricted Project
Meinersbur updated the diff for D55952: [libomptarget] Introduce LIBOMPTARGET_ENABLE_DEBUG cmake option..
  • Define OMPTARGET_DEBUG preprocessor symbol if LIBOMPTARGET_ENABLE_DEBUG
  • Fix regression test execution
Mon, Mar 11, 4:43 PM · Restricted Project
Meinersbur added a comment to D58202: [Polly][ScheduleOptimizer] Hoist extension nodes after schedule optimization..

ping

Mon, Mar 11, 2:32 PM · Restricted Project
Meinersbur added a comment to D57978: [CodeGen] Generate follow-up metadata for loops with more than one transformation..

The Polly-powered additional transformations now also generate this kind of metadata.

Mon, Mar 11, 2:32 PM · Restricted Project
Herald added a project to D55290: [docs] Update llvm.loop metadata documentation.: Restricted Project.

ping

Mon, Mar 11, 2:25 PM · Restricted Project

Fri, Mar 8

Meinersbur committed rG65c5821e3fac: [RegionPass] Fix forgotten "!". (authored by Meinersbur).
[RegionPass] Fix forgotten "!".
Fri, Mar 8, 1:02 PM
Meinersbur committed rL355733: [RegionPass] Fix forgotten "!"..
[RegionPass] Fix forgotten "!".
Fri, Mar 8, 1:02 PM

Mon, Feb 25

Meinersbur committed rG0336c75c365a: [OpenMP 5.0] Parsing/sema support for from clause with mapper modifier. (authored by Meinersbur).
[OpenMP 5.0] Parsing/sema support for from clause with mapper modifier.
Mon, Feb 25, 12:34 PM
Meinersbur committed rC354817: [OpenMP 5.0] Parsing/sema support for from clause with mapper modifier..
[OpenMP 5.0] Parsing/sema support for from clause with mapper modifier.
Mon, Feb 25, 12:34 PM
Meinersbur committed rL354817: [OpenMP 5.0] Parsing/sema support for from clause with mapper modifier..
[OpenMP 5.0] Parsing/sema support for from clause with mapper modifier.
Mon, Feb 25, 12:34 PM
Meinersbur closed D58638: [OpenMP 5.0] Parsing/sema support for from clause with mapper modifier.
Mon, Feb 25, 12:33 PM · Restricted Project, Restricted Project, Restricted Project
Meinersbur added a comment to D58535: [OptRemarks] Make OptRemarks more generic: rename OptRemarks to Remarks.

What is the motivation of the rename? What are the plans for more generic OptRemarks?

Mon, Feb 25, 8:43 AM · Restricted Project

Fri, Feb 22

Meinersbur committed rG01f670df8f37: [OpenMP 5.0] Parsing/sema support for to clause with mapper modifier. (authored by Meinersbur).
[OpenMP 5.0] Parsing/sema support for to clause with mapper modifier.
Fri, Feb 22, 2:29 PM
Meinersbur committed rC354698: [OpenMP 5.0] Parsing/sema support for to clause with mapper modifier..
[OpenMP 5.0] Parsing/sema support for to clause with mapper modifier.
Fri, Feb 22, 2:29 PM
Meinersbur committed rL354698: [OpenMP 5.0] Parsing/sema support for to clause with mapper modifier..
[OpenMP 5.0] Parsing/sema support for to clause with mapper modifier.
Fri, Feb 22, 2:29 PM
Meinersbur closed D58523: [OpenMP 5.0] Parsing/sema support for to clause with mapper modifier.
Fri, Feb 22, 2:29 PM · Restricted Project, Restricted Project, Restricted Project

Tue, Feb 19

Meinersbur committed rG4304e9d14399: [OpenMP 5.0] Parsing/sema support for map clause with mapper modifier. (authored by Meinersbur).
[OpenMP 5.0] Parsing/sema support for map clause with mapper modifier.
Tue, Feb 19, 8:40 AM
Meinersbur committed rC354347: [OpenMP 5.0] Parsing/sema support for map clause with mapper modifier..
[OpenMP 5.0] Parsing/sema support for map clause with mapper modifier.
Tue, Feb 19, 8:40 AM
Meinersbur committed rL354347: [OpenMP 5.0] Parsing/sema support for map clause with mapper modifier..
[OpenMP 5.0] Parsing/sema support for map clause with mapper modifier.
Tue, Feb 19, 8:39 AM
Meinersbur closed D58074: [OpenMP 5.0] Parsing/sema support for map clause with mapper modifier.
Tue, Feb 19, 8:39 AM · Restricted Project, Restricted Project, Restricted Project

Mon, Feb 18

Meinersbur added inline comments to D57978: [CodeGen] Generate follow-up metadata for loops with more than one transformation..
Mon, Feb 18, 9:19 AM · Restricted Project

Feb 13 2019

Meinersbur created D58202: [Polly][ScheduleOptimizer] Hoist extension nodes after schedule optimization..
Feb 13 2019, 12:54 PM · Restricted Project

Feb 12 2019

Meinersbur committed rL353864: Remove completed project.
Remove completed project
Feb 12 2019, 9:47 AM

Feb 11 2019

Meinersbur committed rG77a614a6e154: Refactor setAlreadyUnrolled() and setAlreadyVectorized(). (authored by Meinersbur).
Refactor setAlreadyUnrolled() and setAlreadyVectorized().
Feb 11 2019, 11:46 AM
Meinersbur committed rL353738: Refactor setAlreadyUnrolled() and setAlreadyVectorized()..
Refactor setAlreadyUnrolled() and setAlreadyVectorized().
Feb 11 2019, 11:45 AM
Meinersbur closed D57566: Refactor setAlreadyUnrolled() and setAlreadyVectorized()..
Feb 11 2019, 11:45 AM · Restricted Project
Meinersbur updated the diff for D57566: Refactor setAlreadyUnrolled() and setAlreadyVectorized()..
  • Use APInt to construct constant in metadata
  • Undo unnecessary test case changes
Feb 11 2019, 9:40 AM · Restricted Project

Feb 8 2019

Meinersbur created D57978: [CodeGen] Generate follow-up metadata for loops with more than one transformation..
Feb 8 2019, 2:35 PM · Restricted Project

Feb 4 2019

Meinersbur committed rG70560a0a2cce: [WarnMissedTransforms] Do not warn about already vectorized loops. (authored by Meinersbur).
[WarnMissedTransforms] Do not warn about already vectorized loops.
Feb 4 2019, 11:57 AM
Meinersbur committed rL353082: [WarnMissedTransforms] Do not warn about already vectorized loops..
[WarnMissedTransforms] Do not warn about already vectorized loops.
Feb 4 2019, 11:56 AM
Meinersbur closed D57542: [WarnMissedTransforms] Do not warn about already vectorized loops..
Feb 4 2019, 11:55 AM · Restricted Project
Meinersbur updated the diff for D57566: Refactor setAlreadyUnrolled() and setAlreadyVectorized()..
  • Use a TemporaryMDNode instead of nullptr
Feb 4 2019, 10:12 AM · Restricted Project

Feb 1 2019

Meinersbur committed rL352906: [OpenMP 5.0] Parsing/sema support for "omp declare mapper" directive..
[OpenMP 5.0] Parsing/sema support for "omp declare mapper" directive.
Feb 1 2019, 12:25 PM
Meinersbur committed rC352906: [OpenMP 5.0] Parsing/sema support for "omp declare mapper" directive..
[OpenMP 5.0] Parsing/sema support for "omp declare mapper" directive.
Feb 1 2019, 12:25 PM
Meinersbur closed D56326: [OpenMP 5.0] Parsing/sema support for "omp declare mapper" directive.
Feb 1 2019, 12:25 PM · Restricted Project, Restricted Project
Meinersbur added a comment to D56326: [OpenMP 5.0] Parsing/sema support for "omp declare mapper" directive.

@lildmh Thanks for the patch. I can commit this patch for you. The tests run fine on my machine. However, for committing on other's behalf, we were asked to make the contributor aware that the contribution will be published under a new license. Are you OK with the new license?

Feb 1 2019, 11:43 AM · Restricted Project, Restricted Project

Jan 31 2019

Meinersbur created D57566: Refactor setAlreadyUnrolled() and setAlreadyVectorized()..
Jan 31 2019, 10:36 PM · Restricted Project
Meinersbur created D57542: [WarnMissedTransforms] Do not warn about already vectorized loops..
Jan 31 2019, 1:39 PM · Restricted Project

Jan 28 2019

Meinersbur accepted D57260: Check bool attribute value in getOptionalBoolLoopAttribute..

Thanks for catching this. Looks like I did not think about calling getZExtValue on ConstantInt, but didn't notive because ConstantInt* converts to bool implicitly.

Jan 28 2019, 5:48 PM

Jan 25 2019

Meinersbur added a comment to D56928: Support attribute used in member funcs of class templates.

Thanks for looking into my almost 6 year old bug!

Jan 25 2019, 3:54 PM · Restricted Project
Meinersbur added a comment to D57260: Check bool attribute value in getOptionalBoolLoopAttribute..

The check hasVectorizeTransformation(L) in front checks whether vectorization is forced by the user. It is not necessary that the user also fixes the simd width/interleave count. Hence, the warning is valid in this case. It is the same behavior as before rL348944 (which I tried to replicate).

Jan 25 2019, 3:39 PM

Jan 11 2019

Meinersbur added a comment to D56403: add pragmas to control Software Pipelining optimisation.
In D56403#1353857, @alexey.lapshin wrote:

Would it be OK to integrate this patch in it`s current state and create another patch for warning about missed transformation for #pragma clang loop pipeline ?

Jan 11 2019, 12:43 PM

Jan 8 2019

Meinersbur added a comment to D56403: add pragmas to control Software Pipelining optimisation.

I recently added a WarnMissedTransformationsPass (lib/Transforms/Scalar/WarnMissedTransformations.cpp) which emits a warning if a #pragma clang loop has not been applied, for any reason. It would be nince if it would work for #pragma clang loop as well. For instance, if the target backend does not support yet. My implementation is located after the midend loop passes, so it cannot be adapted directly.

Jan 8 2019, 12:02 PM

Dec 20 2018

Meinersbur committed rL349823: [CodeGen] Generate llvm.loop.parallel_accesses instead of llvm.mem..
[CodeGen] Generate llvm.loop.parallel_accesses instead of llvm.mem.
Dec 20 2018, 1:28 PM
Meinersbur committed rC349823: [CodeGen] Generate llvm.loop.parallel_accesses instead of llvm.mem..
[CodeGen] Generate llvm.loop.parallel_accesses instead of llvm.mem.
Dec 20 2018, 1:28 PM
Meinersbur closed D52117: Generate llvm.loop.parallel_accesses instead of llvm.mem.parallel_loop_access metadata..
Dec 20 2018, 1:28 PM
Meinersbur created D55952: [libomptarget] Introduce LIBOMPTARGET_ENABLE_DEBUG cmake option..
Dec 20 2018, 1:06 PM · Restricted Project
Meinersbur committed rL349774: [InstCombine] Preserve access-group metadata..
[InstCombine] Preserve access-group metadata.
Dec 20 2018, 9:14 AM

Dec 19 2018

Meinersbur committed rL349725: Introduce llvm.loop.parallel_accesses and llvm.access.group metadata..
Introduce llvm.loop.parallel_accesses and llvm.access.group metadata.
Dec 19 2018, 9:01 PM
Meinersbur closed D52116: Introduce llvm.loop.parallel_accesses and llvm.access.group metadata..
Dec 19 2018, 9:01 PM
Meinersbur updated the diff for D52116: Introduce llvm.loop.parallel_accesses and llvm.access.group metadata..
  • Rename: isAccessGroup -> isValidAsAccessGroup
Dec 19 2018, 3:32 PM
Meinersbur updated the diff for D52117: Generate llvm.loop.parallel_accesses instead of llvm.mem.parallel_loop_access metadata..
  • Fix typo
Dec 19 2018, 12:32 PM
Meinersbur updated the diff for D52116: Introduce llvm.loop.parallel_accesses and llvm.access.group metadata..
  • Address @hfinkel's review
  • clang-format
Dec 19 2018, 12:29 PM

Dec 18 2018

Meinersbur added a comment to D55851: Implement basic loop fusion pass.

Thanks for bringing us loop fusion! Here are some review comments, but I did not check the functionality yet.

Dec 18 2018, 10:06 PM · Restricted Project
Meinersbur committed rL349513: [LoopVectorize] Rename pass options. NFC..
[LoopVectorize] Rename pass options. NFC.
Dec 18 2018, 9:49 AM
Meinersbur closed D55785: [LoopVectorize] Rename pass options. NFC..
Dec 18 2018, 9:49 AM
Meinersbur updated the diff for D55785: [LoopVectorize] Rename pass options. NFC..
  • Rebase after commit of D55716
Dec 18 2018, 9:39 AM
Meinersbur committed rL349509: [LoopUnroll] Honor '#pragma unroll' even with -fno-unroll-loops..
[LoopUnroll] Honor '#pragma unroll' even with -fno-unroll-loops.
Dec 18 2018, 9:19 AM
Meinersbur closed D55716: [LoopUnroll] Honor '#pragma unroll' even with -fno-unroll-loops..
Dec 18 2018, 9:19 AM

Dec 17 2018

Meinersbur created D55785: [LoopVectorize] Rename pass options. NFC..
Dec 17 2018, 12:47 PM
Meinersbur updated the diff for D55716: [LoopUnroll] Honor '#pragma unroll' even with -fno-unroll-loops..
  • Rename AlwaysUnroll -> !OnlyWhenForced
  • Add comments describing OnlyWhenForced
Dec 17 2018, 11:25 AM
Meinersbur accepted D55726: [test-suite] Adding CoMD Proxy App.

Thanks for updating the patch. It ran successfully on my machine. LGMT.

Dec 17 2018, 8:43 AM

Dec 14 2018

Meinersbur added a comment to D55726: [test-suite] Adding CoMD Proxy App.

The diff seems strange, no files are flagged as new. For instance, the .depend file is seen in Phabricator as "The contents of this file were not changed." I assume the file is empty? Can it be removed from the diff?

Dec 14 2018, 6:25 PM
Meinersbur added inline comments to D55716: [LoopUnroll] Honor '#pragma unroll' even with -fno-unroll-loops..
Dec 14 2018, 1:33 PM
Meinersbur created D55716: [LoopUnroll] Honor '#pragma unroll' even with -fno-unroll-loops..
Dec 14 2018, 1:29 PM
Meinersbur committed rL349184: [TransformWarning] Do not warn missed transformations in optnone functions..
[TransformWarning] Do not warn missed transformations in optnone functions.
Dec 14 2018, 11:49 AM
Meinersbur closed D55690: [TransformWarning] Do not warn missed transformations in optnone functions..
Dec 14 2018, 11:49 AM
Meinersbur committed rL349170: [Transforms] Preserve metadata when converting invoke to call..
[Transforms] Preserve metadata when converting invoke to call.
Dec 14 2018, 10:18 AM
Meinersbur closed D55666: [Transforms] Preserve metadata when converting invoke to call..
Dec 14 2018, 10:18 AM

Dec 13 2018

Meinersbur retitled D55666: [Transforms] Preserve metadata when converting invoke to call. from [Transforms] Preserve access metadata. to [Transforms] Preserve metadata when converting invoke to call..
Dec 13 2018, 9:42 PM
Meinersbur updated the diff for D55666: [Transforms] Preserve metadata when converting invoke to call..
  • Generalize copyMetadata
Dec 13 2018, 9:41 PM
Meinersbur created D55690: [TransformWarning] Do not warn missed transformations in optnone functions..
Dec 13 2018, 9:29 PM
Meinersbur updated the diff for D55666: [Transforms] Preserve metadata when converting invoke to call..

Add test case

Dec 13 2018, 1:01 PM
Meinersbur created D55666: [Transforms] Preserve metadata when converting invoke to call..
Dec 13 2018, 12:25 PM
Meinersbur added inline comments to D52116: Introduce llvm.loop.parallel_accesses and llvm.access.group metadata..
Dec 13 2018, 10:36 AM
Meinersbur updated the diff for D52116: Introduce llvm.loop.parallel_accesses and llvm.access.group metadata..
  • Reinsert parts of LangRef.rst that went missing after merge.
Dec 13 2018, 10:36 AM
Meinersbur updated the diff for D52116: Introduce llvm.loop.parallel_accesses and llvm.access.group metadata..
  • Rebase to trunk
Dec 13 2018, 10:25 AM

Dec 12 2018

Meinersbur closed D55288: [test] Fix tests for changed optimizer warning message.

Committed together with D49281 in rC348944.

Dec 12 2018, 12:40 PM
Meinersbur committed rL348949: [LV] Fix signed/unsigned comparison warning..
[LV] Fix signed/unsigned comparison warning.
Dec 12 2018, 10:11 AM
Meinersbur committed rL348947: [docs] Use correct ending quotes..
[docs] Use correct ending quotes.
Dec 12 2018, 10:03 AM
Meinersbur committed rL348944: [Unroll/UnrollAndJam/Vectorizer/Distribute] Add followup loop attributes..
[Unroll/UnrollAndJam/Vectorizer/Distribute] Add followup loop attributes.
Dec 12 2018, 9:39 AM
Meinersbur committed rC348944: [Unroll/UnrollAndJam/Vectorizer/Distribute] Add followup loop attributes..
[Unroll/UnrollAndJam/Vectorizer/Distribute] Add followup loop attributes.
Dec 12 2018, 9:39 AM
Meinersbur closed D49281: [Unroll/UnrollAndJam/Vectorizer/Distribute] Add followup loop attributes..
Dec 12 2018, 9:38 AM

Dec 11 2018

Meinersbur updated the diff for D49281: [Unroll/UnrollAndJam/Vectorizer/Distribute] Add followup loop attributes..
  • [test] Revise tests
    • Consistent disable_nonforced testing
    • Unify followup-attribute testing. The previous approach was to copy existing test cases and emulate the behavior of the loop transformation passes using followup attributes. This had the disadvantage that the pass would pass even if the followup-attribute was ignored (indeed, some were misspelled) since the result is the same. Instead, use a new "followup.ll" test per loop pass that checks the presence of new attributes specific for each followup.
  • [docs] Add followup attribute recommendations
Dec 11 2018, 9:27 PM

Dec 10 2018

Meinersbur committed rL348762: Use zip_longest for iterator range comparisons. NFC..
Use zip_longest for iterator range comparisons. NFC.
Dec 10 2018, 7:20 AM
Meinersbur committed rC348762: Use zip_longest for iterator range comparisons. NFC..
Use zip_longest for iterator range comparisons. NFC.
Dec 10 2018, 7:20 AM
Meinersbur closed D55468: Use zip_longest for iterator range comparisons. NFC..
Dec 10 2018, 7:19 AM

Dec 9 2018

Meinersbur added inline comments to D55468: Use zip_longest for iterator range comparisons. NFC..
Dec 9 2018, 3:38 PM
Meinersbur updated the diff for D55468: Use zip_longest for iterator range comparisons. NFC..
  • Fix xor
  • Store tuple elements in variables
Dec 9 2018, 3:20 PM
Meinersbur added inline comments to D55468: Use zip_longest for iterator range comparisons. NFC..
Dec 9 2018, 1:27 PM
Meinersbur updated the diff for D55468: Use zip_longest for iterator range comparisons. NFC..
Dec 9 2018, 1:26 PM

Dec 7 2018

Meinersbur retitled D55468: Use zip_longest for iterator range comparisons. NFC. from Use zip_longest for list comparisons. to Use zip_longest for iterator range comparisons. NFC..
Dec 7 2018, 7:48 PM
Meinersbur created D55468: Use zip_longest for iterator range comparisons. NFC..
Dec 7 2018, 7:31 PM
Meinersbur updated the diff for D55290: [docs] Update llvm.loop metadata documentation..
  • Add doc about DILocation nodes in LoopIDs
Dec 7 2018, 2:59 PM · Restricted Project