Page MenuHomePhabricator

Please use GitHub pull requests for new patches. Phabricator shutdown timeline

People Whitney

Whitney (Whitney Tsang)
User

Projects

User does not belong to any projects.

User Details

User Since
Jan 29 2019, 6:22 PM (244 w, 2 h)

Recent Activity
View All

May 8 2023

Whitney committed rGa2ab6a5e2b8d: [mlir][CallOpInterface] Add `setCalleeFromCallable` method (authored by Whitney).
[mlir][CallOpInterface] Add `setCalleeFromCallable` method
May 8 2023, 6:08 AM · Restricted Project, Restricted Project, Restricted Project
Whitney closed D149763: [mlir][CallOpInterface] Add `setCalleeFromCallable` method.
May 8 2023, 6:08 AM · Restricted Project, Restricted Project, Restricted Project

May 4 2023

Whitney added a comment to D149763: [mlir][CallOpInterface] Add `setCalleeFromCallable` method.
In D149763#4318835, @gysit wrote:

Thanks for the longer commit message. It seems there is still a compilation error though.

May 4 2023, 10:04 AM · Restricted Project, Restricted Project, Restricted Project
Whitney updated the diff for D149763: [mlir][CallOpInterface] Add `setCalleeFromCallable` method.

changed comment.

May 4 2023, 9:57 AM · Restricted Project, Restricted Project, Restricted Project
Whitney updated the diff for D149763: [mlir][CallOpInterface] Add `setCalleeFromCallable` method.

Added mlir:: in FIROps.td.

May 4 2023, 8:07 AM · Restricted Project, Restricted Project, Restricted Project
Whitney added a comment to D149763: [mlir][CallOpInterface] Add `setCalleeFromCallable` method.
In D149763#4317844, @gysit wrote:

Can you explain the motivation for the change (ideally in the commit message)? Could you achieve your goal also by changing the specific call operation you are working with?

May 4 2023, 7:12 AM · Restricted Project, Restricted Project, Restricted Project
Whitney updated the diff for D149763: [mlir][CallOpInterface] Add `setCalleeFromCallable` method.
May 4 2023, 7:09 AM · Restricted Project, Restricted Project, Restricted Project
Whitney updated the summary of D149763: [mlir][CallOpInterface] Add `setCalleeFromCallable` method.
May 4 2023, 7:03 AM · Restricted Project, Restricted Project, Restricted Project
Whitney updated the summary of D149763: [mlir][CallOpInterface] Add `setCalleeFromCallable` method.
May 4 2023, 6:56 AM · Restricted Project, Restricted Project, Restricted Project

May 3 2023

Whitney requested review of D149763: [mlir][CallOpInterface] Add `setCalleeFromCallable` method.
May 3 2023, 9:24 AM · Restricted Project, Restricted Project, Restricted Project

Feb 24 2023

Whitney abandoned D144648: [MLIR] Allow nested operations to be transformed.

Given that there is a workaround, closing this review.

Feb 24 2023, 7:37 AM · Restricted Project, Restricted Project

Feb 23 2023

Whitney added inline comments to D144648: [MLIR] Allow nested operations to be transformed.
Feb 23 2023, 2:19 PM · Restricted Project, Restricted Project
Whitney added inline comments to D144648: [MLIR] Allow nested operations to be transformed.
Feb 23 2023, 2:17 PM · Restricted Project, Restricted Project
Whitney requested review of D144648: [MLIR] Allow nested operations to be transformed.
Feb 23 2023, 8:26 AM · Restricted Project, Restricted Project

Sep 9 2022

Whitney accepted D133587: Loop names used in reporting can grow very large.
Sep 9 2022, 10:13 AM · Restricted Project, Restricted Project, Restricted Project

Sep 6 2022

Whitney accepted D131606: [Loop Fusion] Sink/hoist memory instructions between loop fusion candidates.

I agree with @congzhe that CodeMoverUtils fits better for the functionality that is added in this patch. I am ok that this being a 2-steps process, where you first add the functionality in loop fusion pass, and then improve CodeMoverUtils.
Given that @aaronkintel added the TODOs, I can approve it, but please do improve CodeMoverUtils.

Sep 6 2022, 12:49 PM · Restricted Project, Restricted Project

Sep 2 2022

Whitney accepted D131469: [Clang] change default storing path of `-ftime-trace`.
Sep 2 2022, 7:25 AM · Restricted Project, Restricted Project

Aug 29 2022

