Page MenuHomePhabricator

jonpa (Jonas Paulsson)
User

Projects

User does not belong to any projects.

User Details

User Since
Feb 24 2015, 1:18 AM (295 w, 42 m)

Recent Activity

Sat, Oct 17

jonpa added a comment to D89447: [MachineInstr] Add support for instructions with multiple memory operands..

I removed that check that one of those two MMOs needs to be store. No change to test now. But, that sounds a little weird that one instruction claims mayLoad and mayStore but only has isLoad MMO. Maybe we need to change that MMO in prefetch to be both isLoad and isStore.

Sat, Oct 17, 4:49 AM · Restricted Project
jonpa updated the diff for D87279: [clang] Fix handling of physical registers in inline assembly operands..

Sorry, here's the test case included as well...

Sat, Oct 17, 3:51 AM · Restricted Project
jonpa added inline comments to D89451: [SystemZ] Mark unsaved argument R6 as live throughout function.
Sat, Oct 17, 3:07 AM · Restricted Project

Fri, Oct 16

jonpa updated the diff for D87279: [clang] Fix handling of physical registers in inline assembly operands..

That makes sense... Patch updated to keep the tying of operands for this case of earlyclobber.

Fri, Oct 16, 6:52 AM · Restricted Project
jonpa added a comment to D87279: [clang] Fix handling of physical registers in inline assembly operands..

The problem seems to be with a tied earlyclobber operand:

Fri, Oct 16, 5:10 AM · Restricted Project
jonpa added a comment to D89447: [MachineInstr] Add support for instructions with multiple memory operands..

@jonpa can you check whether the SystemZ test case you added still checks what it was intended to check here?

Fri, Oct 16, 4:24 AM · Restricted Project
jonpa updated the diff for D89451: [SystemZ] Mark unsaved argument R6 as live throughout function.

Updated per review to use a MIR test instead.

Fri, Oct 16, 3:31 AM · Restricted Project

Thu, Oct 15

jonpa requested review of D89451: [SystemZ] Mark unsaved argument R6 as live throughout function.
Thu, Oct 15, 1:11 AM · Restricted Project

Wed, Oct 14

jonpa committed rG6756d43af97e: [SystemZ] Bugfix in SystemZVectorConstantInfo (authored by jonpa).
[SystemZ] Bugfix in SystemZVectorConstantInfo
Wed, Oct 14, 6:37 AM
jonpa closed D89389: [SystemZ] Bugfix in SystemZVectorConstantInfo.
Wed, Oct 14, 6:37 AM · Restricted Project
jonpa requested review of D89389: [SystemZ] Bugfix in SystemZVectorConstantInfo.
Wed, Oct 14, 5:49 AM · Restricted Project
jonpa requested review of D87279: [clang] Fix handling of physical registers in inline assembly operands..
Wed, Oct 14, 12:04 AM · Restricted Project

Tue, Oct 13

jonpa reopened D87279: [clang] Fix handling of physical registers in inline assembly operands..

Patch temporarily reverted.

Tue, Oct 13, 11:46 PM · Restricted Project
jonpa added a reverting change for D87279: [clang] Fix handling of physical registers in inline assembly operands.: rG625fa4761702: Revert "[clang] Improve handling of physical registers in inline assembly….
Tue, Oct 13, 11:44 PM · Restricted Project
jonpa added a reverting change for rGc78da037783b: [clang] Improve handling of physical registers in inline assembly operands.: rG625fa4761702: Revert "[clang] Improve handling of physical registers in inline assembly….
Tue, Oct 13, 11:44 PM
jonpa committed rG625fa4761702: Revert "[clang] Improve handling of physical registers in inline assembly… (authored by jonpa).
Revert "[clang] Improve handling of physical registers in inline assembly…
Tue, Oct 13, 11:44 PM
jonpa committed rGc78da037783b: [clang] Improve handling of physical registers in inline assembly operands. (authored by jonpa).
[clang] Improve handling of physical registers in inline assembly operands.
Tue, Oct 13, 6:11 AM
jonpa closed D87279: [clang] Fix handling of physical registers in inline assembly operands..
Tue, Oct 13, 6:11 AM · Restricted Project
jonpa updated the diff for D86519: [SystemZ] New pass for domain reassignment from integer to vector..

Latest improvements (still in progress):

Tue, Oct 13, 4:10 AM · Restricted Project

Fri, Oct 9

