Page MenuHomePhabricator

asbirlea (Alina Sbirlea)
User

Projects

User does not belong to any projects.

User Details

User Since
Feb 10 2016, 9:51 AM (175 w, 6 d)

Recent Activity

Fri, Jun 21

asbirlea created D63680: [LoopRotate + MemorySSA] Keep an <instruction-cloned instruction> map..
Fri, Jun 21, 4:18 PM · Restricted Project

Thu, Jun 20

asbirlea committed rGd0b11698cdf8: [LICM & MSSA] Limit unsafe sinking and hoisting. (authored by asbirlea).
[LICM & MSSA] Limit unsafe sinking and hoisting.
Thu, Jun 20, 2:07 PM
asbirlea committed rL363982: [LICM & MSSA] Limit unsafe sinking and hoisting..
[LICM & MSSA] Limit unsafe sinking and hoisting.
Thu, Jun 20, 2:06 PM
asbirlea closed D63582: [LICM & MSSA] Limit unsafe sinking and hoisting..
Thu, Jun 20, 2:06 PM · Restricted Project
asbirlea updated the diff for D63582: [LICM & MSSA] Limit unsafe sinking and hoisting..

Address comments and add testcase.

Thu, Jun 20, 1:41 PM · Restricted Project
asbirlea added a comment to rL362901: [MIR] Add simple PRE pass to MachineCSE.

@anton-afanasyev: FYI, we're still seeing this and it presents itself as a miscompile.
We're still working on getting a test case but it's proving to be very difficult.

Thu, Jun 20, 10:16 AM

Wed, Jun 19

asbirlea created D63582: [LICM & MSSA] Limit unsafe sinking and hoisting..
Wed, Jun 19, 5:18 PM · Restricted Project
asbirlea committed rG109d2ea1532c: [MemorySSA] Cleanup trivial phis. (authored by asbirlea).
[MemorySSA] Cleanup trivial phis.
Wed, Jun 19, 2:34 PM
asbirlea committed rL363880: [MemorySSA] Cleanup trivial phis..
[MemorySSA] Cleanup trivial phis.
Wed, Jun 19, 2:30 PM
asbirlea closed D63354: [MemorySSA] Cleanup trivial phis..
Wed, Jun 19, 2:30 PM · Restricted Project
asbirlea committed rG238b8e62b62d: [MemorySSA] Use GraphDiff info when computing IDF. (authored by asbirlea).
[MemorySSA] Use GraphDiff info when computing IDF.
Wed, Jun 19, 2:18 PM
asbirlea committed rL363879: [MemorySSA] Use GraphDiff info when computing IDF..
[MemorySSA] Use GraphDiff info when computing IDF.
Wed, Jun 19, 2:17 PM
asbirlea closed D63443: [MemorySSA] Use GraphDiff info when computing IDF..
Wed, Jun 19, 2:17 PM · Restricted Project

Mon, Jun 17

asbirlea updated the diff for D63147: [MemorySSA] Add additional verification for phis..

Update per the comment suggestion.

Mon, Jun 17, 2:37 PM · Restricted Project
asbirlea added inline comments to D63147: [MemorySSA] Add additional verification for phis..
Mon, Jun 17, 2:28 PM · Restricted Project
asbirlea committed rG7a0098aa6e36: [MemorySSA] Don't use template when the clone is a simplified instruction. (authored by asbirlea).
[MemorySSA] Don't use template when the clone is a simplified instruction.
Mon, Jun 17, 11:57 AM
asbirlea committed rL363597: [MemorySSA] Don't use template when the clone is a simplified instruction..
[MemorySSA] Don't use template when the clone is a simplified instruction.
Mon, Jun 17, 11:57 AM
asbirlea closed D63355: [MemorySSA] Don't use template when the clone is a simplified instruction..
Mon, Jun 17, 11:57 AM · Restricted Project
asbirlea updated the diff for D63355: [MemorySSA] Don't use template when the clone is a simplified instruction..

Address comments.

