Page MenuHomePhabricator

RKSimon (Simon Pilgrim)
User

Projects

User does not belong to any projects.

User Details

User Since
May 5 2014, 7:26 AM (327 w, 4 d)

Recent Activity

Today

RKSimon committed rGe9eb2dc33261: [X86][SSE] Fold HOP(SHUFFLE(X),SHUFFLE(Y)) --> SHUFFLE(HOP(X,Y)) (authored by RKSimon).
[X86][SSE] Fold HOP(SHUFFLE(X),SHUFFLE(Y)) --> SHUFFLE(HOP(X,Y))
Fri, Aug 14, 10:54 AM
RKSimon updated subscribers of rG798eb71c3a5a: [NFC][StackSafety] Dedup callees.

@vitalybuka Please can you investigate this buildbot failure: http://lab.llvm.org:8011/builders/llvm-clang-x86_64-expensive-checks-ubuntu/builds/8198 - it appears to be due to an APInt being self-assigned

Fri, Aug 14, 10:52 AM
RKSimon added inline comments to D81647: MIR Statepoint refactoring. Part 3: Spill GC Ptr regs..
Fri, Aug 14, 8:05 AM · Restricted Project
RKSimon committed rG25ce6341729d: [DemandedBits] Add addition test case from D72423 (authored by RKSimon).
[DemandedBits] Add addition test case from D72423
Fri, Aug 14, 8:02 AM
RKSimon added inline comments to D72423: [DemandedBits] Improve accuracy of Add propagator.
Fri, Aug 14, 7:38 AM · Restricted Project
RKSimon added inline comments to D81647: MIR Statepoint refactoring. Part 3: Spill GC Ptr regs..
Fri, Aug 14, 7:09 AM · Restricted Project

Yesterday

RKSimon committed rG3a2645e42899: [X86][SSE] Add HADD combine regression case from rG9bd97d036398 (authored by RKSimon).
[X86][SSE] Add HADD combine regression case from rG9bd97d036398
Thu, Aug 13, 10:27 AM
RKSimon committed rG63863451d18b: Fix unused variable warning. NFC. (authored by RKSimon).
Fix unused variable warning. NFC.
Thu, Aug 13, 7:44 AM
RKSimon added a reverting change for rG9bd97d036398: [X86][SSE] Fold HOP(SHUFFLE(X),SHUFFLE(Y)) --> SHUFFLE(HOP(X,Y)): rGcd3b850a4c8e: rG9bd97d0363987b582 - Revert "[X86][SSE] Fold HOP(SHUFFLE(X),SHUFFLE(Y)) -->….
Thu, Aug 13, 7:44 AM
RKSimon committed rGcd3b850a4c8e: rG9bd97d0363987b582 - Revert "[X86][SSE] Fold HOP(SHUFFLE(X),SHUFFLE(Y)) -->… (authored by RKSimon).
rG9bd97d0363987b582 - Revert "[X86][SSE] Fold HOP(SHUFFLE(X),SHUFFLE(Y)) -->…
Thu, Aug 13, 7:44 AM
RKSimon committed rGa31d20e67e2b: [X86][SSE] IsElementEquivalent - add HOP(X,X) support (authored by RKSimon).
[X86][SSE] IsElementEquivalent - add HOP(X,X) support
Thu, Aug 13, 4:59 AM
RKSimon committed rG8a41a1f5673e: BranchFolding.cpp - removes includes already included by BranchFolding.h. NFC. (authored by RKSimon).
BranchFolding.cpp - removes includes already included by BranchFolding.h. NFC.
Thu, Aug 13, 4:15 AM
RKSimon committed rGebfa41043328: SplitKit.cpp - removes includes already included by SplitKit.h. NFC. (authored by RKSimon).
SplitKit.cpp - removes includes already included by SplitKit.h. NFC.
Thu, Aug 13, 3:44 AM
RKSimon committed rGc4c1267cadbc: DwarfDebug.cpp - removes includes already included by DwarfDebug.h. NFC. (authored by RKSimon).
DwarfDebug.cpp - removes includes already included by DwarfDebug.h. NFC.
Thu, Aug 13, 3:44 AM
RKSimon added a comment to D81500: [SVE] Remove calls to VectorType::getNumElements from IR.

