Page MenuHomePhabricator

RKSimon (Simon Pilgrim)
User

Projects

User does not belong to any projects.

User Details

User Since
May 5 2014, 7:26 AM (309 w, 2 d)

Recent Activity

Today

RKSimon added inline comments to D74060: [KnownBits] Move AND, OR and XOR logic into KnownBits.
Wed, Apr 8, 2:08 PM · Restricted Project
RKSimon added a comment to D77354: [DO NOT MERGE] added FileCheck colons that broke things.

@jroelofs Please can you rebase again? I think I've dealt with all the low hanging fruit now, so you will probably have to git blame to find code owners to help fix the remaining issues.

Wed, Apr 8, 10:51 AM · Restricted Project
RKSimon committed rG94121c60d622: [InstCombine] Regenerate phi-preserve-ir-flags.ll test checks to fix issue… (authored by RKSimon).
[InstCombine] Regenerate phi-preserve-ir-flags.ll test checks to fix issue…
Wed, Apr 8, 10:19 AM
RKSimon committed rGc8dfd0008b17: [PowerPC] Regenerate optcmp.ll test checks to fix issue reported on D77354 (authored by RKSimon).
[PowerPC] Regenerate optcmp.ll test checks to fix issue reported on D77354
Wed, Apr 8, 10:19 AM
RKSimon committed rGca5af4b649f4: [PowerPC] Fix fold-rlwinm.mir typos in checks to fix issue reported on D77354 (authored by RKSimon).
[PowerPC] Fix fold-rlwinm.mir typos in checks to fix issue reported on D77354
Wed, Apr 8, 9:46 AM
RKSimon committed rG68c6d7e11c51: [PowerPC] Fix fold-frame-offset-using-rr.mir typos in checks to fix issue… (authored by RKSimon).
[PowerPC] Fix fold-frame-offset-using-rr.mir typos in checks to fix issue…
Wed, Apr 8, 9:46 AM
RKSimon committed rG54502476e7bd: [AMDGPU] Refresh fmin_legacy.ll checks to fix issue reported on D77354 (authored by RKSimon).
[AMDGPU] Refresh fmin_legacy.ll checks to fix issue reported on D77354
Wed, Apr 8, 9:13 AM
RKSimon committed rG5f25d22d3f01: [ARM] Fix thumb1_return_sequence typo in check to fix issue reported on D77354 (authored by RKSimon).
[ARM] Fix thumb1_return_sequence typo in check to fix issue reported on D77354
Wed, Apr 8, 8:08 AM
RKSimon committed rG7b65b1ecd118: [ARM] Fix misched-int-basic-thumb2.mir typo in check to fix issue reported on… (authored by RKSimon).
[ARM] Fix misched-int-basic-thumb2.mir typo in check to fix issue reported on…
Wed, Apr 8, 8:08 AM
RKSimon committed rGc6cd0936be50: [AARCH64][GISEL] arm64-fallback.ll - Refresh remarks to fix issue reported on… (authored by RKSimon).
[AARCH64][GISEL] arm64-fallback.ll - Refresh remarks to fix issue reported on…
Wed, Apr 8, 7:35 AM
RKSimon committed rG898e22908c4f: [MemorySSA] invariant-groups.ll - add missing check to fix issue reported on… (authored by RKSimon).
[MemorySSA] invariant-groups.ll - add missing check to fix issue reported on…
Wed, Apr 8, 7:35 AM
RKSimon committed rG0c2ab6368994: [CodeExtractor] Fix typo in check label to fix issue reported on D77354 (authored by RKSimon).
[CodeExtractor] Fix typo in check label to fix issue reported on D77354
Wed, Apr 8, 7:03 AM
RKSimon committed rG7e62684251ed: [AMDGPU] Regenerate vector-extract-insert test checks to fix issue reported on… (authored by RKSimon).
[AMDGPU] Regenerate vector-extract-insert test checks to fix issue reported on…
Wed, Apr 8, 5:24 AM
RKSimon committed rG98181a1f98e7: [AMDGPU] Regenerate si-annotate-cfg-loop-assert test checks to fix issue… (authored by RKSimon).
[AMDGPU] Regenerate si-annotate-cfg-loop-assert test checks to fix issue…
Wed, Apr 8, 5:24 AM
RKSimon committed rG66c18c729da4: [X86][SSE] Combine PTEST(AND(X,Y),AND(X,Y)) -> PTEST(X,Y) and ANDN equivalents (authored by RKSimon).
[X86][SSE] Combine PTEST(AND(X,Y),AND(X,Y)) -> PTEST(X,Y) and ANDN equivalents
Wed, Apr 8, 4:50 AM

