Page MenuHomePhabricator

thegameg (Francis Visoiu Mistrih)
User

Projects

User does not belong to any projects.

User Details

User Since
Nov 23 2015, 4:41 AM (282 w, 1 d)

Recent Activity

Today

thegameg committed rG16a0d80912ed: [CMake][llvm] avoid changing global flags (may be used outside of llvm) (authored by radford).
[CMake][llvm] avoid changing global flags (may be used outside of llvm)
Tue, Apr 20, 4:07 PM
thegameg closed D100900: [CMake][llvm] avoid changing global flags (may be used outside of llvm).
Tue, Apr 20, 4:07 PM · Restricted Project

Mon, Apr 12

thegameg accepted D100168: [GreedyRA ORE] Add debug location for function level report.
Mon, Apr 12, 9:57 AM · Restricted Project
thegameg added a comment to D100016: [GreedyRA ORE] Separate Folder Reloads and Zero Cost Folder Reloads.

@thegameg, I landed NFC before to move from if-else to if-continue. Is it readable now or you still want me to separate added logic into utility function?

Mon, Apr 12, 9:53 AM · Restricted Project

Thu, Apr 8

thegameg added inline comments to D100016: [GreedyRA ORE] Separate Folder Reloads and Zero Cost Folder Reloads.
Thu, Apr 8, 12:14 PM · Restricted Project

Tue, Apr 6

thegameg added a comment to D100017: [GreedyRA ORE] Add stats for copy of virtual registers..

Is this really useful as a user-surfacing remark? Won't it have a really high false positive rate due to coalescing?

To be honest - not sure. But if RA can assigned physical register and can eliminate copy it a good sign.
So probably I also need to check reg map whether physical registers are different or not... Need to check...

Tue, Apr 6, 10:44 PM · Restricted Project
thegameg added a comment to D100017: [GreedyRA ORE] Add stats for copy of virtual registers..

Is this really useful as a user-surfacing remark? Won't it have a really high false positive rate due to coalescing?

Tue, Apr 6, 10:35 PM · Restricted Project
thegameg added inline comments to D100013: [GreedyRA ORE] Extract computeNumberOfSplillsReloads to use in different places. NFC..
Tue, Apr 6, 10:33 PM · Restricted Project
thegameg added inline comments to D100016: [GreedyRA ORE] Separate Folder Reloads and Zero Cost Folder Reloads.
Tue, Apr 6, 10:17 PM · Restricted Project
thegameg accepted D100013: [GreedyRA ORE] Extract computeNumberOfSplillsReloads to use in different places. NFC..
Tue, Apr 6, 9:53 PM · Restricted Project
thegameg accepted D100014: [GreedyRA ORE] Add function level spill/reloads stats.

This looks good too, thanks! Please add a test or extend the ones in llvm/test/CodeGen/AArch64/arm64-spill-remarks.ll.

Tue, Apr 6, 9:51 PM · Restricted Project
thegameg added a comment to D100012: [GreedyRA ORE] Extract stats in RAGreedyStats struct. NFC..

LGTM. Doesn't seem like there are any tests testing this remark, maybe you can add one if you have something handy?

There are some tests
./CodeGen/AArch64/arm64-spill-remarks-treshold-hotness.ll
./CodeGen/AArch64/arm64-spill-remarks.ll

I'll probably add something like this.

Tue, Apr 6, 9:48 PM · Restricted Project
thegameg accepted D100012: [GreedyRA ORE] Extract stats in RAGreedyStats struct. NFC..

LGTM. Doesn't seem like there are any tests testing this remark, maybe you can add one if you have something handy?

Tue, Apr 6, 9:44 PM · Restricted Project

Mar 4 2021

thegameg committed rG365b78396abf: [Remarks] Emit variable info in auto-init remarks (authored by thegameg).
[Remarks] Emit variable info in auto-init remarks
Mar 4 2021, 12:51 PM
thegameg closed D97734: [Remarks] Emit variable info in auto-init remarks.
Mar 4 2021, 12:51 PM · Restricted Project

Mar 2 2021

thegameg added inline comments to D97734: [Remarks] Emit variable info in auto-init remarks.
Mar 2 2021, 11:24 AM · Restricted Project
thegameg updated the diff for D97734: [Remarks] Emit variable info in auto-init remarks.

Address comments + comments in header.

Mar 2 2021, 11:24 AM · Restricted Project

Mar 1 2021

