Page MenuHomePhabricator

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

dancgr (Danilo Carvalho Grael)
User

Projects

User does not belong to any projects.

User Details

User Since
Oct 17 2019, 7:31 AM (205 w, 5 d)

Recent Activity

Jul 27 2021

dancgr committed rG02077da7e7a8: Add jump-threading optimization for deterministic finite automata (authored by alexey.zhikhar).
Add jump-threading optimization for deterministic finite automata
Jul 27 2021, 11:36 AM
dancgr closed D99205: Add jump-threading optimization for deterministic finite automata.
Jul 27 2021, 11:36 AM · Restricted Project, Restricted Project

May 12 2021

dancgr added inline comments to D88595: [TableGen] Add not_all_same constraint check.
May 12 2021, 9:41 AM · Restricted Project, Restricted Project

May 7 2021

dancgr added a comment to D88595: [TableGen] Add not_all_same constraint check.

@paulwalker-arm @sdesmalen I will be picking up this work, I will study this patch and see if I have any questions. Thanks!

May 7 2021, 9:05 AM · Restricted Project, Restricted Project

Apr 15 2021

dancgr committed rG55487079a9bb: [LoopUnrollAndJam] Avoid repeated instructions for UAJ analysis (authored by dancgr).
[LoopUnrollAndJam] Avoid repeated instructions for UAJ analysis
Apr 15 2021, 10:05 AM
dancgr closed D97407: [LoopUnrollAndJam] Avoid repeated instructions for UAJ analysis.
Apr 15 2021, 10:05 AM · Restricted Project
dancgr updated the diff for D97407: [LoopUnrollAndJam] Avoid repeated instructions for UAJ analysis.

Updated last comments.

Apr 15 2021, 9:44 AM · Restricted Project

Apr 6 2021

dancgr added a comment to D97407: [LoopUnrollAndJam] Avoid repeated instructions for UAJ analysis.

@dmgreen I fixed the issues that you pointed out. This patch prevents the test case from getting stuck on the exponential explosion of the algorithm. Could you review it? Thanks!

Apr 6 2021, 11:40 AM · Restricted Project
dancgr added a reviewer for D97407: [LoopUnrollAndJam] Avoid repeated instructions for UAJ analysis: dmgreen.
Apr 6 2021, 11:39 AM · Restricted Project
dancgr updated the diff for D97407: [LoopUnrollAndJam] Avoid repeated instructions for UAJ analysis.

Fixed formatting and add test run line

Apr 6 2021, 11:39 AM · Restricted Project

Feb 25 2021

dancgr added a comment to D97407: [LoopUnrollAndJam] Avoid repeated instructions for UAJ analysis.

@dmgreen I think so. The code is indeed exponential. Adding this check will avoid the exponential case, but we will be visiting each instruction both ways, we just avoid visiting repeated ones. IMO it won't affect the analysis result.

Feb 25 2021, 9:51 AM · Restricted Project

Feb 24 2021

dancgr requested review of D97407: [LoopUnrollAndJam] Avoid repeated instructions for UAJ analysis.
Feb 24 2021, 11:54 AM · Restricted Project

Jan 26 2021

dancgr added a comment to D93478: [LoopVectorizer] Fix invalid scenario that is allowed to interleave.

@dmgreen @fhahn Sorry about that, the test case I posted was incorrect. I updated the test.c file, and in the process of finding the correct test case for you to be able to reproduce I found out the bug is worse than I expected.

Jan 26 2021, 2:11 PM · Restricted Project
dancgr updated the summary of D93478: [LoopVectorizer] Fix invalid scenario that is allowed to interleave.
Jan 26 2021, 12:46 PM · Restricted Project
dancgr updated the summary of D93478: [LoopVectorizer] Fix invalid scenario that is allowed to interleave.
Jan 26 2021, 12:45 PM · Restricted Project

Jan 20 2021

dancgr added a comment to D93478: [LoopVectorizer] Fix invalid scenario that is allowed to interleave.

Are there any opinions on this bugfix?

Jan 20 2021, 6:53 AM · Restricted Project

Jan 19 2021

dancgr added reviewers for D93478: [LoopVectorizer] Fix invalid scenario that is allowed to interleave: fhahn, dmgreen.
Jan 19 2021, 12:08 PM · Restricted Project

Dec 17 2020

dancgr added a reviewer for D93478: [LoopVectorizer] Fix invalid scenario that is allowed to interleave: mkuper.
Dec 17 2020, 11:42 AM · Restricted Project
dancgr requested review of D93478: [LoopVectorizer] Fix invalid scenario that is allowed to interleave.
Dec 17 2020, 11:32 AM · Restricted Project