jonpa committed rGd851495f2fe6: [SystemZ] Use LA instead of AGR in eliminateFrameIndex(). (authored by jonpa).
[SystemZ] Use LA instead of AGR in eliminateFrameIndex().
Fri, Oct 9, 4:07 AM
jonpa closed D89034: [SystemZ] Use LA instead of AGR in eliminateFrameIndex()..
Fri, Oct 9, 4:07 AM · Restricted Project
jonpa updated the diff for D89034: [SystemZ] Use LA instead of AGR in eliminateFrameIndex()..

Updated per review.

Fri, Oct 9, 12:59 AM · Restricted Project
jonpa updated the diff for D87279: [clang] Fix handling of physical registers in inline assembly operands..

Updated message string per review.

Fri, Oct 9, 12:44 AM · Restricted Project

Thu, Oct 8

jonpa requested review of D89034: [SystemZ] Use LA instead of AGR in eliminateFrameIndex()..
Thu, Oct 8, 4:24 AM · Restricted Project
jonpa updated the diff for D87279: [clang] Fix handling of physical registers in inline assembly operands..

Updated per review to use SmallSet.

Thu, Oct 8, 2:27 AM · Restricted Project

Wed, Oct 7

jonpa added inline comments to D87279: [clang] Fix handling of physical registers in inline assembly operands..
Wed, Oct 7, 1:43 AM · Restricted Project
jonpa updated the diff for D87279: [clang] Fix handling of physical registers in inline assembly operands..

Updated per review by eliminating the call to 'count'.

Wed, Oct 7, 1:43 AM · Restricted Project

Tue, Oct 6

jonpa committed rG5588dbce73be: [SystemZAsmParser] Treat VR128 separately in ParseDirectiveInsn(). (authored by jonpa).
[SystemZAsmParser] Treat VR128 separately in ParseDirectiveInsn().
Tue, Oct 6, 5:47 AM
jonpa closed D88888: [SystemZAsmParser] Treat VR128 separately in ParseDirectiveInsn()..
Tue, Oct 6, 5:47 AM · Restricted Project
jonpa added reviewers for D87279: [clang] Fix handling of physical registers in inline assembly operands.: RKSimon, jyu2, t.p.northover, echristo, dylanmckay, aaron.ballman.

Last chance for anyone to speak up about any objections to committing this...

Tue, Oct 6, 5:38 AM · Restricted Project
jonpa requested review of D88888: [SystemZAsmParser] Treat VR128 separately in ParseDirectiveInsn()..
Tue, Oct 6, 4:29 AM · Restricted Project

Mon, Oct 5

jonpa committed rG34b61d6cd58a: [SystemZ] Add support for .insn directives for vector instructions. (authored by jonpa).
[SystemZ] Add support for .insn directives for vector instructions.
Mon, Oct 5, 7:54 AM
jonpa closed D88357: [SystemZ] Add support for .insn directives for vector instructions.
Mon, Oct 5, 7:53 AM · Restricted Project
jonpa added inline comments to D88357: [SystemZ] Add support for .insn directives for vector instructions.
Mon, Oct 5, 3:50 AM · Restricted Project
jonpa updated the diff for D88357: [SystemZ] Add support for .insn directives for vector instructions.

Updated per review.

Mon, Oct 5, 3:50 AM · Restricted Project

Wed, Sep 30

jonpa committed rG9f5da55f5d92: [SystemZ] Support bare nop instructions (authored by jonpa).
[SystemZ] Support bare nop instructions
Wed, Sep 30, 2:25 AM

Tue, Sep 29

jonpa committed rG75a5febe31cb: [SystemZ] Don't emit PC-relative memory accesses to unaligned symbols. (authored by jonpa).
[SystemZ] Don't emit PC-relative memory accesses to unaligned symbols.
Tue, Sep 29, 5:54 AM
jonpa closed D87510: [SystemZ] Don't emit PC-relative memory accesses to unaligned (packed) symbols..
Tue, Sep 29, 5:53 AM · Restricted Project
jonpa added inline comments to D87510: [SystemZ] Don't emit PC-relative memory accesses to unaligned (packed) symbols..
Tue, Sep 29, 2:57 AM · Restricted Project
jonpa updated the diff for D87510: [SystemZ] Don't emit PC-relative memory accesses to unaligned (packed) symbols..

Updated per review.

Tue, Sep 29, 2:57 AM · Restricted Project

Mon, Sep 28

jonpa added a comment to D87279: [clang] Fix handling of physical registers in inline assembly operands..

ping!

Mon, Sep 28, 5:58 AM · Restricted Project

Sat, Sep 26