thegameg requested review of D97734: [Remarks] Emit variable info in auto-init remarks.
Mar 1 2021, 2:01 PM · Restricted Project

Feb 25 2021

thegameg committed rGfee9abe69c0e: [Remarks] Provide more information about auto-init calls (authored by thegameg).
[Remarks] Provide more information about auto-init calls
Feb 25 2021, 3:15 PM
thegameg committed rG4753a69a316b: [Remarks] Provide more information about auto-init stores (authored by thegameg).
[Remarks] Provide more information about auto-init stores
Feb 25 2021, 3:15 PM
thegameg committed rGc49b600b2f82: [Remarks] Emit remarks for "auto-init" !annotations (authored by thegameg).
[Remarks] Emit remarks for "auto-init" !annotations
Feb 25 2021, 3:15 PM
thegameg closed D97489: [Remarks] Provide more information about auto-init calls.
Feb 25 2021, 3:14 PM · Restricted Project
thegameg closed D97412: [Remarks] Provide more information about auto-init stores.
Feb 25 2021, 3:14 PM · Restricted Project
thegameg closed D97405: [Remarks] Emit remarks for "auto-init" !annotations.
Feb 25 2021, 3:14 PM · Restricted Project
thegameg accepted D97499: [PM] Show the pass argument in pre/post-pass IR dumps.

Thank you!

Feb 25 2021, 1:59 PM · Restricted Project, Restricted Project
thegameg requested review of D97489: [Remarks] Provide more information about auto-init calls.
Feb 25 2021, 11:06 AM · Restricted Project

Feb 24 2021

thegameg added inline comments to D97412: [Remarks] Provide more information about auto-init stores.
Feb 24 2021, 4:00 PM · Restricted Project
thegameg updated the diff for D97412: [Remarks] Provide more information about auto-init stores.

Add StoreVolatile: false and StoreAtomic: false under ExtraArgs.

Feb 24 2021, 4:00 PM · Restricted Project
thegameg added inline comments to D97412: [Remarks] Provide more information about auto-init stores.
Feb 24 2021, 3:50 PM · Restricted Project
thegameg requested review of D97412: [Remarks] Provide more information about auto-init stores.
Feb 24 2021, 1:27 PM · Restricted Project
thegameg requested review of D97405: [Remarks] Emit remarks for "auto-init" !annotations.
Feb 24 2021, 11:40 AM · Restricted Project

Feb 4 2021

thegameg accepted D96036: [clang][codegen] Remember string used to create llvm::Regex for optimization remarks.
Feb 4 2021, 7:29 AM · Restricted Project

Jan 22 2021

thegameg accepted D95280: [YAML I/O] Fix bug in emission of empty sequence.

LGTM. Not sure if there's a cleaner way.

Jan 22 2021, 6:03 PM · Restricted Project
thegameg committed rG0cc38acfc4e1: [Matrix] Propagate shape information through fneg (authored by thegameg).
[Matrix] Propagate shape information through fneg
Jan 22 2021, 2:35 PM
thegameg closed D95252: [Matrix] Propagate shape information through fneg.
Jan 22 2021, 2:35 PM · Restricted Project
thegameg added a reviewer for D95252: [Matrix] Propagate shape information through fneg: anemet.
Jan 22 2021, 11:22 AM · Restricted Project
thegameg requested review of D95252: [Matrix] Propagate shape information through fneg.
Jan 22 2021, 11:21 AM · Restricted Project

Jan 8 2021

thegameg added inline comments to D94333: [Inliner] Change inline remark format and update ReplayInlineAdvisor to use it.
Jan 8 2021, 12:43 PM · Restricted Project, Restricted Project

Dec 16 2020

thegameg accepted D91444: [InstCombine] Preserve !annotation on newly created instructions..

LGTM, nice!

Dec 16 2020, 11:06 AM · Restricted Project, Restricted Project
thegameg accepted D93399: [InstCombine] Preserve !annotation for newly created instructions..

LGTM

Dec 16 2020, 11:02 AM · Restricted Project

Dec 4 2020

thegameg committed rG3d381a710220: [llvm-nm][MachO] Don't call getFlags on redacted symbols (authored by thegameg).
[llvm-nm][MachO] Don't call getFlags on redacted symbols
Dec 4 2020, 10:06 PM

Nov 13 2020

thegameg accepted D91195: Add Annotation2MD pass to add !annotate metadata from llvm.global.annotations.

LGTM

