Page MenuHomePhabricator

SjoerdMeijer (Sjoerd Meijer)
User

Projects

User does not belong to any projects.

User Details

User Since
Jan 26 2016, 2:17 AM (242 w, 3 d)

Recent Activity

Yesterday

SjoerdMeijer added inline comments to D86074: [ARM][MVE] Tail-predication: check get.active.lane.mask's TC value.
Thu, Sep 17, 7:15 AM · Restricted Project
SjoerdMeijer added inline comments to D87681: [ARM] Improve VPT predicate tracking.
Thu, Sep 17, 5:36 AM · Restricted Project
SjoerdMeijer committed rG6637d72ddd3c: [Lint] Add check for intrinsic get.active.lane.mask (authored by SjoerdMeijer).
[Lint] Add check for intrinsic get.active.lane.mask
Thu, Sep 17, 1:22 AM
SjoerdMeijer closed D87228: [Lint] Add check for intrinsic get.active.lane.mask.
Thu, Sep 17, 1:22 AM · Restricted Project

Wed, Sep 16

SjoerdMeijer updated the summary of D87769: [ARM][MVE] tail-predication: predicate new checks on force-enabled option.
Wed, Sep 16, 9:12 AM · Restricted Project
SjoerdMeijer committed rGb5c3efeb7bc9: [ARM][MVE] Tail-predication: predicate new elementcount checks on force-enabled (authored by SjoerdMeijer).
[ARM][MVE] Tail-predication: predicate new elementcount checks on force-enabled
Wed, Sep 16, 9:06 AM
SjoerdMeijer closed D87769: [ARM][MVE] tail-predication: predicate new checks on force-enabled option.
Wed, Sep 16, 9:05 AM · Restricted Project
SjoerdMeijer updated the diff for D87769: [ARM][MVE] tail-predication: predicate new checks on force-enabled option.

Thanks Dave, just for completeness, uploading a new diff with the codegen changes gone, which shouldn't have been there.

Wed, Sep 16, 8:59 AM · Restricted Project
SjoerdMeijer requested review of D87769: [ARM][MVE] tail-predication: predicate new checks on force-enabled option.
Wed, Sep 16, 8:44 AM · Restricted Project
SjoerdMeijer accepted D87751: [RDA] Fix getUniqueReachingDef for self loops.

Cheers, LGTM

Wed, Sep 16, 4:39 AM · Restricted Project
SjoerdMeijer added inline comments to D87751: [RDA] Fix getUniqueReachingDef for self loops.
Wed, Sep 16, 4:05 AM · Restricted Project
SjoerdMeijer accepted D87610: [ARM] Fix tail predication predicate tracking.

Thanks for that. There was a lot going on before, but this now looks like a small, nice change.

Wed, Sep 16, 3:55 AM · Restricted Project
SjoerdMeijer accepted D87753: [ARM] Add more validForTailPredication.

Looks reasonable

Wed, Sep 16, 3:48 AM · Restricted Project
SjoerdMeijer committed rGcb1ef0eaff87: Follow up rG635b87511ec3: forgot to add/commit the new test file. NFC. (authored by SjoerdMeijer).
Follow up rG635b87511ec3: forgot to add/commit the new test file. NFC.
Wed, Sep 16, 1:39 AM
SjoerdMeijer added a comment to D82678: [CGP] Set debug locations when optimizing phi types.

I am also not a debug expert, but this looks like an "innocent" patch to me that makes things a bit better, so that's good. What I am wondering about though why setDebugLoc isn't done in the constructor, which makes the code cleaner here and also it won't be forgotten. But I don't want to make this bigger than it is, and since I have never really looked into debug info, I also don't know if there would be any disadvantages doing that. Perhaps others can comment on that.

Wed, Sep 16, 1:13 AM · debug-info, Restricted Project
SjoerdMeijer added a comment to D87610: [ARM] Fix tail predication predicate tracking.

I understood there is a NFC and non-NFC part of this patch. Is worth separating this out?

Wed, Sep 16, 12:52 AM · Restricted Project

Tue, Sep 15