Sep 30 2020

dancgr committed rG8d8cb1ad80b7: [AArch64] Avoid pairing loads when the base reg is modified (authored by congzhe).
[AArch64] Avoid pairing loads when the base reg is modified
Sep 30 2020, 10:08 AM
dancgr closed D86956: [AArch64] Avoid pairing loads when the base reg is modified.
Sep 30 2020, 10:08 AM · Restricted Project, Restricted Project, Restricted Project

Sep 22 2020

dancgr committed rG4edb3d3646c4: [AArch64] Avoid pairing loads with same result reg (authored by congzhe).
[AArch64] Avoid pairing loads with same result reg
Sep 22 2020, 1:27 PM
dancgr closed D86906: [AArch64] Avoid pairing loads with same result reg.
Sep 22 2020, 1:26 PM · Restricted Project, Restricted Project, Restricted Project

Sep 15 2020

dancgr committed rGd417488ef5a6: [AArch64][SVE] Add lowering for llvm fsqrt (authored by Asif).
[AArch64][SVE] Add lowering for llvm fsqrt
Sep 15 2020, 12:29 PM
dancgr closed D87707: [AArch64][SVE] Add lowering for llvm fsqrt.
Sep 15 2020, 12:29 PM · Restricted Project

Sep 4 2020

dancgr committed rG1ffcbe35ae0e: [AArch64][SVE] Add lowering for rounding operations (authored by Asif).
[AArch64][SVE] Add lowering for rounding operations
Sep 4 2020, 8:18 AM
dancgr closed D86793: [AArch64][SVE] Add lowering for rounding operations.
Sep 4 2020, 8:18 AM · Restricted Project

Sep 2 2020

dancgr committed rGec489ae048fd: [IPSCCP] Fix a bug that the "returned" attribute is not cleared when function… (authored by congzhe).
[IPSCCP] Fix a bug that the "returned" attribute is not cleared when function…
Sep 2 2020, 8:23 AM
dancgr closed D84220: [IPSCCP] Fix a bug that the "returned" attribute is not cleared when function is optimized to return undef.
Sep 2 2020, 8:23 AM · Restricted Project, Restricted Project

Aug 26 2020

dancgr committed rGfd536eeed99e: [AArch64][SVE] Add lowering for llvm fceil (authored by Asif).
[AArch64][SVE] Add lowering for llvm fceil
Aug 26 2020, 1:01 PM
dancgr closed D84548: [AArch64][SVE] Add lowering for llvm fceil.
Aug 26 2020, 1:01 PM · Restricted Project

Jul 13 2020

dancgr abandoned D83045: [AArch64][SVE] Add FP unpredicated to predicated two-op codegen.
Jul 13 2020, 11:04 AM · Restricted Project

Jul 2 2020

dancgr created D83045: [AArch64][SVE] Add FP unpredicated to predicated two-op codegen.
Jul 2 2020, 6:26 AM · Restricted Project

Jun 9 2020

dancgr committed rG4dcc0d1958f8: [CodeGen][SVE] Avoid scalarizing zero splat stores on scalable vectors. (authored by hkao13).
[CodeGen][SVE] Avoid scalarizing zero splat stores on scalable vectors.
Jun 9 2020, 10:26 AM
dancgr closed D80824: [CodeGen][SVE] Avoid scalarizing zero splat stores on scalable vectors..
Jun 9 2020, 10:26 AM · Restricted Project

Jun 3 2020

dancgr committed rGc57e41c000c4: [CodeGen][SVE] Replace deprecated calls in getCopyFromPartsVector() (authored by hkao13).
[CodeGen][SVE] Replace deprecated calls in getCopyFromPartsVector()
Jun 3 2020, 8:45 AM
dancgr closed D80826: [CodeGen][SVE] Replace deprecated calls in getCopyFromPartsVector().
Jun 3 2020, 8:45 AM · Restricted Project

May 29 2020

dancgr abandoned D80817: [CodeGen] Fix warning in SelectionDAG getNode for EXTRACT_VECTOR_ELT.

@efriedma Thanks Eli, I will close it, as that other patch already fixed that issue.