Mon, Jun 17, 11:40 AM · Restricted Project
asbirlea committed rG05f77803f45a: [MemorySSA] Add all MemoryPhis before filling their values. (authored by asbirlea).
[MemorySSA] Add all MemoryPhis before filling their values.
Mon, Jun 17, 11:14 AM
asbirlea committed rL363590: [MemorySSA] Add all MemoryPhis before filling their values..
[MemorySSA] Add all MemoryPhis before filling their values.
Mon, Jun 17, 11:14 AM
asbirlea closed D63353: [MemorySSA] Add all MemoryPhis before filling their values..
Mon, Jun 17, 11:14 AM · Restricted Project
asbirlea created D63443: [MemorySSA] Use GraphDiff info when computing IDF..
Mon, Jun 17, 10:07 AM · Restricted Project
asbirlea added a comment to D63389: [IDF] Generalize IDFCalculator to be used with Clang's CFG.

We do very much need GraphDiff in the IDFCalculator when doing updates in MemorySSA. I don't know how I missed this, since it was the very reason it was added, but thanks a lot for catching it.
I'll send a patch with the fix shortly.

Mon, Jun 17, 10:02 AM · Restricted Project

Fri, Jun 14

asbirlea created D63355: [MemorySSA] Don't use template when the clone is a simplified instruction..
Fri, Jun 14, 12:14 PM · Restricted Project
asbirlea created D63354: [MemorySSA] Cleanup trivial phis..
Fri, Jun 14, 12:11 PM · Restricted Project
asbirlea created D63353: [MemorySSA] Add all MemoryPhis before filling their values..
Fri, Jun 14, 12:08 PM · Restricted Project

Wed, Jun 12

asbirlea added a comment to rL362901: [MIR] Add simple PRE pass to MachineCSE.

@anton-afanasyev:
The test failures occur in Halide generated code, with the HVX backend. They are not public.
r363164 does not fix the tests; it does change an erroneous output to another erroneous one.

Wed, Jun 12, 2:11 PM
asbirlea added a comment to rL362901: [MIR] Add simple PRE pass to MachineCSE.

FYI, we're seeing test failures due to this revision. Currently trying to isolate a testcase, but it may be challenging to minimize if the failures are due to a miscompile.

Wed, Jun 12, 12:06 PM

Tue, Jun 11

asbirlea updated the diff for D63147: [MemorySSA] Add additional verification for phis..

Add NDEBUG.

Tue, Jun 11, 12:11 PM · Restricted Project
asbirlea committed rGcb4ed8a7bc9c: [MemorySSA] When applying updates, clean unnecessary Phis. (authored by asbirlea).
[MemorySSA] When applying updates, clean unnecessary Phis.
Tue, Jun 11, 12:08 PM
asbirlea committed rL363094: [MemorySSA] When applying updates, clean unnecessary Phis..
[MemorySSA] When applying updates, clean unnecessary Phis.
Tue, Jun 11, 12:06 PM
asbirlea closed D63033: [MemorySSA] When applying updates, clean unnecessary Phis..
Tue, Jun 11, 12:06 PM · Restricted Project
asbirlea added a comment to D62536: Only passes that preserve MemorySSA must mark it as preserved..

I wasn't yet able to reproduce this with opt -aa-pipeline=default -passes="default<O3>", only with clang -cc1 -O3 -fexperimental-new-pass-manager.
I'll keep working on getting a test, but I checked this in for now, since it's necessary to fix crashes for the combination of MemorySSA + NewPassManager.

Tue, Jun 11, 11:43 AM · Restricted Project
asbirlea committed rG3cef1f7d64f2: Only passes that preserve MemorySSA must mark it as preserved. (authored by asbirlea).
Only passes that preserve MemorySSA must mark it as preserved.
Tue, Jun 11, 11:25 AM
asbirlea committed rL363091: Only passes that preserve MemorySSA must mark it as preserved..
Only passes that preserve MemorySSA must mark it as preserved.
Tue, Jun 11, 11:25 AM
asbirlea closed D62536: Only passes that preserve MemorySSA must mark it as preserved..
Tue, Jun 11, 11:25 AM · Restricted Project
asbirlea updated the diff for D63033: [MemorySSA] When applying updates, clean unnecessary Phis..

Add test.

Tue, Jun 11, 11:23 AM · Restricted Project
asbirlea created D63147: [MemorySSA] Add additional verification for phis..
Tue, Jun 11, 9:57 AM · Restricted Project

Fri, Jun 7