jonpa requested review of D88357: [SystemZ] Add support for .insn directives for vector instructions.
Sat, Sep 26, 6:40 AM · Restricted Project

Fri, Sep 25

jonpa abandoned D87506: [SystemZ] Make LibFunc_bcmp unavailable.

Replaced by https://reviews.llvm.org/D87988

Fri, Sep 25, 9:25 AM · Restricted Project

Thu, Sep 24

jonpa added a comment to D87988: [SystemZ] Optimize bcmp calls (PR47420).

This looks ok to me, but let's wait for @uweigand as well.

Thu, Sep 24, 7:35 AM · Restricted Project

Wed, Sep 23

jonpa closed D88082: [SystemZ] Make sure not to call getZExtValue on a >64 bit constant..

Committed as 370a8c8.

Wed, Sep 23, 6:39 AM · Restricted Project
jonpa committed rG370a8c802558: [SystemZ] Make sure not to call getZExtValue on a >64 bit constant. (authored by jonpa).
[SystemZ] Make sure not to call getZExtValue on a >64 bit constant.
Wed, Sep 23, 6:37 AM
jonpa added inline comments to D88082: [SystemZ] Make sure not to call getZExtValue on a >64 bit constant..
Wed, Sep 23, 6:15 AM · Restricted Project
jonpa added reviewers for D87279: [clang] Fix handling of physical registers in inline assembly operands.: glider, void.
Wed, Sep 23, 3:26 AM · Restricted Project
jonpa removed a reviewer for D87279: [clang] Fix handling of physical registers in inline assembly operands.: void.
Wed, Sep 23, 3:25 AM · Restricted Project
jonpa added a reviewer for D87279: [clang] Fix handling of physical registers in inline assembly operands.: void.
Wed, Sep 23, 3:24 AM · Restricted Project
jonpa updated the diff for D88082: [SystemZ] Make sure not to call getZExtValue on a >64 bit constant..

Thanks for review!

Wed, Sep 23, 3:19 AM · Restricted Project
jonpa added a comment to D87988: [SystemZ] Optimize bcmp calls (PR47420).

Right, I will adjust to make it more clear.

Wed, Sep 23, 1:16 AM · Restricted Project

Tue, Sep 22

jonpa added a comment to D87988: [SystemZ] Optimize bcmp calls (PR47420).

Maybe also have the test case from my original patch which transforms a memcmp to a bcmp...?

Tue, Sep 22, 7:19 AM · Restricted Project
jonpa added inline comments to D87510: [SystemZ] Don't emit PC-relative memory accesses to unaligned (packed) symbols..
Tue, Sep 22, 7:02 AM · Restricted Project
jonpa updated the diff for D87510: [SystemZ] Don't emit PC-relative memory accesses to unaligned (packed) symbols..

Updated per review (see inline comments).

Tue, Sep 22, 7:02 AM · Restricted Project
jonpa added a comment to D87279: [clang] Fix handling of physical registers in inline assembly operands..

ping!

Tue, Sep 22, 3:22 AM · Restricted Project
jonpa requested review of D88082: [SystemZ] Make sure not to call getZExtValue on a >64 bit constant..
Tue, Sep 22, 3:19 AM · Restricted Project

Sep 11 2020

jonpa requested review of D87510: [SystemZ] Don't emit PC-relative memory accesses to unaligned (packed) symbols..
Sep 11 2020, 6:25 AM · Restricted Project
jonpa requested review of D87506: [SystemZ] Make LibFunc_bcmp unavailable.
Sep 11 2020, 4:28 AM · Restricted Project

Sep 8 2020

jonpa committed rG6dc3e22b5752: [DAGTypeLegalizer] Handle ZERO_EXTEND of promoted type in WidenVecRes_Convert. (authored by jonpa).
[DAGTypeLegalizer] Handle ZERO_EXTEND of promoted type in WidenVecRes_Convert.
Sep 8 2020, 7:55 AM
jonpa closed D86268: [DAGTypeLegalizer] Handle ZERO_EXTEND of promoted integer in WidenVecRes_Convert..
Sep 8 2020, 7:55 AM · Restricted Project
jonpa requested review of D87279: [clang] Fix handling of physical registers in inline assembly operands..
Sep 8 2020, 4:56 AM · Restricted Project

Sep 7 2020

jonpa abandoned D87130: [SelectionDAGBuilder] Remember to copy the FMF flags in visitBinary()..

Not needed anymore thanks to D87200.

Sep 7 2020, 1:30 AM · Restricted Project

Sep 5 2020