Nov 13 2020, 3:50 PM · Restricted Project, Restricted Project
thegameg added inline comments to D91444: [InstCombine] Preserve !annotation on newly created instructions..
Nov 13 2020, 3:47 PM · Restricted Project, Restricted Project
thegameg accepted D91417: [IRGen] Add !annotation metadata for auto-init stores..

Looks good, thanks!

Nov 13 2020, 3:44 PM · Restricted Project, Restricted Project

Nov 10 2020

thegameg accepted D91188: Add !annotation metadata and remarks pass..

Looks great, thanks!

Nov 10 2020, 3:01 PM · Restricted Project

Nov 6 2020

thegameg added inline comments to D89240: Add initial infra to convert annotations to MD and display summary remarks (WIP).
Nov 6 2020, 12:07 PM · Restricted Project
thegameg added inline comments to D89240: Add initial infra to convert annotations to MD and display summary remarks (WIP).
Nov 6 2020, 12:02 PM · Restricted Project

Oct 26 2020

thegameg added inline comments to D90201: Try reading a smaller chunk when (p)read fails.
Oct 26 2020, 8:13 PM · Restricted Project
thegameg accepted D90201: Try reading a smaller chunk when (p)read fails.
Oct 26 2020, 8:06 PM · Restricted Project

Oct 23 2020

thegameg added a comment to D89892: [AsmPrinter] Add per BB instruction mix remark..

How about adding a total count as well? That should help quickly eye ball the ratios.

Oct 23 2020, 12:09 AM · Restricted Project

Oct 22 2020

thegameg accepted D89892: [AsmPrinter] Add per BB instruction mix remark..

This looks great! Thanks!

Oct 22 2020, 8:25 PM · Restricted Project

Oct 16 2020

thegameg added a comment to rG0345d88de654: [NFC][ScheduleDAG] Remove unused EntrySU SUnit.

Is this really necessary to remove? For example, our backend is using EntrySU edges for top edges from the basic block entry, in order to account for clearances. It seems to make sense to have a node like this.

Oct 16 2020, 9:42 AM

Sep 21 2020

thegameg added a comment to D87867: [NFC][ScheduleDAG] Remove unused EntrySU SUnit.

I am totally fine with keeping this in-tree, even unused, if it helps several out-of-tree backends. Thanks for speaking up.

Sep 21 2020, 12:25 PM · Restricted Project

Sep 18 2020

thegameg accepted D85626: [tools][remarks-shlib] Don't build libRemarks.so without PIC.

LGTM

Sep 18 2020, 3:06 PM · Restricted Project
thegameg committed rG0345d88de654: [NFC][ScheduleDAG] Remove unused EntrySU SUnit (authored by thegameg).
[NFC][ScheduleDAG] Remove unused EntrySU SUnit
Sep 18 2020, 9:52 AM
thegameg closed D87867: [NFC][ScheduleDAG] Remove unused EntrySU SUnit.
Sep 18 2020, 9:52 AM · Restricted Project

Sep 17 2020

thegameg abandoned D53490: [MIR] Provide a default alignment for stack objects.
Sep 17 2020, 4:31 PM
thegameg requested changes to D85626: [tools][remarks-shlib] Don't build libRemarks.so without PIC.

Can you wrap the condition around the whole file? The rest is not useful without the library anyway.

Sep 17 2020, 4:29 PM · Restricted Project
thegameg requested review of D87867: [NFC][ScheduleDAG] Remove unused EntrySU SUnit.
Sep 17 2020, 3:55 PM · Restricted Project
thegameg abandoned D34544: [libunwind] Don't assume the return address register is always saved and has CFI for it.

Won't be back to this anytime soon.

Sep 17 2020, 3:18 PM
thegameg accepted D85810: [clang] Pass-through remarks options to linker.

LGTM

Sep 17 2020, 2:58 PM · Restricted Project

Aug 28 2020

thegameg accepted D86797: [early-ifcvt] Add OptRemarks.

This looks great, thanks! Since it's a CodeGen pass, I would suggest using a MIR test instead of an IR one, but I think it's probably easier to read it as it is right now.

Aug 28 2020, 11:55 AM · Restricted Project

Aug 14 2020

thegameg added a comment to D85809: [Remarks][1/2] Expand remarks hotness threshold option support in more tools.

Just to clarify, I wasn't suggesting putting it in another patch, but just bringing it back here (plus adding in the gold-plugin support). The other patch I was referring to was the follow on patch D85810 where I also suggested adding support to gold-plugin.cpp.

