miloserd (Vladimir Miloserdov)
Compiler Intern at Synopsys

Projects

User does not belong to any projects.

User Details

User Since
Sep 21 2017, 7:49 AM (52 w, 1 d)

Recent Activity

Nov 15 2017

miloserd added a comment to D38367: [SLP] Added more missed optimiazation remarks.

Just finished testing, seems OK with latest submitted version

Nov 15 2017, 5:35 AM
miloserd added a comment to D38367: [SLP] Added more missed optimiazation remarks.

Sorry, I forgot to test it with all of the backends. Now changed failing tests as they did not support many remarks at once.
Though I haven't checked all the tests yet since I am still waiting for the local build to finish.

Nov 15 2017, 3:33 AM
miloserd updated the diff for D38367: [SLP] Added more missed optimiazation remarks.

Fixed tests

Nov 15 2017, 3:28 AM

Nov 14 2017

miloserd updated the diff for D38367: [SLP] Added more missed optimiazation remarks.

Rebased

Nov 14 2017, 2:57 PM
miloserd added a comment to D38367: [SLP] Added more missed optimiazation remarks.

Thanks for your time, anemet. I've just fixed these issues you told.
Can you please commit this patch? I haven't got r+w access.

Nov 14 2017, 10:49 AM
miloserd updated the diff for D38367: [SLP] Added more missed optimiazation remarks.

Fixed minor issues

Nov 14 2017, 10:47 AM

Nov 13 2017

miloserd added a comment to D38367: [SLP] Added more missed optimiazation remarks.

ping

Nov 13 2017, 6:04 AM

Nov 3 2017

miloserd updated the diff for D38367: [SLP] Added more missed optimiazation remarks.

Moved the code of one remark under the closure

Nov 3 2017, 12:36 PM
miloserd added a comment to D38367: [SLP] Added more missed optimiazation remarks.

Also by any chance, did you run this on some real code base? Some of these may trigger quite a bit and I want to make sure they are not on the top of the list. You can use opt-viewer/opt-stats.py to get a sense how frequently your remark is generated.

Nov 3 2017, 12:22 PM
miloserd updated the diff for D38367: [SLP] Added more missed optimiazation remarks.

Reworked with closure API

Nov 3 2017, 9:20 AM
miloserd added inline comments to D38367: [SLP] Added more missed optimiazation remarks.
Nov 3 2017, 3:06 AM
miloserd updated the diff for D38367: [SLP] Added more missed optimiazation remarks.

Fixed code style, reworked tests to check YAML output

Nov 3 2017, 3:03 AM

Sep 28 2017

miloserd abandoned D38345: [SLP] Added more missed optimiazation remarks.
Sep 28 2017, 9:17 AM
miloserd added a comment to D38345: [SLP] Added more missed optimiazation remarks.

Thanks for your patch! Please include the full context for the diff and add llvm-commits as subscriber; that should be done when creating the review, otherwise the mailing list does not get the complete picture. See http://llvm.org/docs/Phabricator.html for more details.

Please start a new review cc'ing llvm-commit (from the get-go).

Sep 28 2017, 9:16 AM
miloserd created D38367: [SLP] Added more missed optimiazation remarks.
Sep 28 2017, 9:16 AM
miloserd added a reviewer for D38345: [SLP] Added more missed optimiazation remarks: llvm-commits.
Sep 28 2017, 9:04 AM
miloserd removed a reviewer for D38345: [SLP] Added more missed optimiazation remarks: llvm-commits.
Sep 28 2017, 9:04 AM
miloserd updated subscribers of D38345: [SLP] Added more missed optimiazation remarks.
Sep 28 2017, 9:04 AM
miloserd updated the diff for D38345: [SLP] Added more missed optimiazation remarks.

Fixed code style, reworked tests completely. Uploaded diffs with context now.

Sep 28 2017, 9:00 AM
miloserd created D38345: [SLP] Added more missed optimiazation remarks.
Sep 28 2017, 12:50 AM