asbirlea created D63033: [MemorySSA] When applying updates, clean unnecessary Phis..
Fri, Jun 7, 4:08 PM · Restricted Project
asbirlea committed rGeaea538d18c1: [DomTreeUpdater] Add all insert before all delete updates to reduce compile… (authored by asbirlea).
[DomTreeUpdater] Add all insert before all delete updates to reduce compile…
Fri, Jun 7, 1:43 PM
asbirlea committed rL362839: [DomTreeUpdater] Add all insert before all delete updates to reduce compile….
[DomTreeUpdater] Add all insert before all delete updates to reduce compile…
Fri, Jun 7, 1:43 PM
asbirlea closed D62981: [DomTreeUpdater] Add all insert before all delete updates to reduce compile time..
Fri, Jun 7, 1:43 PM · Restricted Project
asbirlea updated the diff for D62981: [DomTreeUpdater] Add all insert before all delete updates to reduce compile time..

Add detalied comment.

Fri, Jun 7, 11:24 AM · Restricted Project
Herald added a project to D54730: [DomTree] Fix order of domtree updates in MergeBlockIntoPredecessor.: Restricted Project.

Since I sent out D62981 with identical changes and the same motivation (thank you for pointing this out @NutshellySima), please let me add a small comment here.

Fri, Jun 7, 9:33 AM · Restricted Project

Thu, Jun 6

asbirlea created D62981: [DomTreeUpdater] Add all insert before all delete updates to reduce compile time..
Thu, Jun 6, 1:53 PM · Restricted Project
asbirlea added a comment to D62751: [Utils] Clean another duplicated util method..

Following up: I can match the previous performance if all update inserts are done first. I'll send a patch for this. But perhaps this should be done inside DTU (i.e. sort updates)?

Thu, Jun 6, 12:33 PM · Restricted Project
asbirlea added a comment to D62751: [Utils] Clean another duplicated util method..

It appears the difference is in the way the DT updates are done.
+@kuhar, @NutshellySima: Could you please take a look at the way the updates are done in MergeBlockIntoPredecessor vs (the removed) foldBlockIntoPredecessor? I tried to use a Lazy strategy and flush after doing all the potential merges, but the regression is still there. Can we get the same performance with the DomTreeUpdater here?

Thu, Jun 6, 12:21 PM · Restricted Project
asbirlea updated subscribers of D62751: [Utils] Clean another duplicated util method..
Thu, Jun 6, 12:20 PM · Restricted Project

Tue, Jun 4

asbirlea committed rGbfceed49cec1: [Utils] Clean another duplicated util method. (authored by asbirlea).
[Utils] Clean another duplicated util method.
Tue, Jun 4, 11:45 AM
asbirlea committed rL362538: [Utils] Clean another duplicated util method..
[Utils] Clean another duplicated util method.
Tue, Jun 4, 11:42 AM
asbirlea closed D62751: [Utils] Clean another duplicated util method..
Tue, Jun 4, 11:42 AM · Restricted Project

Mon, Jun 3

asbirlea updated the summary of D62751: [Utils] Clean another duplicated util method..
Mon, Jun 3, 2:03 PM · Restricted Project
asbirlea updated the diff for D62751: [Utils] Clean another duplicated util method..

No reason for not removing the unused method.
Squashed a local NFC commit I had into this one.

Mon, Jun 3, 2:03 PM · Restricted Project

Fri, May 31

asbirlea created D62751: [Utils] Clean another duplicated util method..
Fri, May 31, 2:22 PM · Restricted Project
asbirlea accepted D58700: [JumpThreading] Replace MergeBasicBlockIntoOnlyPred() with MergeBlockIntoPredecessor().

LGTM.

Fri, May 31, 1:50 PM · Restricted Project
asbirlea added inline comments to D62656: Make SwitchInstProfUpdateWrapper safer.
Fri, May 31, 1:22 PM · Restricted Project

Thu, May 30

asbirlea added a comment to D48202: Generalize MergeBlockIntoPredecessor. Replace uses of MergeBasicBlockIntoOnlyPred..

Am I right that there is yet another method doing the same thing?
lib/Transforms/Utils/LoopUnroll.cpp:foldBlockIntoPredecessor

Thu, May 30, 1:40 PM · Restricted Project

Tue, May 28

asbirlea added a comment to D62126: [CorrelatedValuePropagation] Fix prof branch_weights metadata handling for SwitchInst.