SjoerdMeijer committed rG635b87511ec3: [ARM][MVE] Tail-predication: use unsigned SCEV ranges for tripcount (authored by SjoerdMeijer).
[ARM][MVE] Tail-predication: use unsigned SCEV ranges for tripcount
Tue, Sep 15, 5:23 AM
SjoerdMeijer closed D87608: [ARM][MVE] Tail-predication: use unsigned SCEV ranges for tripcount.
Tue, Sep 15, 5:23 AM · Restricted Project
SjoerdMeijer committed rGb4b1b84106a0: [MVE] fix typo in llvm debug message. NFC. (authored by SjoerdMeijer).
[MVE] fix typo in llvm debug message. NFC.
Tue, Sep 15, 2:14 AM
SjoerdMeijer committed rG487412988cea: [MVE] Rename of tests making them consistent with tail-predication tests. NFC. (authored by SjoerdMeijer).
[MVE] Rename of tests making them consistent with tail-predication tests. NFC.
Tue, Sep 15, 1:25 AM
SjoerdMeijer added a comment to D87228: [Lint] Add check for intrinsic get.active.lane.mask.

Little non-urgent ping, but would be nice to get this little guy out of the way.

Tue, Sep 15, 12:37 AM · Restricted Project

Mon, Sep 14

SjoerdMeijer added inline comments to D86074: [ARM][MVE] Tail-predication: check get.active.lane.mask's TC value.
Mon, Sep 14, 3:55 PM · Restricted Project
SjoerdMeijer added inline comments to D86074: [ARM][MVE] Tail-predication: check get.active.lane.mask's TC value.
Mon, Sep 14, 2:15 PM · Restricted Project
SjoerdMeijer updated the diff for D87608: [ARM][MVE] Tail-predication: use unsigned SCEV ranges for tripcount.

test case clean up.

Mon, Sep 14, 6:11 AM · Restricted Project
SjoerdMeijer requested review of D87608: [ARM][MVE] Tail-predication: use unsigned SCEV ranges for tripcount.
Mon, Sep 14, 6:07 AM · Restricted Project
SjoerdMeijer committed rG676febc044ec: [ARM][MVE] Tail-predication: check get.active.lane.mask's TC value (authored by SjoerdMeijer).
[ARM][MVE] Tail-predication: check get.active.lane.mask's TC value
Mon, Sep 14, 3:32 AM
SjoerdMeijer closed D86074: [ARM][MVE] Tail-predication: check get.active.lane.mask's TC value.
Mon, Sep 14, 3:32 AM · Restricted Project
SjoerdMeijer added a comment to D86074: [ARM][MVE] Tail-predication: check get.active.lane.mask's TC value.

Thanks for that, and agreed with your remarks. I think this is already a bit more generic/flexible and thus better than what we had, but certainly isn't fully generic. I am willing to review this once that becomes important. Then, this logic has to be moved to Scalarevolution and be made generic.

Mon, Sep 14, 2:54 AM · Restricted Project

Thu, Sep 10

SjoerdMeijer added inline comments to D46884: [AArch64] Cortex-A55 scheduler model.
Thu, Sep 10, 10:48 AM
SjoerdMeijer updated the diff for D86074: [ARM][MVE] Tail-predication: check get.active.lane.mask's TC value.

Cheers, comments addressed.

Thu, Sep 10, 5:40 AM · Restricted Project

Wed, Sep 9

SjoerdMeijer added inline comments to D86074: [ARM][MVE] Tail-predication: check get.active.lane.mask's TC value.
Wed, Sep 9, 7:47 AM · Restricted Project
SjoerdMeijer updated the diff for D86074: [ARM][MVE] Tail-predication: check get.active.lane.mask's TC value.

This is a (partial) rewrite of the patch after we changed the semantics of get.active.lane.mask to accept the loop tripcount as its second argument, and not the backedge-taken count. This now implements several checks to see if the tripcount belongs to this loop.