Whitney added inline comments to D131469: [Clang] change default storing path of `-ftime-trace`.
Aug 29 2022, 7:25 PM · Restricted Project, Restricted Project
Whitney added inline comments to D131469: [Clang] change default storing path of `-ftime-trace`.
Aug 29 2022, 8:28 AM · Restricted Project, Restricted Project

Aug 24 2022

Whitney added inline comments to D131606: [Loop Fusion] Sink/hoist memory instructions between loop fusion candidates.
Aug 24 2022, 1:23 PM · Restricted Project, Restricted Project

Aug 22 2022

Whitney added a comment to D132199: [LoopPassManager] Ensure to construct loop nests with the outermost loop.

LGTM other than the comment from Bardia.

Aug 22 2022, 10:12 AM · Unknown Object (Project), Restricted Project, Restricted Project
Whitney added inline comments to D131469: [Clang] change default storing path of `-ftime-trace`.
Aug 22 2022, 10:06 AM · Restricted Project, Restricted Project
Whitney added inline comments to D131606: [Loop Fusion] Sink/hoist memory instructions between loop fusion candidates.
Aug 22 2022, 9:48 AM · Restricted Project, Restricted Project

Aug 19 2022

Whitney added a reviewer for D131606: [Loop Fusion] Sink/hoist memory instructions between loop fusion candidates: Restricted Project.
Aug 19 2022, 7:09 AM · Restricted Project, Restricted Project

Aug 11 2022

Whitney added inline comments to D131469: [Clang] change default storing path of `-ftime-trace`.
Aug 11 2022, 7:19 PM · Restricted Project, Restricted Project
Whitney added inline comments to D131469: [Clang] change default storing path of `-ftime-trace`.
Aug 11 2022, 8:53 AM · Restricted Project, Restricted Project

Aug 9 2022

Whitney added inline comments to D131469: [Clang] change default storing path of `-ftime-trace`.
Aug 9 2022, 6:22 AM · Restricted Project, Restricted Project
Whitney added inline comments to D131469: [Clang] change default storing path of `-ftime-trace`.
Aug 9 2022, 6:10 AM · Restricted Project, Restricted Project

Jun 30 2022

Whitney accepted D128048: Add a new clang option "-ftime-trace=<value>".
Jun 30 2022, 6:54 PM · Restricted Project, Restricted Project
Whitney added inline comments to D128048: Add a new clang option "-ftime-trace=<value>".
Jun 30 2022, 6:29 AM · Restricted Project, Restricted Project

Jun 22 2022

Whitney added a comment to D128048: Add a new clang option "-ftime-trace=<value>".
In D128048#3601579, @jamieschmeiser wrote:

Can you please use git rebase -i to collapse all the changes into a single change? If this isn't done, it is difficult to know what is being reviewed as the changes only show the differences since your last revision, not all of the changes.

Jun 22 2022, 6:44 AM · Restricted Project, Restricted Project

Jun 20 2022

Whitney added inline comments to D128048: Add a new clang option "-ftime-trace=<value>".
Jun 20 2022, 10:21 PM · Restricted Project, Restricted Project

Jun 17 2022

Whitney added a comment to D128048: Add a new clang option "-ftime-trace=<value>".

Can you please add some test cases?

Jun 17 2022, 6:24 AM · Restricted Project, Restricted Project

Apr 11 2022

Whitney committed rG80304c5f88f0: [LoopUnroll] Always respect user unroll pragma (authored by Whitney).
[LoopUnroll] Always respect user unroll pragma
Apr 11 2022, 11:33 AM · Restricted Project, Restricted Project
Whitney closed D119148: [LoopUnroll] Always respect user unroll pragma.
Apr 11 2022, 11:33 AM · Restricted Project, Restricted Project, Unknown Object (Project)

Apr 8 2022

Whitney added a comment to D119148: [LoopUnroll] Always respect user unroll pragma.
In D119148#3417337, @efriedma wrote:

PragmaUnrollThreshold has other uses; are you intentionally leaving those alone?

Apr 8 2022, 6:43 AM · Restricted Project, Restricted Project, Unknown Object (Project)

Apr 1 2022

Whitney accepted D118808: Loop Strength Reduce - Optimize unused IVs to final values in the exit block with SCEV.
Apr 1 2022, 7:02 AM · Restricted Project, Restricted Project, Unknown Object (Project)

Mar 11 2022

Whitney added inline comments to D118808: Loop Strength Reduce - Optimize unused IVs to final values in the exit block with SCEV.
Mar 11 2022, 2:02 PM · Restricted Project, Restricted Project, Unknown Object (Project)

