[PATCH] D127283: [pseudo] Simplify the glrReduce implementation.

2022-06-09 Thread Haojian Wu via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG74e4d5f256fa: [pseudo] Simplify the glrReduce implementation. (authored by hokein). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D127283: [pseudo] Simplify the glrReduce implementation.

2022-06-09 Thread Haojian Wu via Phabricator via cfe-commits
hokein updated this revision to Diff 435463. hokein marked 3 inline comments as done. hokein added a comment. address comments Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D127283/new/ https://reviews.llvm.org/D127283 Files: clang-tools-extra/ps

[PATCH] D127283: [pseudo] Simplify the glrReduce implementation.

2022-06-08 Thread Sam McCall via Phabricator via cfe-commits
sammccall accepted this revision. sammccall added a comment. This revision is now accepted and ready to land. Very nice! Not sure why I didn't see this before :-) Comment at: clang-tools-extra/pseudo/lib/GLR.cpp:266 using Sequence = llvm::SmallVector; - KeyedQueue Sequences

[PATCH] D127283: [pseudo] Simplify the glrReduce implementation.

2022-06-08 Thread Haojian Wu via Phabricator via cfe-commits
hokein created this revision. hokein added a reviewer: sammccall. Herald added a project: All. hokein requested review of this revision. Herald added a subscriber: alextsao1999. Herald added a project: clang-tools-extra. glrReduce maintains two priority queues (one for bases, and the other for Seq