Page MenuHomePhabricator

fpichet (Francois Pichet)
User

Projects

User does not belong to any projects.

User Details

User Since
Jul 4 2017, 6:15 AM (308 w, 3 d)

Recent Activity

Nov 15 2021

fpichet added a comment to D97735: [Globals] Treat nobuiltin fns as maybe-derefined..

I found this problem (PR49022) when running some regression test for an out of tree target.
So, what is the status of patch?
Any reason this hasn't been committed?

Nov 15 2021, 11:47 AM · Restricted Project, Restricted Project

Jan 12 2021

fpichet accepted D94529: [TableGen] Enhance !cast<string> to handle bit and bits types.

This works for me

Jan 12 2021, 10:21 AM · Restricted Project

Jan 11 2021

fpichet added a comment to D93969: [TableGen] Add field kind to the RecordVal class..

Thanks, adding the !cast<int> remove the problem. But it would be nice to be able to convert from bit to string automatically otherwise I would need to add the !cast<int> at dozen of places.

Jan 11 2021, 5:30 PM · Restricted Project
fpichet added a comment to D93969: [TableGen] Add field kind to the RecordVal class..

The identifier come from tablegen class parameter:

Jan 11 2021, 4:05 PM · Restricted Project
fpichet added a comment to D93969: [TableGen] Add field kind to the RecordVal class..

I had to add 'field' for something like: ( out of tree target)

field string BaseOpcodeSTSrcKind;

...

let BaseOpcodeSTSrcKind = asmstr # opVal # DataSize # PtrMode #

OffsetMode # isSub # isDS # "_ri7";

Jan 11 2021, 2:05 PM · Restricted Project
fpichet updated subscribers of D93969: [TableGen] Add field kind to the RecordVal class..

The llvm tablegen reference guide says the keyword *field* is deprecated.
Is this still valid?

Jan 11 2021, 1:06 PM · Restricted Project

Oct 23 2020

fpichet added a comment to D89236: [lldb] Fix bitfield "frame var" for pointers (pr47743).

What is the status of this patch?
Can it be committed?
It helps for our out of tree fork of lldb.

Oct 23 2020, 12:26 PM · Restricted Project

Sep 8 2020

fpichet added a comment to D75808: [ValueTracking] isKnownNonZero, computeKnownBits for freeze.

Another thing: I noticed that SelectionDAG::computeKnownBits doesn't deal with ISD::FREEZE.
Would it be ok to just call computeKnownBits (Op.getOperand(0)) to make it work.
(Since Select_FREEZE just lower to a copy)

Sep 8 2020, 6:52 AM · Restricted Project

Sep 7 2020

fpichet added a comment to D75808: [ValueTracking] isKnownNonZero, computeKnownBits for freeze.

What is the status of this patch?
I've seen cases in a out of tree target where an optimization dependent on ValueTracking was not performed because there was a freeze in the way.

Sep 7 2020, 3:50 PM · Restricted Project

May 29 2020

fpichet added a comment to D53005: Implement machine unroller utility class.

I am working on an out of tree target and this would be useful.
So what is the status of this patch?

May 29 2020, 1:06 PM

Jan 15 2019

fpichet accepted D50432: [DAGCombiner] Reduce load widths of shifted masks.

I tried your patch and the bug is gone. Thanks.

Jan 15 2019, 7:42 AM

Jan 14 2019

fpichet added a comment to D50432: [DAGCombiner] Reduce load widths of shifted masks.

For example:

Jan 14 2019, 8:04 AM

Jan 13 2019

fpichet reopened D50432: [DAGCombiner] Reduce load widths of shifted masks.

I recently resynced an out of tree backend and I got a miscompile because of this commit.
My target is Big Endian.
The C code is :

void swap(unsigned *ptr) {
  *ptr = (*ptr & 0x0000ff00 ) << 8;		
}
Jan 13 2019, 7:37 PM

Jul 25 2017

fpichet committed rL308960: Fix endianness bug in DAGCombiner::visitTRUNCATE and visitEXTRACT_VECTOR_ELT.
Fix endianness bug in DAGCombiner::visitTRUNCATE and visitEXTRACT_VECTOR_ELT
Jul 25 2017, 2:42 AM
fpichet closed D34990: Fix endianness bug in DAGCombiner::visitTRUNCATE and visitEXTRACT_VECTOR_ELT by committing rL308960: Fix endianness bug in DAGCombiner::visitTRUNCATE and visitEXTRACT_VECTOR_ELT.
Jul 25 2017, 2:42 AM

Jul 17 2017

fpichet added a comment to D34990: Fix endianness bug in DAGCombiner::visitTRUNCATE and visitEXTRACT_VECTOR_ELT.

is it ok to commit now?

Jul 17 2017, 11:34 AM

Jul 11 2017

fpichet updated the diff for D34990: Fix endianness bug in DAGCombiner::visitTRUNCATE and visitEXTRACT_VECTOR_ELT.

Update patch with fix for endianness bug in visitEXTRACT_VECTOR_ELT.

Jul 11 2017, 7:47 PM
fpichet added a comment to D34990: Fix endianness bug in DAGCombiner::visitTRUNCATE and visitEXTRACT_VECTOR_ELT.
Jul 11 2017, 1:30 PM
fpichet updated the summary of D34990: Fix endianness bug in DAGCombiner::visitTRUNCATE and visitEXTRACT_VECTOR_ELT.
Jul 11 2017, 1:29 PM
fpichet updated the diff for D34990: Fix endianness bug in DAGCombiner::visitTRUNCATE and visitEXTRACT_VECTOR_ELT.

Update patch taking into account vector size > 2
Also reuse existing isLE variable.

Jul 11 2017, 4:02 AM

Jul 5 2017

fpichet updated the diff for D34990: Fix endianness bug in DAGCombiner::visitTRUNCATE and visitEXTRACT_VECTOR_ELT.

Added test case

Jul 5 2017, 8:00 PM

Jul 4 2017

fpichet created D34990: Fix endianness bug in DAGCombiner::visitTRUNCATE and visitEXTRACT_VECTOR_ELT.
Jul 4 2017, 11:13 AM