May 29 2020, 12:00 PM · Restricted Project
dancgr committed rGb637519eef01: [SVE] Replace deprecated call in changeVectorElementTypeToInteger (authored by bas).
[SVE] Replace deprecated call in changeVectorElementTypeToInteger
May 29 2020, 10:54 AM
dancgr closed D80746: [SVE] Replace deprecated call in changeVectorElementTypeToInteger.
May 29 2020, 10:54 AM · Restricted Project
dancgr created D80817: [CodeGen] Fix warning in SelectionDAG getNode for EXTRACT_VECTOR_ELT.
May 29 2020, 10:21 AM · Restricted Project
dancgr added a reviewer for D80817: [CodeGen] Fix warning in SelectionDAG getNode for EXTRACT_VECTOR_ELT: bas.
May 29 2020, 10:21 AM · Restricted Project

May 28 2020

dancgr added a comment to D80555: [CodeGen] Fix warning in changeVectorElementTypeToInteger.

Hello David, I have this patch here that I was going to upstream (https://reviews.llvm.org/D80746), but I just realized that it is exactly the same as this one.

May 28 2020, 2:52 PM · Restricted Project

Mar 5 2020

dancgr committed rGeae228a292f5: [IndVarSimplify] Extend previous special case for load use instruction to any… (authored by zhongduo).
[IndVarSimplify] Extend previous special case for load use instruction to any…
Mar 5 2020, 1:45 PM
dancgr closed D73059: [IndVarSimplify] Extend previous special case for load use instruction to any narrow type loop variant to avoid extra trunc instruction.
Mar 5 2020, 1:45 PM · Restricted Project

Feb 27 2020

dancgr added a comment to D75197: [AArch64][SVE] Add intrinsics for bitwise permute instructions.

Thanks for the patch @dancgr but these intrinsics have already been implemented in D74912. @sdesmalen shared a spreadsheet [1] with the mailing list to coordinate the intrinsics upstreaming, it's probably not as useful now given they're almost all implemented but you might find it useful if you haven't seen it already.

[1] https://docs.google.com/spreadsheets/d/1leug3a0eHhi_bWtKJOR0MeZHvfQLoIPtP-um7AWafRM

Feb 27 2020, 7:05 AM · Restricted Project
dancgr abandoned D75197: [AArch64][SVE] Add intrinsics for bitwise permute instructions.
Feb 27 2020, 7:03 AM · Restricted Project

Feb 26 2020

dancgr added inline comments to D75197: [AArch64][SVE] Add intrinsics for bitwise permute instructions.
Feb 26 2020, 10:34 AM · Restricted Project
dancgr created D75197: [AArch64][SVE] Add intrinsics for bitwise permute instructions.
Feb 26 2020, 10:34 AM · Restricted Project

Feb 21 2020

dancgr added a comment to D74785: [AArch64][SVE] Add intrinsics for SVE2 bitwise ternary operations.

Latest commit adds a change to the failing MC tests to be exactly the same as the MC tests for eor3 and bcax.

Feb 21 2020, 9:28 AM · Restricted Project
dancgr committed rGdb9c40f5624e: [AArch64][SVE] Add intrinsics for SVE2 bitwise ternary operations (authored by dancgr).
[AArch64][SVE] Add intrinsics for SVE2 bitwise ternary operations
Feb 21 2020, 9:19 AM
dancgr closed D74785: [AArch64][SVE] Add intrinsics for SVE2 bitwise ternary operations.
Feb 21 2020, 9:19 AM · Restricted Project
dancgr reopened D74785: [AArch64][SVE] Add intrinsics for SVE2 bitwise ternary operations.

Will look into that and update this review with the necessary changes.

Feb 21 2020, 8:33 AM · Restricted Project

Feb 20 2020

dancgr committed rGce70e2899879: [AArch64][SVE] Add intrinsics for SVE2 bitwise ternary operations (authored by dancgr).
[AArch64][SVE] Add intrinsics for SVE2 bitwise ternary operations
Feb 20 2020, 8:30 AM
dancgr closed D74785: [AArch64][SVE] Add intrinsics for SVE2 bitwise ternary operations.
Feb 20 2020, 8:30 AM · Restricted Project

Feb 19 2020

dancgr updated the diff for D74785: [AArch64][SVE] Add intrinsics for SVE2 bitwise ternary operations.

Merging all instructions into same multiclass.

Feb 19 2020, 10:37 AM · Restricted Project

Feb 18 2020

dancgr added inline comments to D74785: [AArch64][SVE] Add intrinsics for SVE2 bitwise ternary operations.
Feb 18 2020, 3:15 PM · Restricted Project
dancgr added inline comments to D74785: [AArch64][SVE] Add intrinsics for SVE2 bitwise ternary operations.
Feb 18 2020, 1:49 PM · Restricted Project
dancgr created D74785: [AArch64][SVE] Add intrinsics for SVE2 bitwise ternary operations.
Feb 18 2020, 12:08 PM · Restricted Project

Feb 12 2020

dancgr committed rGfc8d033e9663: [AArch64][SVE] Add addsub carry long instrinsics (authored by dancgr).
[AArch64][SVE] Add addsub carry long instrinsics
Feb 12 2020, 7:46 AM
dancgr closed D74328: [AArch64][SVE] Add addsub carry long instrinsics.
Feb 12 2020, 7:46 AM · Restricted Project

Feb 10 2020

dancgr created D74328: [AArch64][SVE] Add addsub carry long instrinsics.
Feb 10 2020, 8:16 AM · Restricted Project

Jan 31 2020

dancgr committed rG44a4f5fc6a5a: [AArch64][SVE] Add SVE2 mla unpredicated intrinsics. (authored by dancgr).
[AArch64][SVE] Add SVE2 mla unpredicated intrinsics.
Jan 31 2020, 8:48 AM
dancgr closed D73746: [AArch64][SVE] Add SVE2 mla unpredicated intrinsics..
Jan 31 2020, 8:47 AM · Restricted Project
dancgr added a comment to D73746: [AArch64][SVE] Add SVE2 mla unpredicated intrinsics..

Thanks for these changes, @dancgr! I just have a couple of comments on the tests in sve2-mla-unpredicated.ll

Jan 31 2020, 7:50 AM · Restricted Project
dancgr updated the diff for D73746: [AArch64][SVE] Add SVE2 mla unpredicated intrinsics..

Add missing i16 tests.

Jan 31 2020, 7:40 AM · Restricted Project

Jan 30 2020

dancgr created D73746: [AArch64][SVE] Add SVE2 mla unpredicated intrinsics..
Jan 30 2020, 2:20 PM · Restricted Project
dancgr committed rG0610637aac9c: [AArch64][SVE] Add remaining SVE2 mla indexed intrinsics. (authored by dancgr).
[AArch64][SVE] Add remaining SVE2 mla indexed intrinsics.
Jan 30 2020, 10:31 AM
dancgr closed D73633: [AArch64][SVE] Add remaining SVE2 mla indexed intrinsics..
Jan 30 2020, 10:30 AM · Restricted Project

Jan 29 2020

dancgr updated the summary of D73633: [AArch64][SVE] Add remaining SVE2 mla indexed intrinsics..
Jan 29 2020, 1:40 PM · Restricted Project
dancgr added a comment to D73633: [AArch64][SVE] Add remaining SVE2 mla indexed intrinsics..

@sdesmalen Hopefully all comments from D73576 were addressed with this patch.

Jan 29 2020, 8:35 AM · Restricted Project
dancgr created D73633: [AArch64][SVE] Add remaining SVE2 mla indexed intrinsics..
Jan 29 2020, 8:25 AM · Restricted Project
dancgr added a comment to D73576: [AArch64][SVE] Add SVE2 mla indexed intrinsics.

I will make the changes suggested by Sander in a following patch, joined with the saturating multiply-add long intirnsics.

Jan 29 2020, 7:21 AM · Restricted Project

Jan 28 2020

dancgr committed rG1f85dfb2af1a: [AArch64][SVE] Add SVE2 mla indexed intrinsics. (authored by dancgr).
[AArch64][SVE] Add SVE2 mla indexed intrinsics.
Jan 28 2020, 2:16 PM
dancgr closed D73576: [AArch64][SVE] Add SVE2 mla indexed intrinsics.
Jan 28 2020, 2:15 PM · Restricted Project
dancgr retitled D73576: [AArch64][SVE] Add SVE2 mla indexed intrinsics from [AArch64][SVE] Add SVE2 mla indexed intrinsics. to [AArch64][SVE] Add SVE2 mla indexed intrinsics.
Jan 28 2020, 1:22 PM · Restricted Project
dancgr added a reviewer for D73576: [AArch64][SVE] Add SVE2 mla indexed intrinsics: kmclaughlin.
Jan 28 2020, 1:11 PM · Restricted Project
dancgr removed a reviewer for D73576: [AArch64][SVE] Add SVE2 mla indexed intrinsics: dancgr.
Jan 28 2020, 1:11 PM · Restricted Project
dancgr created D73576: [AArch64][SVE] Add SVE2 mla indexed intrinsics.
Jan 28 2020, 1:11 PM · Restricted Project

Jan 23 2020

dancgr committed rG58ceb81d318b: [SVE] Add SVE2 patterns for unpredicated multiply instructions (authored by dancgr).
[SVE] Add SVE2 patterns for unpredicated multiply instructions
Jan 23 2020, 10:28 AM
dancgr closed D72799: [SVE] Add SVE2 patterns for unpredicated multiply instructions.
Jan 23 2020, 10:28 AM · Restricted Project

Jan 22 2020

dancgr added inline comments to D72799: [SVE] Add SVE2 patterns for unpredicated multiply instructions.
Jan 22 2020, 10:42 AM · Restricted Project
dancgr updated the diff for D72799: [SVE] Add SVE2 patterns for unpredicated multiply instructions.

Remove _m from intrinsics, as it is the default behaviour. Also, add unpredicated patterns for SVE2 smulh, umulh.

Jan 22 2020, 10:42 AM · Restricted Project

Jan 20 2020

dancgr added a comment to D72799: [SVE] Add SVE2 patterns for unpredicated multiply instructions.

I'm not sure on some parts, but I have prepared a major update for this patch that I hope will fix most of @sdesmalen concerns.

Jan 20 2020, 8:35 AM · Restricted Project

Jan 16 2020

dancgr updated the diff for D72799: [SVE] Add SVE2 patterns for unpredicated multiply instructions.

Tentative changes to address some comments from the reviewers.

Jan 16 2020, 11:49 AM · Restricted Project
dancgr added a comment to D72799: [SVE] Add SVE2 patterns for unpredicated multiply instructions.

I'm not sure on some parts, but I have prepared a major update for this patch that I hope will fix most of @sdesmalen concerns.

Jan 16 2020, 11:39 AM · Restricted Project

Jan 15 2020

dancgr added a comment to D72799: [SVE] Add SVE2 patterns for unpredicated multiply instructions.

Can we also also add mul patterns for targets that have SVE, but not SVE2?

That instruction is restricted to SVE2. Do we have unpredicated vector mul instructions for SVE targets as well?

We can synthesize a predicate using ptrue. We do this in a few places currently; for example, to implement bswap.

Jan 15 2020, 2:06 PM · Restricted Project
dancgr committed rG34ba96a3d49e: [NFC][IndVarSimplify] remove duplicate code in widenWithVariantLoadUseCodegen. (authored by zhongduo).
[NFC][IndVarSimplify] remove duplicate code in widenWithVariantLoadUseCodegen.
Jan 15 2020, 1:29 PM
dancgr closed D72652: [NFC][IndVarSimplify] remove duplicate code in widenWithVariantLoadUseCodegen.
Jan 15 2020, 1:29 PM · Restricted Project
dancgr added a comment to D72799: [SVE] Add SVE2 patterns for unpredicated multiply instructions.

Can we also also add mul patterns for targets that have SVE, but not SVE2?

Jan 15 2020, 12:32 PM · Restricted Project
dancgr updated the summary of D72799: [SVE] Add SVE2 patterns for unpredicated multiply instructions.
Jan 15 2020, 12:15 PM · Restricted Project
dancgr created D72799: [SVE] Add SVE2 patterns for unpredicated multiply instructions.
Jan 15 2020, 12:14 PM · Restricted Project

Jan 14 2020

dancgr committed rG26d96126a0d2: [SVE] Add patterns for MUL immediate instruction. (authored by dancgr).
[SVE] Add patterns for MUL immediate instruction.
Jan 14 2020, 12:32 PM
dancgr closed D72654: [SVE] Add patterns for MUL immediate instruction..
Jan 14 2020, 12:30 PM · Restricted Project
dancgr added inline comments to D72654: [SVE] Add patterns for MUL immediate instruction..
Jan 14 2020, 12:02 PM · Restricted Project
dancgr updated the diff for D72654: [SVE] Add patterns for MUL immediate instruction..

Fix whitespace errors and add a couple negative tests for invalid input.

Jan 14 2020, 11:12 AM · Restricted Project
dancgr added a comment to D72654: [SVE] Add patterns for MUL immediate instruction..

Do we have any negative tests, for immediates that are outside the range of the sve mul immediate instruction?

Jan 14 2020, 10:44 AM · Restricted Project

Jan 13 2020

dancgr created D72654: [SVE] Add patterns for MUL immediate instruction..
Jan 13 2020, 2:48 PM · Restricted Project
dancgr committed rG2d7e757a836a: [AArch64][SVE] Add patterns for some arith SVE instructions. (authored by dancgr).
[AArch64][SVE] Add patterns for some arith SVE instructions.
Jan 13 2020, 8:41 AM
dancgr closed D71779: [AArch64][SVE] Add patterns for signed and unsigned min/max instructions.
Jan 13 2020, 8:41 AM · Restricted Project