Wed, Sep 9, 7:38 AM · Restricted Project
SjoerdMeijer committed rG8cb8cea1bd7f: [ARM] Fixup of a few test cases. NFC. (authored by SjoerdMeijer).
[ARM] Fixup of a few test cases. NFC.
Wed, Sep 9, 3:16 AM

Tue, Sep 8

SjoerdMeijer accepted D87280: [ARM] Try to rematerialize VCTP instructions.

Rebased after pre-committing the test, of which I've changed the function name too.

Tue, Sep 8, 6:58 AM · Restricted Project
SjoerdMeijer added inline comments to D87280: [ARM] Try to rematerialize VCTP instructions.
Tue, Sep 8, 5:43 AM · Restricted Project

Mon, Sep 7

SjoerdMeijer added inline comments to D86784: [ARM] Skip combining base updates for vld1x NEON intrinsics.
Mon, Sep 7, 7:21 AM · Restricted Project
SjoerdMeijer committed rG288c582fc939: Follow up of rG5f1cad4d296a, slightly reduced test case. NFC. (authored by SjoerdMeijer).
Follow up of rG5f1cad4d296a, slightly reduced test case. NFC.
Mon, Sep 7, 7:12 AM
SjoerdMeijer added a comment to D75512: [LoopVectorizer][ARM] Add preferInloopReduction target hook..

There are some tests for 64bit reductions. We will probably want to enable inloop reductions for them in the future too, as we have the instructions. That will require a lot of costmodel improvements though.

Mon, Sep 7, 3:21 AM · Restricted Project
SjoerdMeijer updated the diff for D87228: [Lint] Add check for intrinsic get.active.lane.mask.

Updated test

Mon, Sep 7, 2:39 AM · Restricted Project
SjoerdMeijer requested review of D87228: [Lint] Add check for intrinsic get.active.lane.mask.
Mon, Sep 7, 2:34 AM · Restricted Project
SjoerdMeijer accepted D86525: [ARM][CostModel] CodeSize costs for i1 arith ops.
Mon, Sep 7, 1:24 AM · Restricted Project
SjoerdMeijer added inline comments to D86525: [ARM][CostModel] CodeSize costs for i1 arith ops.
Mon, Sep 7, 1:16 AM · Restricted Project
SjoerdMeijer added a comment to D86147: [LangRef] Revise semantics of get.active.lane.mask.

Hi Luke, thanks for sharing your thoughts. I agree with your analysis. The in-tree vector extension that I am aware of that supports first faulting loads is Arm's SVE. While I work on Arm's MVE, I hope and think this is useful for SVE (and other targets) too, i.e. I think ffirst mask capability can be used. But since the devil is in the details here, an implementation would need to prove this. Hopefully that happens soon.

Mon, Sep 7, 12:42 AM · Restricted Project
SjoerdMeijer accepted D75512: [LoopVectorizer][ARM] Add preferInloopReduction target hook..

Looks good to me.

Mon, Sep 7, 12:27 AM · Restricted Project

Fri, Aug 28

SjoerdMeijer committed rG5f1cad4d296a: [ARM] Skip combining base updates for vld1x NEON intrinsics (authored by SjoerdMeijer).
[ARM] Skip combining base updates for vld1x NEON intrinsics
Fri, Aug 28, 12:32 PM
SjoerdMeijer closed D86784: [ARM] Skip combining base updates for vld1x NEON intrinsics.
Fri, Aug 28, 12:31 PM · Restricted Project
SjoerdMeijer added inline comments to D86784: [ARM] Skip combining base updates for vld1x NEON intrinsics.
Fri, Aug 28, 12:26 PM · Restricted Project
SjoerdMeijer requested review of D86784: [ARM] Skip combining base updates for vld1x NEON intrinsics.
Fri, Aug 28, 7:01 AM · Restricted Project
SjoerdMeijer added a comment to D86776: [ARM]{MVE] Enable MVE gathers and scatters by default.

Before we flip the switch, can you give an impression of the performance impact of this? Does this not regress cases, is it overall a win, etc.?

Fri, Aug 28, 5:33 AM · Restricted Project
SjoerdMeijer accepted D86613: [ARM][LowOverheadLoops] Liveouts and reductions.