Thank you for the quick action!

Tue, May 28, 2:34 PM · Restricted Project
asbirlea added a comment to D62126: [CorrelatedValuePropagation] Fix prof branch_weights metadata handling for SwitchInst.

FYI, we're seeing a crash in llvm::SwitchInstProfUpdateWrapper::getProfBranchWeights() following this patch.
I'm trying to get a testcase available.

Tue, May 28, 2:06 PM · Restricted Project
asbirlea added a comment to D60606: [SimpleLoopUnswitch] Implement handling of prof branch_weights metadata for SwitchInst.

I'd prefer to have Chandler approve this, since I'm not familiar with what you're trying to do here.
Some minor comments inline.

Tue, May 28, 11:26 AM · Restricted Project
asbirlea created D62536: Only passes that preserve MemorySSA must mark it as preserved..
Tue, May 28, 11:04 AM · Restricted Project

May 24 2019

asbirlea committed rGb4c756dc1cde: Mark tests as x86. (authored by asbirlea).
Mark tests as x86.
May 24 2019, 2:51 PM
asbirlea committed rL361674: Mark tests as x86..
Mark tests as x86.
May 24 2019, 2:51 PM
asbirlea committed rC361674: Mark tests as x86..
Mark tests as x86.
May 24 2019, 2:46 PM
asbirlea committed rG21efe2afed7b: [NewPassManager] Add tuning option: LoopUnrolling [clang-change] (authored by asbirlea).
[NewPassManager] Add tuning option: LoopUnrolling [clang-change]
May 24 2019, 10:39 AM
asbirlea committed rC361653: [NewPassManager] Add tuning option: LoopUnrolling [clang-change].
[NewPassManager] Add tuning option: LoopUnrolling [clang-change]
May 24 2019, 10:39 AM
asbirlea committed rL361653: [NewPassManager] Add tuning option: LoopUnrolling [clang-change].
[NewPassManager] Add tuning option: LoopUnrolling [clang-change]
May 24 2019, 10:39 AM
asbirlea closed D61620: [NewPassManager] Add tuning option: LoopUnrolling [clang-change].
May 24 2019, 10:39 AM · Restricted Project, Restricted Project

May 23 2019

asbirlea committed rGf2e41dd6ed1a: Use clang_cc1 instead of clang in CodeGen test. (authored by asbirlea).
Use clang_cc1 instead of clang in CodeGen test.
May 23 2019, 3:08 PM
asbirlea committed rL361562: Use clang_cc1 instead of clang in CodeGen test..
Use clang_cc1 instead of clang in CodeGen test.
May 23 2019, 3:08 PM
asbirlea committed rC361562: Use clang_cc1 instead of clang in CodeGen test..
Use clang_cc1 instead of clang in CodeGen test.
May 23 2019, 3:08 PM
asbirlea committed rGd82ddfa7c37d: [NewPassManager] Add tuning option: ForgetAllSCEVInLoopUnroll [NFC]. (authored by asbirlea).
[NewPassManager] Add tuning option: ForgetAllSCEVInLoopUnroll [NFC].
May 23 2019, 2:52 PM
asbirlea committed rL361560: [NewPassManager] Add tuning option: ForgetAllSCEVInLoopUnroll [NFC]..
[NewPassManager] Add tuning option: ForgetAllSCEVInLoopUnroll [NFC].
May 23 2019, 2:52 PM
asbirlea closed D61612: [NewPassManager] Add tuning option: ForgetAllSCEVInLoopUnroll [NFC]..
May 23 2019, 2:52 PM · Restricted Project
asbirlea committed rG9925ef78ce2c: Update breaking test. (authored by asbirlea).
Update breaking test.
May 23 2019, 12:49 PM
asbirlea committed rC361542: Update breaking test..
Update breaking test.
May 23 2019, 12:48 PM
asbirlea committed rL361542: Update breaking test..
Update breaking test.
May 23 2019, 12:48 PM
asbirlea committed rGe4b27869c60c: [NewPassManager] Add tuning option: LoopUnrolling [NFC]. (authored by asbirlea).
[NewPassManager] Add tuning option: LoopUnrolling [NFC].
May 23 2019, 12:37 PM
asbirlea committed rL361540: [NewPassManager] Add tuning option: LoopUnrolling [NFC]..
[NewPassManager] Add tuning option: LoopUnrolling [NFC].
May 23 2019, 12:37 PM
asbirlea closed D61618: [NewPassManager] Add tuning option: LoopUnrolling [NFC]..
May 23 2019, 12:37 PM · Restricted Project
asbirlea added a comment to D61618: [NewPassManager] Add tuning option: LoopUnrolling [NFC]..