Aug 14 2020, 11:49 AM · Restricted Project
thegameg added a comment to D85809: [Remarks][1/2] Expand remarks hotness threshold option support in more tools.

I have removed the change to llvm-lto

Aug 14 2020, 11:41 AM · Restricted Project

Aug 12 2020

thegameg accepted D85809: [Remarks][1/2] Expand remarks hotness threshold option support in more tools.

LGTM, thanks for plugging all these options through!

Aug 12 2020, 3:55 PM · Restricted Project

Jul 29 2020

thegameg added a reverting change for rGd8e8e32d85f1: build: always add support for assembly targets on Darwin: rG926c14798d31: Revert "build: always add support for assembly targets on Darwin".
Jul 29 2020, 3:45 PM
thegameg committed rG926c14798d31: Revert "build: always add support for assembly targets on Darwin" (authored by thegameg).
Revert "build: always add support for assembly targets on Darwin"
Jul 29 2020, 3:45 PM
thegameg added a reverting change for D84333: build: always add support for assembly targets on Darwin: rG926c14798d31: Revert "build: always add support for assembly targets on Darwin".
Jul 29 2020, 3:45 PM · Restricted Project
thegameg added a comment to D84333: build: always add support for assembly targets on Darwin.

Hi @compnerd, it looks like this fails to link: http://green.lab.llvm.org/green/job/clang-stage1-RA/13171/console

Jul 29 2020, 3:30 PM · Restricted Project
thegameg added a comment to D84690: Make iterators of BitVector easier to use by inheriting iterator_facade_base.

Indeed, sorry that fell through & I didn't understand enough about iterators to instead suggest falling back to an input iterator instead of figuring out how to fit the semantics of the forward iterator.

Jul 29 2020, 2:06 PM · Restricted Project

Jul 27 2020

thegameg added a comment to D84690: Make iterators of BitVector easier to use by inheriting iterator_facade_base.

I had an attempt here: https://reviews.llvm.org/D33419, I think similar concerns came up in that review.

Jul 27 2020, 2:12 PM · Restricted Project

Jul 21 2020

thegameg added a comment to D84207: [compiler-rt][test][profile] Fix missing include.

Not sure why our public bots didn't catch it.

Jul 21 2020, 11:39 AM · Restricted Project

Jul 20 2020

thegameg added a comment to D81645: MIR Statepoint refactoring. Part 1: Basic MI level changes..

Hi, I see CodeGen/X86/statepoint-vreg.mir failing with UBSAN enabled on our internal bots:

Should be fixed already.
Does your sources include e04fe9aefd4?

Jul 20 2020, 9:08 AM · Restricted Project

Jul 17 2020

thegameg added a comment to D81645: MIR Statepoint refactoring. Part 1: Basic MI level changes..

Hi, I see CodeGen/X86/statepoint-vreg.mir failing with UBSAN enabled on our internal bots:

Jul 17 2020, 6:03 PM · Restricted Project

Jul 14 2020