Mar 7 2022

Whitney added a comment to D118808: Loop Strength Reduce - Optimize unused IVs to final values in the exit block with SCEV.

IndVarSimply calls rewriteLoopExitValues from LoopUtils to do this optimization. I tried to call the same function in LoopStrengthReduce, however rewriteLoopExitValues asserts that it requires: L->isRecursivelyLCSSAForm(*DT, *LI).

When calling rewriteLoopExitValues from LoopStrengthReduce, this assert is not met.

Mar 7 2022, 5:42 PM · Restricted Project, Restricted Project, Unknown Object (Project)

Feb 22 2022

Whitney added a comment to D119109: [clang][AIX] Enable LTO and teach the driver to pass options to the plugin.

I don't have any other comments other than those clang-format ones.

Feb 22 2022, 7:24 PM · Restricted Project
Whitney added a comment to D118808: Loop Strength Reduce - Optimize unused IVs to final values in the exit block with SCEV.

Is it possible to abstract the portion of the code in IndVarSimplify to simplify this code pattern, and call that function instead of creating your own ReplaceExitPHIsWithFinalVal?

Feb 22 2022, 4:04 PM · Restricted Project, Restricted Project, Unknown Object (Project)

Feb 18 2022

Whitney committed rGe7afbea8ca4e: [MemorySSA] Clear VisitedBlocks per query (authored by Whitney).
[MemorySSA] Clear VisitedBlocks per query
Feb 18 2022, 12:36 PM
Whitney closed D119898: [MemorySSA] Clear VisitedBlocks per query.
Feb 18 2022, 12:36 PM · Restricted Project, Unknown Object (Project)

Feb 17 2022

Whitney added a comment to D119148: [LoopUnroll] Always respect user unroll pragma.
In D119148#3330896, @efriedma wrote:

I think the limit is there just to prevent the compiler from exploding. If you try to unroll too much, you're going to end up with effectively infinite compile time and/or run out of memory.

I would expect it's unlikely to trigger the limit in practice; do you have a practical case where it matters?

Feb 17 2022, 8:16 PM · Restricted Project, Restricted Project, Unknown Object (Project)
Whitney added a comment to D119148: [LoopUnroll] Always respect user unroll pragma.

ping

Feb 17 2022, 10:43 AM · Restricted Project, Restricted Project, Unknown Object (Project)
Whitney updated the diff for D119898: [MemorySSA] Clear VisitedBlocks per query.

Verified that clearing at the beginning of insertDef and insertUse can resolve the problem shown in the test case.

Feb 17 2022, 10:43 AM · Restricted Project, Unknown Object (Project)

Feb 15 2022

Whitney added a reviewer for D118076: Sinking or hoisting instructions between loops before fusion: kbarton.
Feb 15 2022, 3:16 PM · Restricted Project, Restricted Project
Whitney updated the summary of D119898: [MemorySSA] Clear VisitedBlocks per query.
Feb 15 2022, 3:15 PM · Restricted Project, Unknown Object (Project)
Whitney requested review of D119898: [MemorySSA] Clear VisitedBlocks per query.
Feb 15 2022, 3:13 PM · Restricted Project, Unknown Object (Project)

Feb 14 2022

Whitney accepted D119107: [NFCI][clang][driver] Identify LTO plugin types and refine opt passing.
Feb 14 2022, 10:24 AM

Feb 8 2022

Whitney added inline comments to D119107: [NFCI][clang][driver] Identify LTO plugin types and refine opt passing.
Feb 8 2022, 5:34 PM
Whitney added inline comments to D119109: [clang][AIX] Enable LTO and teach the driver to pass options to the plugin.
Feb 8 2022, 5:23 PM · Restricted Project

Feb 7 2022

Whitney updated the diff for D119148: [LoopUnroll] Always respect user unroll pragma.
Feb 7 2022, 9:29 AM · Restricted Project, Restricted Project, Unknown Object (Project)
Whitney requested review of D119148: [LoopUnroll] Always respect user unroll pragma.
Feb 7 2022, 8:45 AM · Restricted Project, Restricted Project, Unknown Object (Project)

Jan 13 2022

Whitney committed rGcb6b9d3ae2cb: [LoopNest] Add new utilites (authored by Whitney).
[LoopNest] Add new utilites
Jan 13 2022, 2:19 PM
Whitney closed D115590: [LoopNest] Add new utilites.
Jan 13 2022, 2:19 PM · Restricted Project, Unknown Object (Project)
Whitney updated the diff for D115590: [LoopNest] Add new utilites.

