Page MenuHomePhabricator

Please use GitHub pull requests for new patches. Phabricator shutdown timeline

MarcoFalke (Marco Falke)
User

Projects

User does not belong to any projects.

User Details

User Since
Jun 12 2018, 2:03 PM (276 w, 1 d)

Recent Activity

Jul 28 2023

MarcoFalke committed rG52526065bc80: [NFC][compiler-rt] Add missing space in libfuzzer -help docs (authored by MarcoFalke).
[NFC][compiler-rt] Add missing space in libfuzzer -help docs
Jul 28 2023, 1:00 AM · Unknown Object (Project), Restricted Project

Jul 17 2023

MarcoFalke added a comment to D155206: [compiler-rt][NFC] Avoid implicit-integer-sign-change in FuzzedDataProvider::ConsumeIntegralInRange.

I went ahead and merged this, because it should be trivially correct. Feel free to revert or let me know if there are any issues.

Jul 17 2023, 12:35 AM · Restricted Project, Unknown Object (Project)
MarcoFalke committed rGfa8401f9bfe8: [compiler-rt][NFC] Avoid implicit-integer-sign-change in FuzzedDataProvider… (authored by MarcoFalke).
[compiler-rt][NFC] Avoid implicit-integer-sign-change in FuzzedDataProvider…
Jul 17 2023, 12:33 AM · Unknown Object (Project), Restricted Project
MarcoFalke closed D155206: [compiler-rt][NFC] Avoid implicit-integer-sign-change in FuzzedDataProvider::ConsumeIntegralInRange.
Jul 17 2023, 12:33 AM · Restricted Project, Unknown Object (Project)

Jul 13 2023

MarcoFalke updated the diff for D155206: [compiler-rt][NFC] Avoid implicit-integer-sign-change in FuzzedDataProvider::ConsumeIntegralInRange.

Fix both places

Jul 13 2023, 7:54 AM · Restricted Project, Unknown Object (Project)
MarcoFalke requested review of D155206: [compiler-rt][NFC] Avoid implicit-integer-sign-change in FuzzedDataProvider::ConsumeIntegralInRange.
Jul 13 2023, 7:34 AM · Restricted Project, Unknown Object (Project)

Mar 23 2023

MarcoFalke committed rG8c10256734cd: clang-tidy: Detect use-after-move in CXXCtorInitializer (authored by MarcoFalke).
clang-tidy: Detect use-after-move in CXXCtorInitializer
Mar 23 2023, 2:24 AM · Restricted Project, Restricted Project
MarcoFalke closed D146288: clang-tidy: Detect use-after-move in CXXCtorInitializer.
Mar 23 2023, 2:23 AM · Restricted Project, Restricted Project

Mar 22 2023

MarcoFalke added a comment to D146288: clang-tidy: Detect use-after-move in CXXCtorInitializer.

Minor NFC, I plan to land this tomorrow or Friday

Mar 22 2023, 3:38 AM · Restricted Project, Restricted Project
MarcoFalke updated the diff for D146288: clang-tidy: Detect use-after-move in CXXCtorInitializer.

NFC

Mar 22 2023, 3:37 AM · Restricted Project, Restricted Project

Mar 20 2023

MarcoFalke added a comment to D146288: clang-tidy: Detect use-after-move in CXXCtorInitializer.

Thanks, addressed some more review comments.

Mar 20 2023, 10:07 AM · Restricted Project, Restricted Project
MarcoFalke updated the diff for D146288: clang-tidy: Detect use-after-move in CXXCtorInitializer.

address feedback about matcher, clang-format

Mar 20 2023, 10:02 AM · Restricted Project, Restricted Project
MarcoFalke added inline comments to D146288: clang-tidy: Detect use-after-move in CXXCtorInitializer.
Mar 20 2023, 7:56 AM · Restricted Project, Restricted Project
MarcoFalke updated the diff for D146288: clang-tidy: Detect use-after-move in CXXCtorInitializer.

Took some review suggestions (Thanks!)

Mar 20 2023, 7:54 AM · Restricted Project, Restricted Project

Mar 17 2023

MarcoFalke updated the diff for D146288: clang-tidy: Detect use-after-move in CXXCtorInitializer.

sort doc

Mar 17 2023, 7:37 AM · Restricted Project, Restricted Project
MarcoFalke updated the diff for D146288: clang-tidy: Detect use-after-move in CXXCtorInitializer.

clang-format+doc