LGTM

Fri, Aug 28, 12:25 AM · Restricted Project

Thu, Aug 27

SjoerdMeijer added inline comments to D86613: [ARM][LowOverheadLoops] Liveouts and reductions.
Thu, Aug 27, 6:50 AM · Restricted Project
SjoerdMeijer added a comment to D86702: [ARM] Fold predicate_cast(load) into vldr p0.

Sorry for kind of asking the usual testing question....but was curious if there's a negative test with a pattern where its condition isn't met, so alignment < 4, if that makes sense.

Thu, Aug 27, 6:38 AM · Restricted Project
SjoerdMeijer added a comment to D86301: [Verifier] Additional check for get.active.lane.mask.

Ah yes, thanks Eli!
This is reverted in rG1d8af682ef1d, and I will move this to Lint.

Thu, Aug 27, 3:06 AM · Restricted Project
SjoerdMeijer committed rGff6dbb231923: Follow up of rGca243b07276a: fixed a typo. NFC. (authored by SjoerdMeijer).
Follow up of rGca243b07276a: fixed a typo. NFC.
Thu, Aug 27, 2:54 AM
SjoerdMeijer added a reverting change for rG8d5f64c4edbc: [Verifier] Additional check for intrinsic get.active.lane.mask: rG1d8af682ef1d: Revert "[Verifier] Additional check for intrinsic get.active.lane.mask".
Thu, Aug 27, 1:28 AM
SjoerdMeijer committed rG1d8af682ef1d: Revert "[Verifier] Additional check for intrinsic get.active.lane.mask" (authored by SjoerdMeijer).
Revert "[Verifier] Additional check for intrinsic get.active.lane.mask"
Thu, Aug 27, 1:28 AM
SjoerdMeijer added a reverting change for D86301: [Verifier] Additional check for get.active.lane.mask: rG1d8af682ef1d: Revert "[Verifier] Additional check for intrinsic get.active.lane.mask".
Thu, Aug 27, 1:28 AM · Restricted Project
SjoerdMeijer committed rGca243b07276a: [LangRef] get.active.lane.mask can produce poison value (authored by SjoerdMeijer).
[LangRef] get.active.lane.mask can produce poison value
Thu, Aug 27, 12:58 AM
SjoerdMeijer closed D86637: [LangRef] get.active.lane.mask can produce poison.
Thu, Aug 27, 12:58 AM · Restricted Project

Wed, Aug 26

SjoerdMeijer accepted D86646: [llvm] [Thumb2] Test unusual length for active lane mask.

Thanks!

Wed, Aug 26, 11:07 AM · Restricted Project
SjoerdMeijer requested review of D86637: [LangRef] get.active.lane.mask can produce poison.
Wed, Aug 26, 9:33 AM · Restricted Project
SjoerdMeijer committed rGbda8fbe2d2af: [LV] Fallback strategies if tail-folding fails (authored by SjoerdMeijer).
[LV] Fallback strategies if tail-folding fails
Wed, Aug 26, 8:58 AM
SjoerdMeijer closed D79783: [LV] Fallback strategies if tail-folding fails.
Wed, Aug 26, 8:58 AM · Restricted Project
SjoerdMeijer added a comment to D86613: [ARM][LowOverheadLoops] Liveouts and reductions.

That's good amount of red/deletions!
Read this for the first time, and looks very reasonable. I have one question inline for now while I go over this again.

Wed, Aug 26, 8:33 AM · Restricted Project
SjoerdMeijer added a comment to D79783: [LV] Fallback strategies if tail-folding fails.

Thanks Dave, I will fix the spelling before committing.

Wed, Aug 26, 6:57 AM · Restricted Project
SjoerdMeijer added a comment to D86607: [RDA] Don't visit the BB of the instruction in getUniqueReachingMIDef.

I was just writing that I understood that creating a test for this one was very difficult? I.e., creating a small test case, was that this case? Looks like Sam has one now......should it not be part of this change? But anyway, it's fine I guess.