Ack, will do!

May 23 2019, 12:25 PM · Restricted Project
asbirlea committed rG63729b0c49d2: [SLPVectorizer] Set flag to previous default. (authored by asbirlea).
[SLPVectorizer] Set flag to previous default.
May 23 2019, 12:05 PM
asbirlea added a comment to D61616: [NewPassManager] Add tuning option: SLPVectorization [NFC]..

@jvesely: Yes, you are right. D61955 should resolve this.

May 23 2019, 12:05 PM · Restricted Project
asbirlea committed rL361537: [SLPVectorizer] Set flag to previous default..
[SLPVectorizer] Set flag to previous default.
May 23 2019, 12:05 PM
asbirlea closed D61955: [SLPVectorizer] Set flag to previous default..
May 23 2019, 12:05 PM · Restricted Project
asbirlea committed rG267ac925fb4e: [NewPassManager] Add tuning option: SLPVectorization [clang-change] (authored by asbirlea).
[NewPassManager] Add tuning option: SLPVectorization [clang-change]
May 23 2019, 11:52 AM
asbirlea committed rL361534: [NewPassManager] Add tuning option: SLPVectorization [clang-change].
[NewPassManager] Add tuning option: SLPVectorization [clang-change]
May 23 2019, 11:52 AM
asbirlea committed rC361534: [NewPassManager] Add tuning option: SLPVectorization [clang-change].
[NewPassManager] Add tuning option: SLPVectorization [clang-change]
May 23 2019, 11:52 AM
asbirlea closed D61617: [NewPassManager] Add tuning option: SLPVectorization [clang-change].
May 23 2019, 11:51 AM · Restricted Project

May 17 2019

asbirlea added a comment to rL356351: [CodeGen] Defined MVTs v3i32, v3f32, v5i32, v5f32.

Please see: https://bugs.llvm.org/show_bug.cgi?id=41916

May 17 2019, 10:55 AM

May 15 2019

asbirlea created D61955: [SLPVectorizer] Set flag to previous default..
May 15 2019, 11:01 AM · Restricted Project

May 14 2019

asbirlea committed rG80c6e79602f3: [MemorySSA] LoopSimplify preserves MemorySSA only when flag is flipped. (authored by asbirlea).
[MemorySSA] LoopSimplify preserves MemorySSA only when flag is flipped.
May 14 2019, 11:06 AM
asbirlea committed rL360697: [MemorySSA] LoopSimplify preserves MemorySSA only when flag is flipped..
[MemorySSA] LoopSimplify preserves MemorySSA only when flag is flipped.
May 14 2019, 11:04 AM
asbirlea accepted D61904: [LICM] Allow AliasSetMap to contain top-level loops..

Thank you for this! This LGTM.
The caching of AliasSetTracker objects will go away when MemorySSA is enabled.

May 14 2019, 9:11 AM · Restricted Project

May 8 2019

asbirlea added a comment to D61618: [NewPassManager] Add tuning option: LoopUnrolling [NFC]..

I think the test in the clang patch that needs this should do?

May 8 2019, 11:32 AM · Restricted Project
asbirlea updated the diff for D61620: [NewPassManager] Add tuning option: LoopUnrolling [clang-change].

Update test.

May 8 2019, 11:29 AM · Restricted Project, Restricted Project
asbirlea committed rG458c7339e106: [NewPassManager] Add tuning option: SLPVectorization [NFC]. (authored by asbirlea).
[NewPassManager] Add tuning option: SLPVectorization [NFC].
May 8 2019, 10:58 AM
asbirlea committed rL360276: [NewPassManager] Add tuning option: SLPVectorization [NFC]..
[NewPassManager] Add tuning option: SLPVectorization [NFC].
May 8 2019, 10:58 AM
asbirlea closed D61616: [NewPassManager] Add tuning option: SLPVectorization [NFC]..
May 8 2019, 10:57 AM · Restricted Project