Mar 17 2023, 5:55 AM · Restricted Project, Restricted Project
MarcoFalke requested review of D146288: clang-tidy: Detect use-after-move in CXXCtorInitializer.
Mar 17 2023, 5:04 AM · Restricted Project, Restricted Project

Mar 6 2023

MarcoFalke committed rGfa491fefb0f8: clang-tidy: Count template constructors in modernize-use-default-member-init (authored by MarcoFalke).
clang-tidy: Count template constructors in modernize-use-default-member-init
Mar 6 2023, 2:06 AM · Restricted Project, Restricted Project
MarcoFalke closed D143375: clang-tidy: Count template constructors in modernize-use-default-member-init.
Mar 6 2023, 2:05 AM · Restricted Project, Restricted Project

Mar 2 2023

MarcoFalke updated the diff for D143375: clang-tidy: Count template constructors in modernize-use-default-member-init.

mv release note, rebase

Mar 2 2023, 7:02 AM · Restricted Project, Restricted Project

Feb 27 2023

MarcoFalke updated the diff for D143375: clang-tidy: Count template constructors in modernize-use-default-member-init.

Thanks. Added release note. Lmk if I should go ahead and merge this.

Feb 27 2023, 2:30 AM · Restricted Project, Restricted Project

Feb 10 2023

MarcoFalke added a reviewer for D143375: clang-tidy: Count template constructors in modernize-use-default-member-init: oleg.smolsky.
Feb 10 2023, 1:55 AM · Restricted Project, Restricted Project

Feb 9 2023

MarcoFalke updated the diff for D143375: clang-tidy: Count template constructors in modernize-use-default-member-init.

Remove both test cases that pass on current main on Linux, but fail on Windows. Seems unrelated to add/fix them here.

Feb 9 2023, 6:00 AM · Restricted Project, Restricted Project
MarcoFalke updated the diff for D143375: clang-tidy: Count template constructors in modernize-use-default-member-init.

Simplify patch and remove test case failing on Windows, but passing on Linux

Feb 9 2023, 5:02 AM · Restricted Project, Restricted Project

Feb 6 2023

MarcoFalke requested review of D143375: clang-tidy: Count template constructors in modernize-use-default-member-init.
Feb 6 2023, 2:00 AM · Restricted Project, Restricted Project

Jan 25 2023

MarcoFalke committed rG007b35369501: doc: Fix typo in clang-tidy no-automatic-move doc (authored by MarcoFalke).
doc: Fix typo in clang-tidy no-automatic-move doc
Jan 25 2023, 6:40 AM · Restricted Project, Restricted Project

Nov 10 2021

MarcoFalke committed rGfa1729067c74: [compiler-rt] Fix typo in DeadlockDetector (chanding->changing) (authored by MarcoFalke).
[compiler-rt] Fix typo in DeadlockDetector (chanding->changing)
Nov 10 2021, 12:52 PM
MarcoFalke committed rGfaa019c0e38e: [libc++] Fix segmentation fault in __do_put_integral (authored by MarcoFalke).
[libc++] Fix segmentation fault in __do_put_integral
Nov 10 2021, 12:27 PM
MarcoFalke closed D113600: [libc++] Fix segmentation fault in __do_put_integral.
Nov 10 2021, 12:26 PM · Restricted Project
MarcoFalke added a comment to D113600: [libc++] Fix segmentation fault in __do_put_integral.

There are tests, but the issue (segmentation fault) is only observable on 32-bit AFAICT. On other platforms it is "only" UB, but doing the right thing.

Nov 10 2021, 12:15 PM · Restricted Project
MarcoFalke requested review of D113600: [libc++] Fix segmentation fault in __do_put_integral.
Nov 10 2021, 11:51 AM · Restricted Project

May 23 2021

MarcoFalke added a comment to D97572: [libc++] Include <__config_site> from <__config>.

I suspect this broke the OSS-Fuzz integration

May 23 2021, 3:08 AM · Restricted Project, Restricted Project, Unknown Object (Project), Restricted Project

Mar 9 2021

MarcoFalke added a comment to D98057: [compiler-rt] FuzzedDataProvider: Add missing include.

I merged this before review because without the patch we are unable to compile and it should be trivially correct. Feel free to revert if there are any issues with it.

Mar 9 2021, 3:28 AM · Unknown Object (Project)
MarcoFalke committed rG0cccccf0d2cb: [compiler-rt] FuzzedDataProvider: Add missing include (authored by MarcoFalke).
[compiler-rt] FuzzedDataProvider: Add missing include
Mar 9 2021, 3:27 AM
MarcoFalke closed D98057: [compiler-rt] FuzzedDataProvider: Add missing include.
Mar 9 2021, 3:27 AM · Unknown Object (Project)