jonpa committed rG714ceefad9b9: [SelectionDAG] Always intersect SDNode flags during getNode() node memoization. (authored by jonpa).
[SelectionDAG] Always intersect SDNode flags during getNode() node memoization.
Sep 5 2020, 1:32 AM
jonpa closed D86871: [SelectionDAG] Let NSW/NUW flags be cleared by default in call to getNode()..
Sep 5 2020, 1:32 AM · Restricted Project

Sep 4 2020

jonpa added a comment to D87130: [SelectionDAGBuilder] Remember to copy the FMF flags in visitBinary()..

I think this makes sense to me, but I want to point out that this partially reverts the commit https://reviews.llvm.org/D46854. I guess there may be other parts of this commit that should be reverted?

Sep 4 2020, 7:31 AM · Restricted Project
jonpa updated the diff for D86268: [DAGTypeLegalizer] Handle ZERO_EXTEND of promoted integer in WidenVecRes_Convert..

But we do not know anything about the relationship between C and B necessarily.

Added a check for this.

Sep 4 2020, 6:56 AM · Restricted Project
jonpa added a comment to D86871: [SelectionDAG] Let NSW/NUW flags be cleared by default in call to getNode()..

The problem occurs when setFlags is done after memoization was already performed.

Ah, now I see what you mean :-)

Sep 4 2020, 3:32 AM · Restricted Project
jonpa removed reviewers for D87132: [Partial Inliner] Compute intrinsic cost through TTI: jnspaulsson, jonpa.
Sep 4 2020, 2:54 AM · Restricted Project, Restricted Project
jonpa requested review of D87130: [SelectionDAGBuilder] Remember to copy the FMF flags in visitBinary()..
Sep 4 2020, 1:45 AM · Restricted Project
jonpa updated the diff for D86871: [SelectionDAG] Let NSW/NUW flags be cleared by default in call to getNode()..

We have to favor correctness over performance, so that's the right change. But if you already have some idea/example of how we are losing the flags on those SPEC tests, it would be great to add reduced versions as regression tests.

Sep 4 2020, 1:10 AM · Restricted Project

Sep 3 2020

jonpa updated the diff for D86871: [SelectionDAG] Let NSW/NUW flags be cleared by default in call to getNode()..

Patch updated per review.

Sep 3 2020, 8:57 AM · Restricted Project

Sep 1 2020

jonpa added reviewers for D86871: [SelectionDAG] Let NSW/NUW flags be cleared by default in call to getNode().: craig.topper, eli.friedman.
Sep 1 2020, 7:22 AM · Restricted Project

Aug 31 2020

jonpa requested review of D86871: [SelectionDAG] Let NSW/NUW flags be cleared by default in call to getNode()..
Aug 31 2020, 4:35 AM · Restricted Project

Aug 30 2020

jonpa updated the diff for D86519: [SystemZ] New pass for domain reassignment from integer to vector..

Latest improvements and fixes.

Aug 30 2020, 1:45 AM · Restricted Project

Aug 25 2020

jonpa added a comment to D86268: [DAGTypeLegalizer] Handle ZERO_EXTEND of promoted integer in WidenVecRes_Convert..

ping!

Aug 25 2020, 7:11 AM · Restricted Project
jonpa added a comment to D86519: [SystemZ] New pass for domain reassignment from integer to vector..

Note: This patch still has a way to go before being ready to commit...

Aug 25 2020, 3:51 AM · Restricted Project
jonpa requested review of D86519: [SystemZ] New pass for domain reassignment from integer to vector..
Aug 25 2020, 3:50 AM · Restricted Project

Aug 24 2020

jonpa committed rG8ac70694b9f9: [SystemZ] Preserve the MachineMemOperand in emitCondStore() in all cases. (authored by jonpa).
[SystemZ] Preserve the MachineMemOperand in emitCondStore() in all cases.
Aug 24 2020, 5:09 AM

Aug 20 2020

jonpa added inline comments to D86268: [DAGTypeLegalizer] Handle ZERO_EXTEND of promoted integer in WidenVecRes_Convert..
Aug 20 2020, 1:19 AM · Restricted Project
jonpa requested review of D86268: [DAGTypeLegalizer] Handle ZERO_EXTEND of promoted integer in WidenVecRes_Convert..
Aug 20 2020, 1:12 AM · Restricted Project

Jun 30 2020

jonpa committed rGef7aad0db49f: [SystemZ] Improve handling of ZERO_EXTEND_VECTOR_INREG. (authored by jonpa).
[SystemZ] Improve handling of ZERO_EXTEND_VECTOR_INREG.
Jun 30 2020, 12:30 AM
jonpa closed D78486: [SystemZ] Expand vector zero-extend into a shuffle..
Jun 30 2020, 12:30 AM · Restricted Project