Yesterday

RKSimon committed rG6f46e9af8a0e: [X86][SSE] Add PTEST(AND(X,Y),AND(X,Y)) tests derived from PR42035 examples (authored by RKSimon).
[X86][SSE] Add PTEST(AND(X,Y),AND(X,Y)) tests derived from PR42035 examples
Tue, Apr 7, 10:20 AM
RKSimon updated subscribers of rL231475: [AsmPrinter][TLOF] 32-bit MachO support for replacing GOT equivalents.
Tue, Apr 7, 10:18 AM
RKSimon committed rGe3b60597769f: [X86][SSE] combineX86ShufflesConstants - early out for zeroable vectors… (authored by RKSimon).
[X86][SSE] combineX86ShufflesConstants - early out for zeroable vectors…
Tue, Apr 7, 7:02 AM
RKSimon added inline comments to D77621: Change BitcodeWriter buffer to std::vector instead of SmallVector..
Tue, Apr 7, 3:12 AM · Restricted Project, Restricted Project

Mon, Apr 6

RKSimon added a comment to rGfabe52a7412d: Fix uninitialized variable warning. NFC..

That one is a bit of blur (a lots happened since then.....) but I think it was one of the more obscure buildbots? Maybe one of the clang-ppc64 bots running a funny gcc version?

Mon, Apr 6, 11:57 AM
RKSimon accepted D77393: [X86] Fix implicit sign conversion warnings in X86 headers..

LGTM with the minor addition an extra CHECK-LABEL

Mon, Apr 6, 11:57 AM · Restricted Project
RKSimon added a comment to D77463: [ARM] Remove condition that could never be true.

Thanks for fixing the warning. LGTM but an ARM specialist should accept based on the test case.

Mon, Apr 6, 9:44 AM · Restricted Project
RKSimon added a comment to D77393: [X86] Fix implicit sign conversion warnings in X86 headers..