a few minors - you might also want to keep an eye on D85794

Thu, Aug 13, 3:41 AM · Restricted Project

Wed, Aug 12

RKSimon added a comment to D85165: [X86][MC][Target] Initial backend support a tune CPU to support -mtune.

The -misched-print-dags approach seems to be doable, and not too unpretty.

Wed, Aug 12, 11:47 AM · Restricted Project
RKSimon committed rG39de63aef953: Fix signed/unsigned comparison warnings. NFC. (authored by RKSimon).
Fix signed/unsigned comparison warnings. NFC.
Wed, Aug 12, 11:23 AM
RKSimon committed rG13d6cf0951f4: [X86][SSE] Pull out BUILD_VECTOR operand equivalence tests. NFC. (authored by RKSimon).
[X86][SSE] Pull out BUILD_VECTOR operand equivalence tests. NFC.
Wed, Aug 12, 10:20 AM
RKSimon committed rG9bd97d036398: [X86][SSE] Fold HOP(SHUFFLE(X),SHUFFLE(Y)) --> SHUFFLE(HOP(X,Y)) (authored by RKSimon).
[X86][SSE] Fold HOP(SHUFFLE(X),SHUFFLE(Y)) --> SHUFFLE(HOP(X,Y))
Wed, Aug 12, 4:21 AM
RKSimon committed rGa0c2c6aa420e: [X86][AVX] Fold CONCAT(HOP(X,Y),HOP(Z,W)) -> HOP(CONCAT(X,Z),CONCAT(Y,W)) for… (authored by RKSimon).
[X86][AVX] Fold CONCAT(HOP(X,Y),HOP(Z,W)) -> HOP(CONCAT(X,Z),CONCAT(Y,W)) for…
Wed, Aug 12, 3:44 AM

Tue, Aug 11

RKSimon added a comment to D85165: [X86][MC][Target] Initial backend support a tune CPU to support -mtune.

@andreadb How well could we test this using the llvm-mca resource schedule tests?

Tue, Aug 11, 1:40 PM · Restricted Project
RKSimon added a comment to D82995: [UpdateTestChecks] Allow $ in function names.

@greened Any luck with a test case?

Tue, Aug 11, 12:57 PM · Restricted Project
RKSimon added inline comments to D74730: [FPEnv][X86] Implement lowering of llvm.set.rounding.
Tue, Aug 11, 12:56 PM · Restricted Project
RKSimon added inline comments to D83036: [X86][FPEnv] Lowering of {get,set,reset}_fpmode.
Tue, Aug 11, 12:55 PM · Restricted Project
RKSimon added a comment to D85165: [X86][MC][Target] Initial backend support a tune CPU to support -mtune.

Do the x86 scheduling models do something sane with "impossible" instructions, like if you use AVX512 instructions with Atom tuning?

Tue, Aug 11, 12:47 PM · Restricted Project
RKSimon added a reviewer for D85759: [SLPVectorizer] Fix regression from cost model refactoring: ABataev.

Pre-commit the test case and rebase to show the diff?