Jun 29 2020

jonpa abandoned D76275: [SystemZ] Use vllez + vle to load, shuffle and zero extend a vector..
Jun 29 2020, 11:57 PM · Restricted Project
jonpa abandoned D78488: [SystemZ] Emit VLLEZ from tryShuffleWithZeroVec().
Jun 29 2020, 11:57 PM · Restricted Project

Jun 17 2020

jonpa committed rGd3f7448e3ca5: [SystemZ] Bugfix in storeLoadCanUseBlockBinary(). (authored by jonpa).
[SystemZ] Bugfix in storeLoadCanUseBlockBinary().
Jun 17 2020, 1:03 AM
jonpa closed D81671: [SystemZ] Bugfix in storeLoadCanUseBlockBinary().
Jun 17 2020, 1:02 AM · Restricted Project

Jun 16 2020

jonpa added a comment to D81671: [SystemZ] Bugfix in storeLoadCanUseBlockBinary().

Shouldn't the check be done inside canUseBlockOperation? I believe the other user (storeLoadCanUseMVC) really needs the same check.

Jun 16 2020, 12:31 AM · Restricted Project

Jun 11 2020

jonpa created D81671: [SystemZ] Bugfix in storeLoadCanUseBlockBinary().
Jun 11 2020, 10:27 AM · Restricted Project

Jun 6 2020

jonpa committed rG515bfc66eace: [SystemZ] Implement -fstack-clash-protection (authored by jonpa).
[SystemZ] Implement -fstack-clash-protection
Jun 6 2020, 10:08 AM
jonpa closed D78717: [SystemZ] Implement -fstack-clash-protection.
Jun 6 2020, 10:08 AM · Restricted Project, Restricted Project
jonpa added a comment to D78717: [SystemZ] Implement -fstack-clash-protection.

Thanks for review - committed as 515bfc6 after last updates.

Jun 6 2020, 10:08 AM · Restricted Project, Restricted Project

Jun 4 2020

jonpa updated the diff for D78717: [SystemZ] Implement -fstack-clash-protection.
  • Patch rebased (using cfiDefCfaOffset() instead of createDefCfaOffset()).
Jun 4 2020, 4:50 AM · Restricted Project, Restricted Project

May 27 2020

jonpa added inline comments to D78717: [SystemZ] Implement -fstack-clash-protection.
May 27 2020, 1:02 AM · Restricted Project, Restricted Project

May 22 2020

jonpa added a comment to D78717: [SystemZ] Implement -fstack-clash-protection.

That sounds very nice. If you go that way I'll happily update the X86 code accordingly. Or If you point me to some example / doc / code on pseudo MachineInstruction, I can implement that too.

May 22 2020, 1:49 AM · Restricted Project, Restricted Project

May 21 2020

jonpa added inline comments to D78486: [SystemZ] Expand vector zero-extend into a shuffle..
May 21 2020, 2:08 AM · Restricted Project
jonpa updated the diff for D78488: [SystemZ] Emit VLLEZ from tryShuffleWithZeroVec().

Patch rebased on again on top of https://reviews.llvm.org/D78486, and also reworked a bit according to latest changes to that patch.

May 21 2020, 1:36 AM · Restricted Project

May 20 2020

jonpa added a comment to D78486: [SystemZ] Expand vector zero-extend into a shuffle..

About the LCPI vector constants: It seems that those 200 extra constants I noticed were not really due to the optimization of eliminating the zero-vector for PERMUTE. That optimization eliminates 239 VGBMs on SPEC, while adding only 5 more LCPIs. All the other new LCPIs caused by this patch seem to be regular VPERM masks for zero extension that are duplicated once for each function. In other words, I see the same mask in the constant pool with one copy for each function. I looked at this at file scope only (.s / .o), and I am hoping that either that is not a problem, or maybe that the linker optimizes this?

May 20 2020, 12:30 AM · Restricted Project

May 19 2020

jonpa updated the diff for D78486: [SystemZ] Expand vector zero-extend into a shuffle..

Patch rebased.

May 19 2020, 5:22 AM · Restricted Project
jonpa committed rGb3bd0c37eced: [SystemZ] Eliminate the need to create a zero vector by reusing the VPERM mask. (authored by jonpa).
[SystemZ] Eliminate the need to create a zero vector by reusing the VPERM mask.
May 19 2020, 1:03 AM