Page MenuHomePhabricator

tauril (Guillaume Marques)
User

Projects

User does not belong to any projects.

User Details

User Since
Mar 29 2018, 6:00 AM (55 w, 14 h)

Recent Activity

Wed, Mar 27

tauril updated the diff for D56201: [LegalizeVectorTypes] Allow single loads and stores for more short vectors.

Hello @bogner, thanks for accepting this patch!

Wed, Mar 27, 10:03 AM · Restricted Project

Mar 12 2019

tauril added a comment to D56201: [LegalizeVectorTypes] Allow single loads and stores for more short vectors.

Ping.

Mar 12 2019, 6:26 AM · Restricted Project

Feb 25 2019

tauril added a comment to D56201: [LegalizeVectorTypes] Allow single loads and stores for more short vectors.

Ping.

Feb 25 2019, 4:16 AM · Restricted Project

Feb 15 2019

tauril added a comment to D56201: [LegalizeVectorTypes] Allow single loads and stores for more short vectors.

If no one else has any remark regarding this patch, could someone with the rights to approve the patch? @bogner

Feb 15 2019, 2:58 AM · Restricted Project

Feb 7 2019

tauril added a comment to D56201: [LegalizeVectorTypes] Allow single loads and stores for more short vectors.

Ping @bogner.

Feb 7 2019, 8:45 AM · Restricted Project

Jan 29 2019

tauril added a comment to D56201: [LegalizeVectorTypes] Allow single loads and stores for more short vectors.

I also want to point out I don't have commit access if the patch were to be accepted.

Jan 29 2019, 2:50 AM · Restricted Project

Jan 24 2019

tauril updated the diff for D56201: [LegalizeVectorTypes] Allow single loads and stores for more short vectors.

I added the nounwind attribute to the new test function's definition as @craig.topper suggested.

Jan 24 2019, 2:09 AM · Restricted Project

Jan 23 2019

tauril added a comment to D56201: [LegalizeVectorTypes] Allow single loads and stores for more short vectors.

Ping.

Jan 23 2019, 1:26 AM · Restricted Project

Jan 14 2019

tauril updated the diff for D56201: [LegalizeVectorTypes] Allow single loads and stores for more short vectors.

I renamed the new test from test/CodeGen/X86/load-local-i1.ll to test/CodeGen/X86/load-local-v3i1.ll
because I made a mistake the first time.
I also precommitted that same new test so that we can see the difference that this patch introduces with it,
as suggested by @lebedev.ri.
The relevant information to notice is the removal of the negb instruction followed by the filling of the
arguments of the masked_load_v3 function with the same value (as it is explained in the beginning of
the test).

Jan 14 2019, 10:57 AM · Restricted Project
tauril updated the diff for D56201: [LegalizeVectorTypes] Allow single loads and stores for more short vectors.

I modified some AMDGPU tests to track more registers where possible as @jvesely suggested, and I added some missing new relevant generated instructions (BFE_INT).

Jan 14 2019, 8:23 AM · Restricted Project

Jan 9 2019

tauril added inline comments to D56201: [LegalizeVectorTypes] Allow single loads and stores for more short vectors.
Jan 9 2019, 2:20 AM · Restricted Project

Jan 3 2019

tauril updated the summary of D56201: [LegalizeVectorTypes] Allow single loads and stores for more short vectors.
Jan 3 2019, 3:28 AM · Restricted Project

Jan 2 2019

tauril created D56201: [LegalizeVectorTypes] Allow single loads and stores for more short vectors.
Jan 2 2019, 8:17 AM · Restricted Project

Mar 29 2018

tauril added a comment to D33361: [InstCombine] Fix inbounds gep for addrspacecasts.

Hello, I've recently come across the same problem as expressed by this revision.
Has there been any new discussions regarding this topic?

Mar 29 2018, 8:00 AM