Tue, Aug 11, 10:58 AM · Restricted Project
RKSimon committed rGb9aaf32f4649: Fix MSVC "not all control paths return a value" warning. NFC. (authored by RKSimon).
Fix MSVC "not all control paths return a value" warning. NFC.
Tue, Aug 11, 10:14 AM
RKSimon committed rG2655bd51d6a3: [X86][SSE] combineShuffleWithHorizOp - canonicalize SHUFFLE(HOP(X,Y),HOP(Y,X))… (authored by RKSimon).
[X86][SSE] combineShuffleWithHorizOp - canonicalize SHUFFLE(HOP(X,Y),HOP(Y,X))…
Tue, Aug 11, 10:14 AM
RKSimon committed rGfe1f36986b23: [X86][SSE] combineShuffleWithHorizOp - avoid unnecessary subtraction. NFCI. (authored by RKSimon).
[X86][SSE] combineShuffleWithHorizOp - avoid unnecessary subtraction. NFCI.
Tue, Aug 11, 9:12 AM
RKSimon committed rG91d59cbf1b38: [X86][SSE] Add HADD/SUB support to combineHorizOpWithShuffle (authored by RKSimon).
[X86][SSE] Add HADD/SUB support to combineHorizOpWithShuffle
Tue, Aug 11, 8:16 AM
RKSimon committed rGd7d172b4aae0: [X86][SSE] Add tests for 256-bit HOP(SHUFFLE(X,Y),SHUFFLE(X,Y)) --> SHUFFLE(HOP… (authored by RKSimon).
[X86][SSE] Add tests for 256-bit HOP(SHUFFLE(X,Y),SHUFFLE(X,Y)) --> SHUFFLE(HOP…
Tue, Aug 11, 6:03 AM
RKSimon committed rGc5683ffeb194: [X86][SSE] Add tests for HOP(LOSUBVECTOR(SHUFFLE(X)),HISUBVECTOR(SHUFFLE(X)))… (authored by RKSimon).
[X86][SSE] Add tests for HOP(LOSUBVECTOR(SHUFFLE(X)),HISUBVECTOR(SHUFFLE(X)))…
Tue, Aug 11, 6:03 AM
RKSimon committed rG49193e1fe7e1: Fix Wdocumentation unknown param warnings. NFC. (authored by RKSimon).
Fix Wdocumentation unknown param warnings. NFC.
Tue, Aug 11, 4:53 AM
RKSimon committed rGee17f72e13b8: Fix Wdocumentation unknown param warning. NFC. (authored by RKSimon).
Fix Wdocumentation unknown param warning. NFC.
Tue, Aug 11, 4:53 AM
RKSimon added a comment to D77152: [SelectionDAG] Better legalization for FSHL and FSHR.

@asb Any thoughts on the RISCV changes? This is the last blocker on the patch.

Tue, Aug 11, 3:45 AM · Restricted Project
RKSimon committed rG49016eeab6bc: [X86] Rename combineVectorPackWithShuffle -> combineHorizOpWithShuffle. NFC. (authored by RKSimon).
[X86] Rename combineVectorPackWithShuffle -> combineHorizOpWithShuffle. NFC.
Tue, Aug 11, 3:39 AM
RKSimon committed rG73a6a3646946: [clangd] RIFF.cpp - Use logical && instead of bitwise & for padding check (authored by RKSimon).
[clangd] RIFF.cpp - Use logical && instead of bitwise & for padding check
Tue, Aug 11, 3:39 AM
RKSimon added inline comments to D57779: [SLP] Add support for throttling..
Tue, Aug 11, 2:36 AM · Restricted Project

Mon, Aug 10

RKSimon added inline comments to D57779: [SLP] Add support for throttling..
Mon, Aug 10, 8:42 AM · Restricted Project
RKSimon committed rG9a368d2b0088: [X86][SSE] shuffle(hop,hop) - canonicalize unary hop(x,x) shuffle masks (authored by RKSimon).
[X86][SSE] shuffle(hop,hop) - canonicalize unary hop(x,x) shuffle masks
Mon, Aug 10, 8:10 AM
RKSimon committed rG90f721404ff8: [SLP] Regenerate load-merge.ll tests (authored by RKSimon).
[SLP] Regenerate load-merge.ll tests
Mon, Aug 10, 8:10 AM
RKSimon committed rG07e673a02bb1: [X86][SSE] Pull out shuffle(hop,hop) combine into combineShuffleWithHorizOp… (authored by RKSimon).
[X86][SSE] Pull out shuffle(hop,hop) combine into combineShuffleWithHorizOp…
Mon, Aug 10, 7:10 AM
RKSimon committed rG891ddfc64380: [X86][SSE] Add single undef element HADD test cases from PR34724 (authored by RKSimon).
[X86][SSE] Add single undef element HADD test cases from PR34724
Mon, Aug 10, 6:35 AM
RKSimon committed rGe6dc2c8ce7db: [X86][SSE] combineTargetShuffle - rearrange shuffle(hop,hop) matching to delay… (authored by RKSimon).
[X86][SSE] combineTargetShuffle - rearrange shuffle(hop,hop) matching to delay…
Mon, Aug 10, 6:35 AM
RKSimon added inline comments to D57779: [SLP] Add support for throttling..
Mon, Aug 10, 6:30 AM · Restricted Project
RKSimon updated the diff for D66004: [WIP][X86][SSE] SimplifyDemandedVectorEltsForTargetNode - add general shuffle combining support.

rebase

Mon, Aug 10, 3:46 AM · Restricted Project
RKSimon committed rG0b26c9eddc4f: [ScalarizeMaskedMemIntrin][X86] Refresh missed transform test cases from… (authored by RKSimon).
[ScalarizeMaskedMemIntrin][X86] Refresh missed transform test cases from…
Mon, Aug 10, 3:14 AM
RKSimon committed rGc0c3b9a25fee: [ScalarizeMaskedMemIntrin] Scalarize constant mask expandload as shuffle… (authored by RKSimon).
[ScalarizeMaskedMemIntrin] Scalarize constant mask expandload as shuffle…
Mon, Aug 10, 3:06 AM
RKSimon closed D85416: [ScalarizeMaskedMemIntrin] Scalarize constant mask expandload as shuffle(build_vector,pass_through).
Mon, Aug 10, 3:06 AM · Restricted Project
RKSimon added a comment to D85499: [X86] Canonicalize andnp for bitmask arithmetic.

Abandon this now D85553 has landed?

Mon, Aug 10, 2:01 AM · Restricted Project

Sun, Aug 9

RKSimon added inline comments to D85593: [InstCombine] ~(~X + Y) -> X - Y.
Sun, Aug 9, 1:07 AM · Restricted Project

Sat, Aug 8

RKSimon committed rGcc15380f103f: [X86][SSE] combineTargetShuffle - use scaleShuffleMask helper to widen shuffle… (authored by RKSimon).
[X86][SSE] combineTargetShuffle - use scaleShuffleMask helper to widen shuffle…
Sat, Aug 8, 11:37 AM
RKSimon committed rGf13e92d4b273: [InstCombine] Use CreateVectorSplat(ElementCount) variant directly (authored by RKSimon).
[InstCombine] Use CreateVectorSplat(ElementCount) variant directly
Sat, Aug 8, 11:26 AM
RKSimon committed rG090f9d5a5533: Fix MSVC "not all control paths return a value" warning. NFC. (authored by RKSimon).
Fix MSVC "not all control paths return a value" warning. NFC.
Sat, Aug 8, 11:26 AM
RKSimon added a comment to D64142: [SLP] try to create vector loads from bitcasted scalar pointers.

@spatel Has this been superseded by D81766?

Sat, Aug 8, 3:40 AM · Restricted Project
RKSimon accepted D81766: [VectorCombine] try to create vector loads from scalar loads.

LGTM - the getScalarizationOverhead() change is OK

Sat, Aug 8, 1:24 AM · Restricted Project

Fri, Aug 7

RKSimon added a comment to D81766: [VectorCombine] try to create vector loads from scalar loads.

LGTM with one minor @nikic Does this look OK now?

Fri, Aug 7, 7:48 AM · Restricted Project
RKSimon committed rG66a163f32817: [DAG] GetDemandedBits - remove custom AND handling. (authored by RKSimon).
[DAG] GetDemandedBits - remove custom AND handling.
Fri, Aug 7, 4:56 AM
RKSimon committed rGfcefb5322251: Remove unreachable break. NFC (authored by RKSimon).
Remove unreachable break. NFC
Fri, Aug 7, 4:38 AM
RKSimon added inline comments to D55307: [analyzer] MoveChecker Pt.6: Suppress the warning for the few move-safe STL classes..
Fri, Aug 7, 3:57 AM
RKSimon committed rG18212cbcdadd: Remove duplicate/unreachable break (PR47029) (authored by RKSimon).
Remove duplicate/unreachable break (PR47029)
Fri, Aug 7, 3:49 AM
RKSimon committed rGf35992b75b4e: [SLP][X86] Add smax intrinsic reduction tests (authored by RKSimon).
[SLP][X86] Add smax intrinsic reduction tests
Fri, Aug 7, 3:49 AM
RKSimon committed rGffd258f7bbba: Remove unreachable return (PR47028) (authored by RKSimon).
Remove unreachable return (PR47028)
Fri, Aug 7, 3:33 AM
RKSimon committed rG24cca30f7f0d: Remove unreachable return (PR47026) (authored by RKSimon).
Remove unreachable return (PR47026)
Fri, Aug 7, 3:24 AM
RKSimon committed rGaa38e97ad5e4: [SLP][X86] Add abs/smax/smin/umax/umin intrinsic vectorization tests (authored by RKSimon).
[SLP][X86] Add abs/smax/smin/umax/umin intrinsic vectorization tests
Fri, Aug 7, 3:24 AM
RKSimon added a comment to D85499: [X86] Canonicalize andnp for bitmask arithmetic.

Are you saying you think we can get away with removing X86ISD::ANDNP entirely? We don't have it for the BMI scalar variant, and it /usually/ works.

Fri, Aug 7, 2:14 AM · Restricted Project
RKSimon added a comment to D85463: [GlobalISel] Add combine for (x & mask) -> x when (x & mask) == x.

Also yeah, it looks like this is handled in SelectionDAG::GetDemandedBits:

case ISD::AND: {
  // X & -1 -> X (ignoring bits which aren't demanded).
  // Also handle the case where masked out bits in X are known to be zero.
  if (ConstantSDNode *RHSC = isConstOrConstSplat(V.getOperand(1))) {
    const APInt &AndVal = RHSC->getAPIntValue();
    if (DemandedBits.isSubsetOf(AndVal) ||
        DemandedBits.isSubsetOf(computeKnownBits(V.getOperand(0)).Zero |
                                AndVal))
      return V.getOperand(0);
  }

So I guess that it might make sense to partially move this into GISelKnownBits.

Fri, Aug 7, 2:05 AM · Restricted Project

Thu, Aug 6

RKSimon added a comment to D85446: [InstCombine] Add vector support to mul(add(x,c),negpow2) -> mul(sub(-c,x),pow2) folds.

Thanks @lebedev.ri !

Thu, Aug 6, 1:58 PM · Restricted Project
RKSimon added inline comments to D83779: [SLP] Fix order of `insertelement`/`insertvalue` seed operands.
Thu, Aug 6, 10:03 AM · Restricted Project
RKSimon committed rG3b93464dcf2f: [SLP][X86] Regenerate sdiv test noticed in D83779. NFC. (authored by RKSimon).
[SLP][X86] Regenerate sdiv test noticed in D83779. NFC.
Thu, Aug 6, 10:00 AM
RKSimon added a comment to D85446: [InstCombine] Add vector support to mul(add(x,c),negpow2) -> mul(sub(-c,x),pow2) folds.

We miss a similar thing for sdiv exact %x, -1<<C --> -(ashr exact %x, C)
I can take over this patch if you prefer.

Thu, Aug 6, 9:57 AM · Restricted Project
RKSimon requested review of D85446: [InstCombine] Add vector support to mul(add(x,c),negpow2) -> mul(sub(-c,x),pow2) folds.
Thu, Aug 6, 9:34 AM · Restricted Project
RKSimon added a comment to D83779: [SLP] Fix order of `insertelement`/`insertvalue` seed operands.

Thanks, no more comments from me - the reverse_hadd_v4f32 fix will be very useful to x86 backend hadd formation!

Thu, Aug 6, 9:16 AM · Restricted Project
RKSimon committed rG8f5b2cb82807: [InstCombine] Add tests for mul(add(x,c),negpow2) -> mul(sub(-c,x),pow2) fold (authored by RKSimon).
[InstCombine] Add tests for mul(add(x,c),negpow2) -> mul(sub(-c,x),pow2) fold
Thu, Aug 6, 9:15 AM
RKSimon committed rGd1a91d947f49: [InstCombine] Add tests for mul(sub(x,y),negpow2) -> mul(sub(y,x),pow2) fold (authored by RKSimon).
[InstCombine] Add tests for mul(sub(x,y),negpow2) -> mul(sub(y,x),pow2) fold
Thu, Aug 6, 8:33 AM
RKSimon committed rGb7b1a38d41ca: PDBExtras.h - remove unnecessary raw_ostream forward declaration. NFCI. (authored by RKSimon).
PDBExtras.h - remove unnecessary raw_ostream forward declaration. NFCI.
Thu, Aug 6, 8:33 AM
RKSimon added a comment to rG87c5437afd27: PDBExtras.h - remove unnecessary raw_ostream forward declaration. NFCI..

I had to revert this as this includes several headers in the middle of a function which just doesn't work. See inline comment.

Thu, Aug 6, 8:11 AM
RKSimon added inline comments to D85416: [ScalarizeMaskedMemIntrin] Scalarize constant mask expandload as shuffle(build_vector,pass_through).
Thu, Aug 6, 8:01 AM · Restricted Project
RKSimon added inline comments to D83779: [SLP] Fix order of `insertelement`/`insertvalue` seed operands.
Thu, Aug 6, 6:05 AM · Restricted Project
RKSimon added a comment to D85399: [StackSafety] Use `Value::stripPointerCasts` to collect lifetime marker in StackLifetime.

test case?

Thu, Aug 6, 5:50 AM · Restricted Project
RKSimon committed rG3d10050e3748: BitstreamRemarkParser.h - remove unnecessary includes. NFCI. (authored by RKSimon).
BitstreamRemarkParser.h - remove unnecessary includes. NFCI.
Thu, Aug 6, 5:19 AM
RKSimon committed rG55ead5bfffdc: Fix include sorting order. NFC (authored by RKSimon).
Fix include sorting order. NFC
Thu, Aug 6, 4:00 AM
RKSimon committed rG807467009d6f: [X86] getX86MaskVec - replace mask limit from NumElts < 8 with NumElts <= 4 (authored by RKSimon).
[X86] getX86MaskVec - replace mask limit from NumElts < 8 with NumElts <= 4
Thu, Aug 6, 4:00 AM
RKSimon committed rG87c5437afd27: PDBExtras.h - remove unnecessary raw_ostream forward declaration. NFCI. (authored by RKSimon).
PDBExtras.h - remove unnecessary raw_ostream forward declaration. NFCI.
Thu, Aug 6, 3:29 AM
RKSimon committed rG13b4db4ec206: [X86][SSE] Expose all memory offsets in expand load tests (authored by RKSimon).
[X86][SSE] Expose all memory offsets in expand load tests
Thu, Aug 6, 3:29 AM
RKSimon requested review of D85416: [ScalarizeMaskedMemIntrin] Scalarize constant mask expandload as shuffle(build_vector,pass_through).
Thu, Aug 6, 3:04 AM · Restricted Project

Wed, Aug 5

RKSimon updated the diff for D66004: [WIP][X86][SSE] SimplifyDemandedVectorEltsForTargetNode - add general shuffle combining support.

rebase

Wed, Aug 5, 10:50 AM · Restricted Project
RKSimon committed rGb60f99885945: [X86][SSE] Fold 128-bit PACK(EXTEND(X),EXTEND(Y)) -> CONCAT(X,Y) subvectors (authored by RKSimon).
[X86][SSE] Fold 128-bit PACK(EXTEND(X),EXTEND(Y)) -> CONCAT(X,Y) subvectors
Wed, Aug 5, 10:29 AM
RKSimon accepted D85230: [PatternMatch] allow intrinsic form of min/max with existing matchers.

Thanks for checking! LGTM

Wed, Aug 5, 9:57 AM · Restricted Project
RKSimon committed rG6a06c7a0a768: [X86] isHorizontalBinOp - only update LHS/RHS references on success (authored by RKSimon).
[X86] isHorizontalBinOp - only update LHS/RHS references on success
Wed, Aug 5, 7:10 AM
RKSimon committed rGa57bfb44bc79: [X86][AVX] Fold CONCAT(HOP(X,Y),HOP(Z,W)) -> HOP(CONCAT(X,Z),CONCAT(Y,W)) for… (authored by RKSimon).
[X86][AVX] Fold CONCAT(HOP(X,Y),HOP(Z,W)) -> HOP(CONCAT(X,Z),CONCAT(Y,W)) for…
Wed, Aug 5, 7:10 AM
RKSimon added a comment to D83779: [SLP] Fix order of `insertelement`/`insertvalue` seed operands.

A couple of minor style comments - trivial scope reductions

Wed, Aug 5, 6:45 AM · Restricted Project
RKSimon committed rG7b993903e044: DWARFVerifier.h - remove unnecessary forward declarations and includes. NFCI. (authored by RKSimon).
DWARFVerifier.h - remove unnecessary forward declarations and includes. NFCI.
Wed, Aug 5, 4:43 AM
RKSimon committed rG315e1daf7f45: GISelWorkList.h - remove unnecessary includes. NFCI. (authored by RKSimon).
GISelWorkList.h - remove unnecessary includes. NFCI.
Wed, Aug 5, 4:01 AM
RKSimon committed rGe3d3657b9be1: CallLowering.h - remove unnecessary CCState forward declaration. NFCI. (authored by RKSimon).
CallLowering.h - remove unnecessary CCState forward declaration. NFCI.
Wed, Aug 5, 4:01 AM
RKSimon committed rG300899b9c4ed: [X86][AVX] Add test showing unnecessary duplicate HADD instructions (authored by RKSimon).
[X86][AVX] Add test showing unnecessary duplicate HADD instructions
Wed, Aug 5, 4:01 AM
RKSimon committed rG4aaf301fb8f4: [DAG] Fold vector (aext (load x)) -> (zext (truncate (zextload x))) (authored by RKSimon).
[DAG] Fold vector (aext (load x)) -> (zext (truncate (zextload x)))
Wed, Aug 5, 3:23 AM
RKSimon closed D85129: [DAG] Fold vector (aext (load x)) -> (zext (truncate (zextload x))).
Wed, Aug 5, 3:22 AM · Restricted Project

Tue, Aug 4

RKSimon closed D85229: Revert "[HotColdSplit] Add test case for unlikely attribute in outlined function".

This landed at rGf8cc94a61afe48a4b77f111a8ad313fc3b9417de

Tue, Aug 4, 2:22 PM · Restricted Project
RKSimon added a comment to D85230: [PatternMatch] allow intrinsic form of min/max with existing matchers.

Maybe make it clear at the start of InstCombinerImpl::foldSelectOpOp where we use the matchers that we can safely use the matchers for the select(icmp) pattern OR the intrinsics?

Tue, Aug 4, 1:57 PM · Restricted Project