address review comment

Jan 13 2022, 12:04 PM · Restricted Project, Unknown Object (Project)

Jan 11 2022

Whitney added inline comments to rG67a3331e4f6d: Inline extractMallocCall to sole use and delete [NFC].
Jan 11 2022, 5:27 PM
Whitney added inline comments to rG67a3331e4f6d: Inline extractMallocCall to sole use and delete [NFC].
Jan 11 2022, 4:28 PM

Jan 10 2022

Whitney added a comment to D115590: [LoopNest] Add new utilites.

ping

Jan 10 2022, 6:35 AM · Restricted Project, Unknown Object (Project)

Dec 20 2021

Whitney added inline comments to D115590: [LoopNest] Add new utilites.
Dec 20 2021, 7:18 AM · Restricted Project, Unknown Object (Project)

Dec 15 2021

Whitney updated the diff for D115590: [LoopNest] Add new utilites.
Dec 15 2021, 9:44 AM · Restricted Project, Unknown Object (Project)

Dec 11 2021

Whitney requested review of D115590: [LoopNest] Add new utilites.
Dec 11 2021, 6:13 PM · Restricted Project, Unknown Object (Project)

Dec 8 2021

Whitney accepted D114886: [LoopOptWG][LoopUnroll] Disable loop unroll when user explicitly asks for unroll-and-jam.
Dec 8 2021, 7:29 AM · Unknown Object (Project), Restricted Project

Dec 1 2021

Whitney added a project to D114886: [LoopOptWG][LoopUnroll] Disable loop unroll when user explicitly asks for unroll-and-jam: Unknown Object (Project).
Dec 1 2021, 9:11 AM · Unknown Object (Project), Restricted Project
Whitney added a project to D108087: [SimpleLoopUnswitch] Create SimpleLoopNestUnswitch pass: Unknown Object (Project).
Dec 1 2021, 8:51 AM · Unknown Object (Project), Restricted Project
Whitney added a project to D104636: [WIP][LoopIdiom] Teach LNIR to versioning loops and add runtime check: Unknown Object (Project).
Dec 1 2021, 8:27 AM · Restricted Project, Unknown Object (Project), Restricted Project
Whitney added a project to D104179: [LoopIdiom] Introduce LoopNestIdiomRecognize as an alternative: Unknown Object (Project).
Dec 1 2021, 8:27 AM · Unknown Object (Project), Restricted Project
Whitney added a project to D108112: [LoopIdiom] Let LIR fold memset pointer / stride SCEV regarding loop guards: Unknown Object (Project).
Dec 1 2021, 8:26 AM · Unknown Object (Project), Restricted Project

Nov 24 2021

Whitney added inline comments to D113939: [runtime-unroll] Remove restriction about unrolling multiple exit loops.
Nov 24 2021, 8:48 AM · Unknown Object (Project), Restricted Project

Nov 5 2021

Whitney committed rG93421108d255: Add NoOpLoopNestPass and LOOPNEST_PASS macro (authored by Whitney).
Add NoOpLoopNestPass and LOOPNEST_PASS macro
Nov 5 2021, 9:12 AM
Whitney closed D113185: Add NoOpLoopNestPass and LOOPNEST_PASS macro.
Nov 5 2021, 9:12 AM · Restricted Project

Nov 4 2021

Whitney updated the diff for D113185: Add NoOpLoopNestPass and LOOPNEST_PASS macro.

clang-format

Nov 4 2021, 8:23 AM · Restricted Project
Whitney requested review of D113185: Add NoOpLoopNestPass and LOOPNEST_PASS macro.
Nov 4 2021, 6:19 AM · Restricted Project
Whitney accepted D112980: [LoopInfo] Fix a bug in the function getInductionVariable.
Nov 4 2021, 6:11 AM · Restricted Project

Nov 2 2021

Whitney added a comment to D112980: [LoopInfo] Fix a bug in the function getInductionVariable.

Thanks for fixing this. Can you please provide a test case?

Nov 2 2021, 7:50 PM · Restricted Project

Oct 28 2021

Whitney accepted D112745: Point replay file to non-existent dummy.
Oct 28 2021, 1:08 PM · Restricted Project

Oct 26 2021

Whitney added reviewers for D112549: Adding patch and unittest to generalize ignorable induction casts in the LoopVectorizationLegality analysis.: bmahjour, Whitney.
Oct 26 2021, 9:39 AM · Restricted Project

Oct 1 2021

Whitney accepted D110955: [AIX] Don't pass namedsects in LTO mode.
Oct 1 2021, 12:00 PM · Restricted Project