@craig.topper Any comments? Hopefully we can add extra tests as time goes on (for instance I'm intending to add Wdocumentation test coverage for PR35949).

Mon, Apr 6, 8:38 AM · Restricted Project
RKSimon committed rG9bc5b1a489c6: [X86][SSE] combineVectorSignBitsTruncation - remove minimum vector length… (authored by RKSimon).
[X86][SSE] combineVectorSignBitsTruncation - remove minimum vector length…
Mon, Apr 6, 4:50 AM
RKSimon accepted D51737: DAG: Combine extract_vector_elt of concat_vectors.

LGTM

Mon, Apr 6, 4:17 AM

Sun, Apr 5

RKSimon committed rGa43e23360657: Remove unused function 'isInRange'. NFCI. (authored by RKSimon).
Remove unused function 'isInRange'. NFCI.
Sun, Apr 5, 3:30 PM
RKSimon committed rG4431a29c60e7: [X86][SSE] Combine unary shuffle(HORIZOP,HORIZOP) -> HORIZOP (authored by RKSimon).
[X86][SSE] Combine unary shuffle(HORIZOP,HORIZOP) -> HORIZOP
Sun, Apr 5, 2:58 PM
RKSimon committed rG3079e518589c: [X86][SSE] Generalize shuffle(HORIZOP,HORIZOP) -> HORIZOP combine (authored by RKSimon).
[X86][SSE] Generalize shuffle(HORIZOP,HORIZOP) -> HORIZOP combine
Sun, Apr 5, 4:15 AM
RKSimon committed rGa17de6b91cff: [X86][SSE] truncateVectorWithPACK - upper undef for 128->64 packing (authored by RKSimon).
[X86][SSE] truncateVectorWithPACK - upper undef for 128->64 packing
Sun, Apr 5, 4:15 AM

Sat, Apr 4

RKSimon committed rGbe84d2b5b7e9: [CostModel][X86] Add some insert subvector cost tests for… (authored by RKSimon).
[CostModel][X86] Add some insert subvector cost tests for…
Sat, Apr 4, 2:54 PM
RKSimon committed rG3380d4d75e84: [X86] Cleanup vectorcall test checks (authored by RKSimon).
[X86] Cleanup vectorcall test checks
Sat, Apr 4, 2:54 PM
RKSimon committed rG6a57ba17c032: [CostModel][X86] Add shuffle cost tests for sub-128bit vectors (authored by RKSimon).
[CostModel][X86] Add shuffle cost tests for sub-128bit vectors
Sat, Apr 4, 5:15 AM
RKSimon committed rG87fd686f6fdd: [CostModel][X86] Add insert/extract cost tests for sub-128bit vXi8/vXi16 vectors (authored by RKSimon).
[CostModel][X86] Add insert/extract cost tests for sub-128bit vXi8/vXi16 vectors
Sat, Apr 4, 5:15 AM
RKSimon committed rGe5e719d8854a: [X86][SSE] lowerV8I16Shuffle - lower compaction shuffles using PACKUSDW(PBLENDW… (authored by RKSimon).
[X86][SSE] lowerV8I16Shuffle - lower compaction shuffles using PACKUSDW(PBLENDW…
Sat, Apr 4, 5:15 AM
RKSimon added a comment to D77354: [DO NOT MERGE] added FileCheck colons that broke things.

@jroelofs please can you rebase to see what test are still broken?

Sat, Apr 4, 3:02 AM · Restricted Project

Fri, Apr 3

RKSimon added inline comments to D77301: [TargetLowering] Improve expansion of FSHL/FSHR.
Fri, Apr 3, 2:38 PM · Restricted Project
RKSimon committed rG22257975679d: [YAMLParser] Scanner::setError - ensure we use the StringRef::iterator argument… (authored by RKSimon).
[YAMLParser] Scanner::setError - ensure we use the StringRef::iterator argument…
Fri, Apr 3, 11:21 AM
RKSimon closed D76512: [YAMLParser] Scanner::setError - ensure we use the StringRef::iterator argument (PR45043).
Fri, Apr 3, 11:20 AM · Restricted Project
RKSimon committed rG34a497b7653b: [X86][SSE] lowerShuffleWithPACK - extend to use chained PACKs for larger… (authored by RKSimon).
[X86][SSE] lowerShuffleWithPACK - extend to use chained PACKs for larger…
Fri, Apr 3, 10:47 AM
RKSimon committed rG396b1ee0e0b0: [LoopStrengthReduce] Fix test checks to fix issue reported on D77227 (authored by RKSimon).
[LoopStrengthReduce] Fix test checks to fix issue reported on D77227
Fri, Apr 3, 10:16 AM
RKSimon committed rG30053c842ca4: [AArch64] Fix swap-compare-operands test names to fix issue reported on D77354 (authored by RKSimon).
[AArch64] Fix swap-compare-operands test names to fix issue reported on D77354
Fri, Apr 3, 10:15 AM
RKSimon committed rG6d24dd7ed119: [InstSimplify] Regenerate compares tests to fix issue reported on D77354 (authored by RKSimon).
[InstSimplify] Regenerate compares tests to fix issue reported on D77354
Fri, Apr 3, 9:42 AM
RKSimon committed rG43d2fc7ed74e: [LoopRotate] Cleanup test checks to fix issue reported on D77354 (authored by RKSimon).
[LoopRotate] Cleanup test checks to fix issue reported on D77354
Fri, Apr 3, 9:42 AM
RKSimon committed rG5f47f613de50: [PowerPC] Regenerate f128 test to fix issue reported on D77354 (authored by RKSimon).
[PowerPC] Regenerate f128 test to fix issue reported on D77354
Fri, Apr 3, 9:09 AM
RKSimon committed rG80d4df5be661: [X86] Remove defunct section checks from emulated TLS tests to fix issue… (authored by RKSimon).
[X86] Remove defunct section checks from emulated TLS tests to fix issue…
Fri, Apr 3, 9:08 AM
RKSimon committed rG40fc3de369a3: [X86] Fix weak global label issue reported on D77354 (authored by RKSimon).
[X86] Fix weak global label issue reported on D77354
Fri, Apr 3, 8:36 AM
RKSimon committed rG58c242e7b874: [X86] Fix gisel copy tests to fix issue reported on D77354 (authored by RKSimon).
[X86] Fix gisel copy tests to fix issue reported on D77354
Fri, Apr 3, 8:36 AM
RKSimon committed rGe9511c0206d4: [X86] Fix strong local function/global label issue reported on D77354 (authored by RKSimon).
[X86] Fix strong local function/global label issue reported on D77354
Fri, Apr 3, 8:36 AM
RKSimon committed rGaa8434fa3d1d: [X86] Cleanup emulated TLS test checks. NFC (authored by RKSimon).
[X86] Cleanup emulated TLS test checks. NFC
Fri, Apr 3, 8:36 AM
RKSimon added a reviewer for D77393: [X86] Fix implicit sign conversion warnings in X86 headers.: RKSimon.

Rename test file to x86-header-warnings.c or something similarly specific?

Fri, Apr 3, 7:29 AM · Restricted Project
RKSimon committed rG7521f3c2f068: [X86] Regenerate soft fp legalization test to fix issue reported on D77354 (authored by RKSimon).
[X86] Regenerate soft fp legalization test to fix issue reported on D77354
Fri, Apr 3, 6:59 AM
RKSimon committed rG74f00c66dd36: [X86] Regenerate stack clash test to fix issue reported on D77354 (authored by RKSimon).
[X86] Regenerate stack clash test to fix issue reported on D77354
Fri, Apr 3, 6:59 AM
RKSimon committed rG5e426363ba4b: [X86][AVX] Add tests showing failure to use chained PACKSS/PACKUS for multi… (authored by RKSimon).
[X86][AVX] Add tests showing failure to use chained PACKSS/PACKUS for multi…
Fri, Apr 3, 6:25 AM
RKSimon added reviewers for D72423: [DemandedBits] Improve accuracy of Add propagator: regehr, nikic.

Adding @regehr and @nikic as they might have some thoughts on this

Fri, Apr 3, 5:20 AM · Restricted Project
RKSimon added a comment to D75567: [x86] Enable bypassing 64-bit division on generic x86-64.

Test cases?

Fri, Apr 3, 3:11 AM · Restricted Project
RKSimon added reviewers for D76512: [YAMLParser] Scanner::setError - ensure we use the StringRef::iterator argument (PR45043): spatel, rnk, thomasfinch.
Fri, Apr 3, 3:11 AM · Restricted Project
RKSimon accepted D77300: [X86] Improve combineVectorShiftImm.

LGTM

Fri, Apr 3, 3:11 AM · Restricted Project

Thu, Apr 2

RKSimon accepted D76623: [VectorCombine] try to form a better extractelement.

LGTM

Thu, Apr 2, 2:38 PM · Restricted Project
RKSimon added a comment to D40633: [PCG] Poor shuffle lane tracking (PR35454 ).

@kbelochapka I think we can abandon this now the vector combine pass handles PR35454

Thu, Apr 2, 2:38 PM
RKSimon added inline comments to D77301: [TargetLowering] Improve expansion of FSHL/FSHR.
Thu, Apr 2, 10:50 AM · Restricted Project
RKSimon added inline comments to D77319: [DAGCombine] Remove the getNegatibleCost to avoid the out of sync with getNegatedExpression.
Thu, Apr 2, 10:17 AM · Restricted Project
RKSimon added reviewers for D77319: [DAGCombine] Remove the getNegatibleCost to avoid the out of sync with getNegatedExpression: RKSimon, spatel, efriedma, arsenm.
Thu, Apr 2, 10:17 AM · Restricted Project
RKSimon added inline comments to D77300: [X86] Improve combineVectorShiftImm.
Thu, Apr 2, 9:12 AM · Restricted Project
RKSimon added a comment to D74976: [CostModel][X86] Improve extract/insert element costs (PR43605).

We got a report that this slows down PDF rendering in pdfium rendering of one pdf by 11%, and it slowed down rendering of 6% of pdfs of a test corpus by 2+%.

https://bugs.chromium.org/p/chromium/issues/detail?id=1067111 has repro steps. Could you take a look?

Thu, Apr 2, 8:06 AM · Restricted Project
RKSimon added inline comments to D77300: [X86] Improve combineVectorShiftImm.
Thu, Apr 2, 7:33 AM · Restricted Project
RKSimon committed rGb02c7a81523f: Fix "result of 32-bit shift implicitly converted to 64 bits" MSVC warning. NFCI. (authored by RKSimon).
Fix "result of 32-bit shift implicitly converted to 64 bits" MSVC warning. NFCI.
Thu, Apr 2, 4:19 AM
RKSimon committed rG316e220cf432: [llvm-mca] Cleanup unnecessary includes from headers (authored by RKSimon).
[llvm-mca] Cleanup unnecessary includes from headers
Thu, Apr 2, 4:19 AM
RKSimon closed D77073: [llvm-mca] Cleanup unnecessary includes from headers.
Thu, Apr 2, 4:18 AM · Restricted Project

Wed, Apr 1

RKSimon committed rGeb8880562e69: [X86][SSE] combinePTESTCC - fold TESTZ(X,~Y) -> TESTC(Y,X) (authored by RKSimon).
[X86][SSE] combinePTESTCC - fold TESTZ(X,~Y) -> TESTC(Y,X)
Wed, Apr 1, 7:41 AM
RKSimon committed rGbe7a233e9376: Fix operator precedence warning. NFCI. (authored by RKSimon).
Fix operator precedence warning. NFCI.
Wed, Apr 1, 7:08 AM
RKSimon committed rG552e46ea1efd: Fix unused variable warnings. NFCI. (authored by RKSimon).
Fix unused variable warnings. NFCI.
Wed, Apr 1, 7:08 AM
RKSimon committed rG481413d39404: [X86][SSE] matchShuffleWithPACK - generalize zero/signbits matching for any… (authored by RKSimon).
[X86][SSE] matchShuffleWithPACK - generalize zero/signbits matching for any…
Wed, Apr 1, 6:35 AM
RKSimon committed rG43eca880c6ed: Fix "control reaches end of non-void function" warning. NFCI. (authored by RKSimon).
Fix "control reaches end of non-void function" warning. NFCI.
Wed, Apr 1, 6:35 AM
RKSimon added a comment to D76512: [YAMLParser] Scanner::setError - ensure we use the StringRef::iterator argument (PR45043).

ping?

Wed, Apr 1, 5:30 AM · Restricted Project
RKSimon committed rG918ccb64b022: [X86][SSE] Handle basic inversion of PTEST/TESTP operands (PR38522) (authored by RKSimon).
[X86][SSE] Handle basic inversion of PTEST/TESTP operands (PR38522)
Wed, Apr 1, 3:52 AM
RKSimon closed D76984: [X86][SSE] Handle basic inversion of PTEST/TESTP operands (PR38522).
Wed, Apr 1, 3:52 AM · Restricted Project
RKSimon added a comment to D77201: [CodeGen][SelectionDAG] Flip Booleans More Often.

Please update your description - you're not explaining the mechanics of your change at all.

Wed, Apr 1, 2:43 AM · Restricted Project
RKSimon committed rGf9f401dba179: [X86][AVX] Add additional 256/512-bit test cases for PACKSS/PACKUS shuffle… (authored by RKSimon).
[X86][AVX] Add additional 256/512-bit test cases for PACKSS/PACKUS shuffle…
Wed, Apr 1, 12:31 AM
RKSimon committed rG3c9064ed963e: [X86] Run XOP vector rotation tests with/without AVX2 (authored by RKSimon).
[X86] Run XOP vector rotation tests with/without AVX2
Wed, Apr 1, 12:31 AM
RKSimon accepted D77136: [X86] Do not assume types are legal in getFauxShuffleMask.

LGTM - cheers

Wed, Apr 1, 12:30 AM · Restricted Project
RKSimon accepted D77183: [VectorUtils][X86] De-templatize scaleShuffleMask and 2 X86 shuffle mask helpers and move their implementation to cpp files.

LGTM - cheers

Wed, Apr 1, 12:30 AM · Restricted Project

Tue, Mar 31

RKSimon updated the diff for D76984: [X86][SSE] Handle basic inversion of PTEST/TESTP operands (PR38522).

Tweaked new ptestnzc_256_invert0_commute test for X86::COND_BE case

Tue, Mar 31, 2:53 PM · Restricted Project
RKSimon updated the diff for D76984: [X86][SSE] Handle basic inversion of PTEST/TESTP operands (PR38522).

Rebased tests and added COND_BE testnzc case.

Tue, Mar 31, 2:53 PM · Restricted Project
RKSimon committed rG30436a1ce7f3: [X86][SSE] Add additional PTEST/TESTP inversion tests (authored by RKSimon).
[X86][SSE] Add additional PTEST/TESTP inversion tests
Tue, Mar 31, 10:03 AM
RKSimon committed rG8b925440d11b: [X86][SSE] Simplify PTEST/TESTP tests for D76984 (authored by RKSimon).
[X86][SSE] Simplify PTEST/TESTP tests for D76984
Tue, Mar 31, 10:03 AM
RKSimon updated the diff for D76984: [X86][SSE] Handle basic inversion of PTEST/TESTP operands (PR38522).

Move combine inside combineSetCCEFLAGS

Tue, Mar 31, 8:49 AM · Restricted Project
RKSimon committed rG7e0e5fa499c7: Revert rGefe59d6717dcdf7777acb9b7a734e1a520bdf22a "[X86][SSE]… (authored by RKSimon).
Revert rGefe59d6717dcdf7777acb9b7a734e1a520bdf22a "[X86][SSE]…
Tue, Mar 31, 8:17 AM
RKSimon added a reverting change for rGefe59d6717dc: [X86][SSE] lowerShuffleWithPACK - extend to use chained PACKs for larger…: rG7e0e5fa499c7: Revert rGefe59d6717dcdf7777acb9b7a734e1a520bdf22a "[X86][SSE]….
Tue, Mar 31, 8:17 AM
RKSimon committed rG38619fa7da38: Fix enumeral mismatch warning. NFCI. (authored by RKSimon).
Fix enumeral mismatch warning. NFCI.
Tue, Mar 31, 7:45 AM
RKSimon committed rGefe59d6717dc: [X86][SSE] lowerShuffleWithPACK - extend to use chained PACKs for larger… (authored by RKSimon).
[X86][SSE] lowerShuffleWithPACK - extend to use chained PACKs for larger…
Tue, Mar 31, 7:10 AM
RKSimon added inline comments to D77136: [X86] Do not assume types are legal in getFauxShuffleMask.
Tue, Mar 31, 5:32 AM · Restricted Project
RKSimon committed rG98357dee1c6b: [X86] Combine concat(palignr,palignr) -> palignr(concat,concat) (authored by RKSimon).
[X86] Combine concat(palignr,palignr) -> palignr(concat,concat)
Tue, Mar 31, 3:20 AM
RKSimon committed rG7a4a98a9c4f3: [X86] Move canLowerByDroppingEvenElements earlier to be with… (authored by RKSimon).
[X86] Move canLowerByDroppingEvenElements earlier to be with…
Tue, Mar 31, 3:19 AM

Mon, Mar 30

RKSimon created D77073: [llvm-mca] Cleanup unnecessary includes from headers.
Mon, Mar 30, 10:16 AM · Restricted Project
RKSimon committed rGe95d04f4f19d: [X86][AVX] lowerV4X128Shuffle - attempt to widen to 2x256 to simplify shuffles (authored by RKSimon).
[X86][AVX] lowerV4X128Shuffle - attempt to widen to 2x256 to simplify shuffles
Mon, Mar 30, 4:50 AM

Sun, Mar 29

RKSimon committed rG9c8ec99c80a5: [X86][AVX] Combine 128/256-bit lane shuffles with zeroable upper subvectors to… (authored by RKSimon).
[X86][AVX] Combine 128/256-bit lane shuffles with zeroable upper subvectors to…
Sun, Mar 29, 12:18 PM
RKSimon committed rGfe0723dc9d45: Fix -Wdocumentation warning. NFC. (authored by RKSimon).
Fix -Wdocumentation warning. NFC.
Sun, Mar 29, 12:18 PM
RKSimon committed rG8206c50cdec1: [X86] Add isAnyZero shuffle mask helper (authored by RKSimon).
[X86] Add isAnyZero shuffle mask helper
Sun, Mar 29, 12:18 PM
RKSimon committed rG443dcc0e008b: [X86][AVX] Add tests for 512-bit shuffle patterns that could reduce to… (authored by RKSimon).
[X86][AVX] Add tests for 512-bit shuffle patterns that could reduce to…
Sun, Mar 29, 10:41 AM
RKSimon committed rGb44f07045c53: Remove unnecessary empty comments from test check lines. NFC. (authored by RKSimon).
Remove unnecessary empty comments from test check lines. NFC.
Sun, Mar 29, 10:41 AM