Mar 5 2021

MarcoFalke added a reviewer for D98057: [compiler-rt] FuzzedDataProvider: Add missing include: Dor1s.
Mar 5 2021, 10:01 AM · Unknown Object (Project)
MarcoFalke updated subscribers of D98057: [compiler-rt] FuzzedDataProvider: Add missing include.

@Dor1s I can commit this myself, but it needs one approval first. Mind taking a look?

Mar 5 2021, 10:01 AM · Unknown Object (Project)
MarcoFalke set the repository for D98057: [compiler-rt] FuzzedDataProvider: Add missing include to rG LLVM Github Monorepo.
Mar 5 2021, 9:59 AM · Unknown Object (Project)
MarcoFalke requested review of D98057: [compiler-rt] FuzzedDataProvider: Add missing include.
Mar 5 2021, 9:57 AM · Unknown Object (Project)

Dec 17 2020

MarcoFalke updated the summary of D93412: [compiler-rt] FuzzedDataProvider: Add PickValueInArray for std::array.
Dec 17 2020, 2:38 AM · Unknown Object (Project)
MarcoFalke added a comment to D93412: [compiler-rt] FuzzedDataProvider: Add PickValueInArray for std::array.

Yes, please. I don't have commit right.

Dec 17 2020, 2:26 AM · Unknown Object (Project)

Dec 16 2020

MarcoFalke added a reviewer for D93412: [compiler-rt] FuzzedDataProvider: Add PickValueInArray for std::array: Dor1s.
Dec 16 2020, 10:15 PM · Unknown Object (Project)
MarcoFalke updated the diff for D93412: [compiler-rt] FuzzedDataProvider: Add PickValueInArray for std::array.

Made unit test pass 😅

Dec 16 2020, 11:25 AM · Unknown Object (Project)
MarcoFalke requested review of D93412: [compiler-rt] FuzzedDataProvider: Add PickValueInArray for std::array.
Dec 16 2020, 11:09 AM · Unknown Object (Project)

Apr 18 2020

MarcoFalke abandoned D78244: compiler-rt: Fix spelling 'chanding' -> 'changing'.
Apr 18 2020, 10:45 AM · Unknown Object (Project)

Apr 15 2020

MarcoFalke created D78244: compiler-rt: Fix spelling 'chanding' -> 'changing'.
Apr 15 2020, 2:55 PM · Unknown Object (Project)

Feb 11 2019

MarcoFalke added a comment to D57528: [tools] Fix python DeprecationWarning: invalid escape sequence.

No, I don't have commit right

Feb 11 2019, 6:57 AM · Restricted Project, Restricted Project

Jan 31 2019

MarcoFalke created D57528: [tools] Fix python DeprecationWarning: invalid escape sequence.
Jan 31 2019, 10:58 AM · Restricted Project, Restricted Project

Jul 29 2018

MarcoFalke added a comment to D48098: clang-format-diff: Make it work with python3 too.

MarcoFalke: do you need someone to submit this for you?

Jul 29 2018, 11:51 AM

Jun 26 2018

MarcoFalke updated the diff for D48098: clang-format-diff: Make it work with python3 too.

Keep python2 by default for now

Jun 26 2018, 3:57 AM

Jun 13 2018

MarcoFalke updated the summary of D48098: clang-format-diff: Make it work with python3 too.
Jun 13 2018, 3:19 PM
MarcoFalke updated the summary of D48098: clang-format-diff: Make it work with python3 too.
Jun 13 2018, 3:11 PM
MarcoFalke updated the diff for D48098: clang-format-diff: Make it work with python3 too.
Jun 13 2018, 3:11 PM
MarcoFalke added a comment to D48098: clang-format-diff: Make it work with python3 too.
why is this switch necessary?
Jun 13 2018, 3:02 PM

Jun 12 2018

MarcoFalke added a reviewer for D48098: clang-format-diff: Make it work with python3 too: djasper.
Jun 12 2018, 2:51 PM
MarcoFalke updated the diff for D48098: clang-format-diff: Make it work with python3 too.

In python2.7:
TypeError: 'encoding' is an invalid keyword argument for this function

Jun 12 2018, 2:46 PM
MarcoFalke created D48098: clang-format-diff: Make it work with python3 too.
Jun 12 2018, 2:40 PM