thegameg committed rG2cffd7c40ea5: [Driver] Only pass LTO remark arguments if the driver asks for it (authored by thegameg).
[Driver] Only pass LTO remark arguments if the driver asks for it
Jul 14 2020, 4:39 PM
thegameg committed rG684a9d17ba95: [Driver] Handle all optimization-record options for Darwin LTO (authored by thegameg).
[Driver] Handle all optimization-record options for Darwin LTO
Jul 14 2020, 4:39 PM
thegameg committed rG8769525acbe2: [MachO] Add cpu(sub)type tests and improve error handling (authored by thegameg).
[MachO] Add cpu(sub)type tests and improve error handling
Jul 14 2020, 4:30 PM
thegameg committed rGb772ba713bc2: [LTO][Legacy] Add new API to query Mach-O CPU (sub)type (authored by thegameg).
[LTO][Legacy] Add new API to query Mach-O CPU (sub)type
Jul 14 2020, 4:30 PM
thegameg committed rG42d75cc662ec: [Target] Remove libObject dependency in lib/Target (authored by thegameg).
[Target] Remove libObject dependency in lib/Target
Jul 14 2020, 4:30 PM
thegameg committed rGf8a4e3f0ae8f: [macho][NFC] Extract all CPU_(SUB_)TYPE logic to BinaryFormat (authored by thegameg).
[macho][NFC] Extract all CPU_(SUB_)TYPE logic to BinaryFormat
Jul 14 2020, 4:30 PM
thegameg committed rG5b8fc9c3fec2: [AArch64] Delete an unneeded dependency on Object after… (authored by MaskRay).
[AArch64] Delete an unneeded dependency on Object after…
Jul 14 2020, 4:30 PM
thegameg committed rG7af3216fdaa8: [AArch64][SVE] Fix -DBUILD_SHARED_LIBS=on builds after… (authored by MaskRay).
[AArch64][SVE] Fix -DBUILD_SHARED_LIBS=on builds after…
Jul 14 2020, 4:30 PM
thegameg committed rG7737edf28d39: [llvm][build] Fix shared lib builds. [NFC] (authored by fpetrogalli).
[llvm][build] Fix shared lib builds. [NFC]
Jul 14 2020, 4:30 PM
thegameg committed rG2c21cfcb35a0: [Remarks] Fix gcc build (authored by thegameg).
[Remarks] Fix gcc build
Jul 14 2020, 4:20 PM
thegameg committed rG1fee4c40a623: Fix warning on trailing `;`. NFC. (authored by hliao).
Fix warning on trailing `;`. NFC.
Jul 14 2020, 4:20 PM
thegameg committed rGe16ee7d4f600: [Remarks] Extend the RemarkStreamer to support other emitters (authored by thegameg).
[Remarks] Extend the RemarkStreamer to support other emitters
Jul 14 2020, 4:20 PM
thegameg committed rG805a8329c87a: [AArch64] Provide Darwin variants of most calling conventions (authored by thegameg).
[AArch64] Provide Darwin variants of most calling conventions
Jul 14 2020, 4:19 PM
Visoiu Mistrih Francis <890283+francisvm@users.noreply.github.com> committed rGdb1b0e41b0cc: Revert " Renable r314928" (authored by thegameg).
Revert " Renable r314928"
Jul 14 2020, 4:18 PM
thegameg added a reverting change for rG4cdc9dab0ac9: Renable r314928: rGdb1b0e41b0cc: Revert " Renable r314928".
Jul 14 2020, 4:18 PM
Visoiu Mistrih Francis <890283+francisvm@users.noreply.github.com> committed rGbcf51f094d48: Revert "[InstCombine] Don't use getFirstNonPHI in FoldIntegerTypedPHI" (authored by thegameg).
Revert "[InstCombine] Don't use getFirstNonPHI in FoldIntegerTypedPHI"
Jul 14 2020, 4:18 PM
thegameg added a reverting change for rGa4c76be50689: [InstCombine] Don't use getFirstNonPHI in FoldIntegerTypedPHI: rGbcf51f094d48: Revert "[InstCombine] Don't use getFirstNonPHI in FoldIntegerTypedPHI".
Jul 14 2020, 4:18 PM
thegameg committed rGfc9cafb98660: [CodeGen] Attach no-builtin attributes to function definitions with no Decl (authored by thegameg).
[CodeGen] Attach no-builtin attributes to function definitions with no Decl
Jul 14 2020, 4:18 PM

Jul 9 2020

thegameg accepted D83456: [NFC][AArch64] Refactor getArgumentPopSize.

LGTM, thanks!

Jul 9 2020, 10:13 AM · Restricted Project

Jul 3 2020

thegameg committed rGaa5ec34e3126: [LoopDeletion] Emit a remark when a dead loop is deleted (authored by thegameg).
[LoopDeletion] Emit a remark when a dead loop is deleted
Jul 3 2020, 3:48 PM
thegameg closed D83113: [LoopDeletion] Emit a remark when a dead loop is deleted.
Jul 3 2020, 3:46 PM · Restricted Project
thegameg added inline comments to D83113: [LoopDeletion] Emit a remark when a dead loop is deleted.
Jul 3 2020, 3:39 PM · Restricted Project
thegameg updated the diff for D83113: [LoopDeletion] Emit a remark when a dead loop is deleted.
Jul 3 2020, 3:39 PM · Restricted Project

Jul 2 2020

thegameg created D83113: [LoopDeletion] Emit a remark when a dead loop is deleted.
Jul 2 2020, 10:39 PM · Restricted Project

Jul 1 2020

thegameg added inline comments to D82763: MIR: Infer not-SSA for subregister defs.
Jul 1 2020, 8:09 AM · Restricted Project

Jun 24 2020

thegameg accepted D82478: [MIR] Fix CFI_INSTRUCTION escape printing.
Jun 24 2020, 1:34 PM · Restricted Project