Wed, Aug 26, 3:37 AM · Restricted Project
SjoerdMeijer added a comment to D86585: [llvm] [DAG] Fix bug in llvm.get.active.lane.mask lowering.

One minor addition, this change is now tested for X86, but I thought it wouldn't do any harm to test this for ARM too. If you can and want to touch the ARM backend test, I wouldn't mind if you e.g. add test @create_mask7 to llvm/test/CodeGen/Thumb2/active_lane_mask.ll, otherwise I will just add that once this lands.

Wed, Aug 26, 2:02 AM · Restricted Project
SjoerdMeijer accepted D86585: [llvm] [DAG] Fix bug in llvm.get.active.lane.mask lowering.

LGTM. Thanks for fixing this.

Wed, Aug 26, 1:26 AM · Restricted Project
SjoerdMeijer added a comment to D86301: [Verifier] Additional check for get.active.lane.mask.

This check isn't appropriate. Given that the second argument to @llvm.get.active.lane.mask.v4i1.i32 is a variable, in general, it isn't appropriate to print an error in cases where we can prove the value is 0. You can add a Lint check if you want.

Wed, Aug 26, 12:57 AM · Restricted Project

Tue, Aug 25

SjoerdMeijer committed rG2002bb487898: [LangRef] Revise semantics of intrinsic get.active.lane.mask (authored by SjoerdMeijer).
[LangRef] Revise semantics of intrinsic get.active.lane.mask
Tue, Aug 25, 8:24 AM
SjoerdMeijer closed D86147: [LangRef] Revise semantics of get.active.lane.mask.
Tue, Aug 25, 8:24 AM · Restricted Project
SjoerdMeijer committed rG8d5f64c4edbc: [Verifier] Additional check for intrinsic get.active.lane.mask (authored by SjoerdMeijer).
[Verifier] Additional check for intrinsic get.active.lane.mask
Tue, Aug 25, 7:45 AM
SjoerdMeijer closed D86301: [Verifier] Additional check for get.active.lane.mask.
Tue, Aug 25, 7:45 AM · Restricted Project
SjoerdMeijer committed rG39522b1e1042: [SelectionDAG] Legalize intrinsic get.active.lane.mask (authored by SjoerdMeijer).
[SelectionDAG] Legalize intrinsic get.active.lane.mask
Tue, Aug 25, 7:01 AM
SjoerdMeijer closed D86302: [SelectionDAG] Modify legalizing intrinsic get.active.lane.mask.
Tue, Aug 25, 7:01 AM · Restricted Project
SjoerdMeijer committed rGc352e7fbda2f: [ARM][MVE] Tail-predication: remove the BTC + 1 overflow checks (authored by SjoerdMeijer).
[ARM][MVE] Tail-predication: remove the BTC + 1 overflow checks
Tue, Aug 25, 6:38 AM
SjoerdMeijer closed D86303: [ARM][MVE] Tail-predication: remove the BTC + 1 overflow checks.
Tue, Aug 25, 6:38 AM · Restricted Project
SjoerdMeijer committed rGae366479e8c6: [LV] get.active.lane.mask consuming tripcount instead of backedge-taken count (authored by SjoerdMeijer).
[LV] get.active.lane.mask consuming tripcount instead of backedge-taken count
Tue, Aug 25, 5:52 AM
SjoerdMeijer closed D86304: [LV] get.active.lane.mask consuming/emitting TripCount instead of backedge-taken count.
Tue, Aug 25, 5:52 AM · Restricted Project
SjoerdMeijer updated the diff for D86301: [Verifier] Additional check for get.active.lane.mask.

Cheers, that's indeed a bit shorter.

Tue, Aug 25, 1:53 AM · Restricted Project
SjoerdMeijer updated the diff for D86147: [LangRef] Revise semantics of get.active.lane.mask.

Thanks for catching that.
(weird, thought I had fixed that, but cheers)

Tue, Aug 25, 1:17 AM · Restricted Project
SjoerdMeijer added inline comments to D86301: [Verifier] Additional check for get.active.lane.mask.
Tue, Aug 25, 12:48 AM · Restricted Project
SjoerdMeijer added inline comments to D86301: [Verifier] Additional check for get.active.lane.mask.
Tue, Aug 25, 12:36 AM · Restricted Project