Sep 28 2021

Whitney accepted D110671: [AIX] Enable PGO without LTO.
Sep 28 2021, 5:59 PM · Restricted Project

Sep 27 2021

Whitney accepted D110456: [CodeMoverUtils] Enhance isSafeToMoveBefore() when control flow equivalence is satisfied .

One minor comment, other than that LGTM.

Sep 27 2021, 12:50 PM · Restricted Project
Whitney added a comment to D110492: [LoopInfo] Return conditional branch of preheader's predecessor if loop's exit has exactly two predecessors which are latch and guard.

Why is the input to DSE not in simplified form? By running -loop-simplify after the IR you provided resolves this problem.

Sep 27 2021, 7:28 AM · Restricted Project
Whitney added inline comments to D110456: [CodeMoverUtils] Enhance isSafeToMoveBefore() when control flow equivalence is satisfied .
Sep 27 2021, 7:09 AM · Restricted Project

Sep 24 2021

Whitney accepted D110422: [AIX] Change the linkage of profiling counter/data to be private.
Sep 24 2021, 9:08 AM · Restricted Project, Restricted Project

Sep 23 2021

Whitney accepted D110378: [CodeMoverUtils] Enhance isSafeToMoveBefore() when moving BBs.
Sep 23 2021, 9:02 PM · Restricted Project
Whitney added inline comments to D110378: [CodeMoverUtils] Enhance isSafeToMoveBefore() when moving BBs.
Sep 23 2021, 7:46 PM · Restricted Project

Sep 1 2021

Whitney added a reviewer for D108087: [SimpleLoopUnswitch] Create SimpleLoopNestUnswitch pass: bmahjour.
Sep 1 2021, 6:52 AM · Unknown Object (Project), Restricted Project

Aug 24 2021

Whitney added inline comments to D108087: [SimpleLoopUnswitch] Create SimpleLoopNestUnswitch pass.
Aug 24 2021, 11:21 AM · Unknown Object (Project), Restricted Project
Whitney added a comment to D104620: [LoopVersioning] Add utility to fetch the runtime check basic block.

This change looks like part of the change in https://reviews.llvm.org/D104631?

Aug 24 2021, 11:15 AM · Restricted Project
Whitney accepted D104631: [LoopVersioning] Allow versionLoop to create plain branch inst when no runtime check is specified.

LGTM.

Aug 24 2021, 10:59 AM · Restricted Project

Aug 17 2021

Whitney committed rG4018d25da8ab: LoopNest Analysis expansion to return instructions that prevent a Loop (authored by madanial).
LoopNest Analysis expansion to return instructions that prevent a Loop
Aug 17 2021, 3:27 PM
Whitney closed D107773: LoopNest Analysis expansion to return instructions that prevent a Loop Nest from being perfect.
Aug 17 2021, 3:27 PM · Restricted Project

Aug 16 2021

Whitney requested review of D108166: [LNICM] Fix infinite loop.
Aug 16 2021, 3:10 PM · Restricted Project
Whitney added a reviewer for D108112: [LoopIdiom] Let LIR fold memset pointer / stride SCEV regarding loop guards: qianzhen.
Aug 16 2021, 7:06 AM · Unknown Object (Project), Restricted Project

Aug 12 2021

Whitney accepted D107353: [LoopIdiom] let the pass deal with runtime memset size.

LGTM

Aug 12 2021, 11:07 AM · Restricted Project

Aug 9 2021

Whitney accepted D107773: LoopNest Analysis expansion to return instructions that prevent a Loop Nest from being perfect.
Aug 9 2021, 4:09 PM · Restricted Project
Whitney added inline comments to D107773: LoopNest Analysis expansion to return instructions that prevent a Loop Nest from being perfect.
Aug 9 2021, 3:59 PM · Restricted Project
Whitney added inline comments to D107773: LoopNest Analysis expansion to return instructions that prevent a Loop Nest from being perfect.
Aug 9 2021, 11:17 AM · Restricted Project
Whitney added reviewers for D107773: LoopNest Analysis expansion to return instructions that prevent a Loop Nest from being perfect: bmahjour, etiotto, Meinersbur.
Aug 9 2021, 11:05 AM · Restricted Project
Whitney committed rG39ca3e5541d0: Update: clang/test/Profile/gcc-flag-compatibility.c to have -flto on AIX (authored by madanial).
Update: clang/test/Profile/gcc-flag-compatibility.c to have -flto on AIX
Aug 9 2021, 7:58 AM