Mon, Aug 24

SjoerdMeijer updated the diff for D86303: [ARM][MVE] Tail-predication: remove the BTC + 1 overflow checks.

Addressed comments

Mon, Aug 24, 7:16 AM · Restricted Project
SjoerdMeijer added inline comments to D86303: [ARM][MVE] Tail-predication: remove the BTC + 1 overflow checks.
Mon, Aug 24, 7:04 AM · Restricted Project
SjoerdMeijer added inline comments to D86303: [ARM][MVE] Tail-predication: remove the BTC + 1 overflow checks.
Mon, Aug 24, 6:30 AM · Restricted Project
SjoerdMeijer updated the diff for D86304: [LV] get.active.lane.mask consuming/emitting TripCount instead of backedge-taken count.

Comments addressed.

Mon, Aug 24, 6:00 AM · Restricted Project
SjoerdMeijer added inline comments to D86304: [LV] get.active.lane.mask consuming/emitting TripCount instead of backedge-taken count.
Mon, Aug 24, 5:22 AM · Restricted Project
SjoerdMeijer updated the diff for D86301: [Verifier] Additional check for get.active.lane.mask.

Yep, thanks, now with extra tests for (i32 1) and (i32 -1).

Mon, Aug 24, 5:10 AM · Restricted Project
SjoerdMeijer updated the diff for D86147: [LangRef] Revise semantics of get.active.lane.mask.

Okidoki, now with that change.

Mon, Aug 24, 4:45 AM · Restricted Project

Fri, Aug 21

SjoerdMeijer added a comment to D79783: [LV] Fallback strategies if tail-folding fails.

Thanks for looking Dave.
I am addressing your remarks, also find some replies inline.

Fri, Aug 21, 7:15 AM · Restricted Project
SjoerdMeijer updated the diff for D86303: [ARM][MVE] Tail-predication: remove the BTC + 1 overflow checks.

Now with all tests fixed.

Fri, Aug 21, 3:39 AM · Restricted Project

Thu, Aug 20

SjoerdMeijer retitled D86304: [LV] get.active.lane.mask consuming/emitting TripCount instead of backedge-taken count from [LV] get.active.lane.mask consuming/emitting TripCount insteat of backedge-taken count to [LV] get.active.lane.mask consuming/emitting TripCount instead of backedge-taken count.
Thu, Aug 20, 11:10 AM · Restricted Project
SjoerdMeijer added a comment to D86147: [LangRef] Revise semantics of get.active.lane.mask.

I have removed the WIP tag because I think this is behaving as expected in the patches that I adapted to this new behaviour (D86304, D86302, D86301, and D86303).

Thu, Aug 20, 11:08 AM · Restricted Project
SjoerdMeijer retitled D86147: [LangRef] Revise semantics of get.active.lane.mask from [LangRef] WIP: Revise semantics of get.active.lane.mask to [LangRef] Revise semantics of get.active.lane.mask.
Thu, Aug 20, 10:59 AM · Restricted Project
SjoerdMeijer added inline comments to D86302: [SelectionDAG] Modify legalizing intrinsic get.active.lane.mask.
Thu, Aug 20, 10:56 AM · Restricted Project
SjoerdMeijer requested review of D86304: [LV] get.active.lane.mask consuming/emitting TripCount instead of backedge-taken count.
Thu, Aug 20, 9:42 AM · Restricted Project
SjoerdMeijer requested review of D86303: [ARM][MVE] Tail-predication: remove the BTC + 1 overflow checks.
Thu, Aug 20, 9:33 AM · Restricted Project
SjoerdMeijer requested review of D86302: [SelectionDAG] Modify legalizing intrinsic get.active.lane.mask.
Thu, Aug 20, 9:22 AM · Restricted Project
SjoerdMeijer requested review of D86301: [Verifier] Additional check for get.active.lane.mask.
Thu, Aug 20, 9:15 AM · Restricted Project