fedor.sergeev (Fedor Sergeev)
User

Projects

User does not belong to any projects.

User Details

User Since
Feb 10 2017, 1:36 AM (79 w, 1 d)

Recent Activity

Yesterday

fedor.sergeev added a dependent revision for D47858: [New PM] Introducing PassInstrumentation framework: D50923: [New PM][PassInstrumentation] IR printing support for New Pass Manager.
Fri, Aug 17, 1:57 PM
fedor.sergeev added a dependency for D50923: [New PM][PassInstrumentation] IR printing support for New Pass Manager: D47858: [New PM] Introducing PassInstrumentation framework.
Fri, Aug 17, 1:57 PM
fedor.sergeev created D50923: [New PM][PassInstrumentation] IR printing support for New Pass Manager.
Fri, Aug 17, 1:57 PM
fedor.sergeev updated the diff for D47858: [New PM] Introducing PassInstrumentation framework.

clang-formatted

Fri, Aug 17, 1:55 PM
fedor.sergeev retitled D47858: [New PM] Introducing PassInstrumentation framework from WIP: Introducing PassInstrumentation to [New PM] Introducing PassInstrumentation framework.
Fri, Aug 17, 1:52 PM
fedor.sergeev updated the diff for D47858: [New PM] Introducing PassInstrumentation framework.

NB: this is still at early stages, in particular PassExecutionCounter is just a stub for now.
Yet I need a review for people to tell me if I'm completely off the track! :)

Fri, Aug 17, 1:50 PM
fedor.sergeev added inline comments to D50888: [NFC][LICM] Remove too conservative variable.
Fri, Aug 17, 6:04 AM
fedor.sergeev added inline comments to D50891: [LICM] Hoist guards from non-header blocks.
Fri, Aug 17, 5:52 AM
fedor.sergeev accepted D50838: [NFC] Remove function isGuaranteedToExecute.

LGTM

Fri, Aug 17, 5:03 AM

Tue, Aug 14

fedor.sergeev committed rL339682: [Inliner] add inliner stats to new pm version of inliner.
[Inliner] add inliner stats to new pm version of inliner
Tue, Aug 14, 8:20 AM

Mon, Aug 13

fedor.sergeev added inline comments to D50426: [NFC][MustExecute] Rework API to start making better analysis, part 2.
Mon, Aug 13, 5:42 AM

Sun, Aug 12

fedor.sergeev added a comment to D50377: [MustExecute] Rework LoopSafetyInfo to make it more optimistic about throws.

I'm really worried about LoopSafetyInfo getting more nontrivial state
(Philip already raised a question in D50426 about CurLoop potentially becoming invalid, and here we get even more with addition of ICF).

Sun, Aug 12, 1:21 PM
fedor.sergeev added inline comments to D50426: [NFC][MustExecute] Rework API to start making better analysis, part 2.
Sun, Aug 12, 1:07 PM
fedor.sergeev added inline comments to D50426: [NFC][MustExecute] Rework API to start making better analysis, part 2.
Sun, Aug 12, 12:47 PM

Fri, Aug 3

fedor.sergeev added a comment to D50247: [GuardWidening] Ignore guards with trivial conditions.

LGTM bar some minor stylistic stuff.

Fri, Aug 3, 5:52 AM
fedor.sergeev accepted D50247: [GuardWidening] Ignore guards with trivial conditions.
Fri, Aug 3, 5:52 AM

Thu, Aug 2

fedor.sergeev added inline comments to D49974: [GuardWidening] Widen guards with conditions of frequently taken dominated branches.
Thu, Aug 2, 11:16 PM

Tue, Jul 31

fedor.sergeev reopened D40425: Extending CFGPrinter and CallPrinter with Heat Colors.

Unfortunately I had to revert this (https://reviews.llvm.org/rL336000) since there were several build-bot breaks.

Shouldnt this be reopened?
Any plans to reintegrate this back - it seems to provide a very useful functionality...

Tue, Jul 31, 12:43 PM

Mon, Jul 30

fedor.sergeev added inline comments to D49974: [GuardWidening] Widen guards with conditions of frequently taken dominated branches.
Mon, Jul 30, 1:32 AM

Jul 4 2018

fedor.sergeev added a comment to D48851: [PM/LoopUnswitch] Fix the first and immediate components of PR37889..

Hmm... downstream testing found an issue.
Here is a reduced testcase (https://reviews.llvm.org/file/data/b4nvnqme23unyxwktst4/PHID-FILE-wny66mgcso2cbh4ciz3x/reduced-unswitch.ll)

Jul 4 2018, 8:52 AM

Jul 3 2018

fedor.sergeev added a comment to D48851: [PM/LoopUnswitch] Fix the first and immediate components of PR37889..

Btw, I'm running downstream testing with this set of changes, but since I did not see anything like 37889 before I dont think its gonna find anything here as well.

Jul 3 2018, 2:47 PM
fedor.sergeev added a comment to D48851: [PM/LoopUnswitch] Fix the first and immediate components of PR37889..

Ok, makes sense.

Jul 3 2018, 2:42 PM
fedor.sergeev accepted D48851: [PM/LoopUnswitch] Fix the first and immediate components of PR37889..

LGTM for the code.

Jul 3 2018, 2:05 PM

Jun 24 2018

fedor.sergeev added a comment to D36463: [Sparc] allow tls_add/tls_call syntax in assembler parser.

What happened with this? As far as I know this still needs to be fixed.

As per my latest comment there was an assert when I ran pre-integration testing.
I dont have any immediate plans to continue investigation on that, so if you can take it over from me - please, do.
Functionally this patch was heavily tested as part of our local clang-on-sparc-Solaris installation back at the time when I was part of Oracle Developer Studio project.

Jun 24 2018, 12:22 PM

Jun 8 2018

fedor.sergeev added a comment to D47683: [PM/LoopUnswitch] Teach the new unswitch to handle nontrivial unswitching of switches..

Not sure where to put this into... but I ran an experiment by adding -passes=loop(unswitch) to the LoopUnswitch tests and here is the result
with the build of all the three dependent fixes:
Failing Tests (11):

LLVM :: Transforms/LoopUnswitch/2011-11-18-TwoSwitches.ll
LLVM :: Transforms/LoopUnswitch/basictest.ll
LLVM :: Transforms/LoopUnswitch/copy-metadata.ll
LLVM :: Transforms/LoopUnswitch/elseif-non-exponential-behavior.ll
LLVM :: Transforms/LoopUnswitch/guards.ll
LLVM :: Transforms/LoopUnswitch/infinite-loop.ll
LLVM :: Transforms/LoopUnswitch/msan.ll
LLVM :: Transforms/LoopUnswitch/simplify-with-nonvalness.ll
LLVM :: Transforms/LoopUnswitch/trivial-unswitch.ll
LLVM :: Transforms/LoopUnswitch/unswitch-equality-undef.ll
LLVM :: Transforms/LoopUnswitch/unswitch-select.ll
Jun 8 2018, 9:05 AM

Jun 6 2018

fedor.sergeev updated the diff for D47858: [New PM] Introducing PassInstrumentation framework.

adding IRUnitT

Jun 6 2018, 6:12 PM
fedor.sergeev created D47858: [New PM] Introducing PassInstrumentation framework.
Jun 6 2018, 4:56 PM

Jun 1 2018

fedor.sergeev added inline comments to D47408: [PM/LoopUnswitch] When using the new SimpleLoopUnswitch pass, schedule loop-cleanup passes at the beginning of the loop pass pipeline, and re-enqueue loops after even trivial unswitching..
Jun 1 2018, 9:32 AM

May 29 2018

fedor.sergeev added inline comments to D47408: [PM/LoopUnswitch] When using the new SimpleLoopUnswitch pass, schedule loop-cleanup passes at the beginning of the loop pass pipeline, and re-enqueue loops after even trivial unswitching..
May 29 2018, 3:07 PM
fedor.sergeev accepted D47408: [PM/LoopUnswitch] When using the new SimpleLoopUnswitch pass, schedule loop-cleanup passes at the beginning of the loop pass pipeline, and re-enqueue loops after even trivial unswitching..
May 29 2018, 3:07 PM

May 28 2018

fedor.sergeev added a comment to D46706: [PM/LoopUnswitch] Support partial trivial unswitching..

Not exactly related to this set of changes...
I just have discovered that old LoopUnswitch performs SCEV cache invalidation when it does nontrivial updates to CFG
(forgetLoop in LoopUnswitch::unswitchNontrivialCondition) .

May 28 2018, 12:59 AM
fedor.sergeev added a comment to D47408: [PM/LoopUnswitch] When using the new SimpleLoopUnswitch pass, schedule loop-cleanup passes at the beginning of the loop pass pipeline, and re-enqueue loops after even trivial unswitching..

Looks ok to me.

May 28 2018, 12:13 AM

May 24 2018

fedor.sergeev added a comment to D46706: [PM/LoopUnswitch] Support partial trivial unswitching..

The pipeline will now be ... truly dynamic.

Hoorray! :)

May 24 2018, 10:08 PM
fedor.sergeev added inline comments to D46706: [PM/LoopUnswitch] Support partial trivial unswitching..
May 24 2018, 3:45 PM

May 22 2018

fedor.sergeev added a comment to D47056: Removed analysis flag from SafepointIRVerifier pass.

Can you, please, describe the reason for this change?

May 22 2018, 10:39 PM

May 11 2018

fedor.sergeev accepted D41241: [Solaris] Only define _REENTRANT if -pthread.

LGTM

May 11 2018, 3:30 PM

May 10 2018

fedor.sergeev added a comment to D46706: [PM/LoopUnswitch] Support partial trivial unswitching..

What about finally renaming SimpleLoopUnswitch into something "less simple"? :)

May 10 2018, 12:35 PM

Apr 24 2018

fedor.sergeev accepted D45943: [PM/LoopUnswitch] Begin teaching SimpleLoopUnswitch to use the new update API for dominators rather than doing manual, hacky updates..

Looks good to me.
Thanks for the effort.
This version finally passes all the tests on our internal pipeline!

Apr 24 2018, 2:07 PM

Apr 19 2018

fedor.sergeev added a comment to D45754: [PM/LoopUnswitch] Detect irreducible control flow within loops and skip unswitching non-trivial edges..

Ahem... sorry for checking it only now, but this does not seem to address a testcase from PR36379, so it must be a different issue.
I will put this to more testing to see if it improves anything on our workflow.

Apr 19 2018, 9:41 PM

Apr 5 2018

fedor.sergeev committed rL329267: allow custom OptBisect classes set to LLVMContext.
allow custom OptBisect classes set to LLVMContext
Apr 5 2018, 3:32 AM
fedor.sergeev closed D44464: allow custom OptBisect classes set to LLVMContext.
Apr 5 2018, 3:32 AM

Apr 4 2018

fedor.sergeev added a comment to D44464: allow custom OptBisect classes set to LLVMContext.

a few nits, LGTM as soon as those are corrected.

Apr 4 2018, 2:48 AM

Apr 3 2018

fedor.sergeev added inline comments to D44464: allow custom OptBisect classes set to LLVMContext.
Apr 3 2018, 1:27 PM

Apr 2 2018

fedor.sergeev accepted D44464: allow custom OptBisect classes set to LLVMContext.

LGTM, but, please, wait for Andrew to ack this.

Apr 2 2018, 11:20 PM

Mar 30 2018

fedor.sergeev added inline comments to D44464: allow custom OptBisect classes set to LLVMContext.
Mar 30 2018, 4:00 AM

Mar 27 2018

fedor.sergeev committed rL328637: [NFC] OptPassGate extracted from OptBisect.
[NFC] OptPassGate extracted from OptBisect
Mar 27 2018, 10:00 AM
fedor.sergeev closed D44821: [NFC] OptPassGate extracted from OptBisect.
Mar 27 2018, 10:00 AM

Mar 26 2018

fedor.sergeev accepted D44821: [NFC] OptPassGate extracted from OptBisect.

LGTM. Though, please, wait for Andrew's feedback.

Mar 26 2018, 7:19 AM

Mar 23 2018

fedor.sergeev committed rL328377: [PM][FunctionAttrs] add NoUnwind attribute inference to PostOrderFunctionAttrs….
[PM][FunctionAttrs] add NoUnwind attribute inference to PostOrderFunctionAttrs…
Mar 23 2018, 2:49 PM
fedor.sergeev closed D44415: [PM][FunctionAttrs] add NoUnwind attribute inference to PostOrderFunctionAttrs pass.
Mar 23 2018, 2:49 PM
fedor.sergeev updated the diff for D44415: [PM][FunctionAttrs] add NoUnwind attribute inference to PostOrderFunctionAttrs pass.

minor semantical update - calculation of return value corrected for AttributeInferer::run,
returning true only when real changes are detected.
Fixes regression introduced by most recent algorithmic update.

Mar 23 2018, 2:47 PM
fedor.sergeev added a comment to D44415: [PM][FunctionAttrs] add NoUnwind attribute inference to PostOrderFunctionAttrs pass.

Thanks for following up till the very end of it! :)
Your feeling of language is a bit over my capabilities, but I do like to learn here.

Mar 23 2018, 1:10 PM
fedor.sergeev added inline comments to D44821: [NFC] OptPassGate extracted from OptBisect.
Mar 23 2018, 12:02 PM
fedor.sergeev added inline comments to D44415: [PM][FunctionAttrs] add NoUnwind attribute inference to PostOrderFunctionAttrs pass.
Mar 23 2018, 11:19 AM
fedor.sergeev updated the diff for D44415: [PM][FunctionAttrs] add NoUnwind attribute inference to PostOrderFunctionAttrs pass.

minor comments cleanup

Mar 23 2018, 11:15 AM
fedor.sergeev accepted D41240: [Solaris] __float128 is supported on Solaris/x86.

LGTM.

Mar 23 2018, 9:57 AM
fedor.sergeev added inline comments to D44821: [NFC] OptPassGate extracted from OptBisect.
Mar 23 2018, 9:51 AM

Mar 22 2018

fedor.sergeev added inline comments to D44415: [PM][FunctionAttrs] add NoUnwind attribute inference to PostOrderFunctionAttrs pass.
Mar 22 2018, 3:55 PM
fedor.sergeev updated the diff for D44415: [PM][FunctionAttrs] add NoUnwind attribute inference to PostOrderFunctionAttrs pass.

more comments cleanup

Mar 22 2018, 3:52 PM
fedor.sergeev updated the diff for D44415: [PM][FunctionAttrs] add NoUnwind attribute inference to PostOrderFunctionAttrs pass.

addressing Justin's comments

Mar 22 2018, 3:46 PM
fedor.sergeev added inline comments to D44415: [PM][FunctionAttrs] add NoUnwind attribute inference to PostOrderFunctionAttrs pass.
Mar 22 2018, 4:08 AM
fedor.sergeev updated the diff for D44415: [PM][FunctionAttrs] add NoUnwind attribute inference to PostOrderFunctionAttrs pass.
trying with erase_if
Mar 22 2018, 4:06 AM

Mar 21 2018

fedor.sergeev updated the diff for D44415: [PM][FunctionAttrs] add NoUnwind attribute inference to PostOrderFunctionAttrs pass.

minor cleanup first

Mar 21 2018, 2:53 PM
fedor.sergeev added inline comments to D44415: [PM][FunctionAttrs] add NoUnwind attribute inference to PostOrderFunctionAttrs pass.
Mar 21 2018, 2:38 PM
fedor.sergeev added inline comments to D44415: [PM][FunctionAttrs] add NoUnwind attribute inference to PostOrderFunctionAttrs pass.
Mar 21 2018, 2:16 PM
fedor.sergeev added inline comments to D44415: [PM][FunctionAttrs] add NoUnwind attribute inference to PostOrderFunctionAttrs pass.
Mar 21 2018, 2:14 PM
fedor.sergeev added a comment to D44415: [PM][FunctionAttrs] add NoUnwind attribute inference to PostOrderFunctionAttrs pass.

just to clarify - I believe this is ready to go :)

Mar 21 2018, 5:02 AM

Mar 20 2018

fedor.sergeev updated the summary of D44415: [PM][FunctionAttrs] add NoUnwind attribute inference to PostOrderFunctionAttrs pass.
Mar 20 2018, 3:49 AM
fedor.sergeev updated the summary of D44415: [PM][FunctionAttrs] add NoUnwind attribute inference to PostOrderFunctionAttrs pass.
Mar 20 2018, 3:46 AM
fedor.sergeev updated the diff for D44415: [PM][FunctionAttrs] add NoUnwind attribute inference to PostOrderFunctionAttrs pass.

making InstrBreaksNonThrowing a bit more readable, introducing -disable-nounwind-inference option just in case

Mar 20 2018, 3:44 AM
fedor.sergeev added a comment to D44464: allow custom OptBisect classes set to LLVMContext.

A few people have suggested that OptBisect could be dropped completely in favor of a debug counter. I'm not sure if that makes supporting your use case easier or harder.

I have seen this suggestion and honestly I dont see how it could be dropped w/o first replacing every invocation of skipPass with a debug-counter's shouldExecute.
Theoretically debug-counters are at least as powerful as OptBisect, but that stays theoretical until somebody really places them into the code.
For now I only see 5 instances of shouldExecute in a whole LLVM tree.

Mar 20 2018, 3:41 AM

Mar 19 2018

fedor.sergeev updated the diff for D44415: [PM][FunctionAttrs] add NoUnwind attribute inference to PostOrderFunctionAttrs pass.

getting rid of bitvectors

Mar 19 2018, 4:13 PM
fedor.sergeev added a comment to D44415: [PM][FunctionAttrs] add NoUnwind attribute inference to PostOrderFunctionAttrs pass.

Btw, I tried Justin's version of "simplified" meta-programmed callbacks... and fully functional version of it
(one that performs early bailouts on ScanAttrHere/ValidAttrs) starts approaching Chandler's suggested tuple/tuple-iteration by complexity.
The problem with bailouts is that you cant just fill the whole array with calculated values of all the predicates,
or else you need to extend all your predicates with shortcut-ting semantics.

Mar 19 2018, 4:58 AM
fedor.sergeev added inline comments to D44415: [PM][FunctionAttrs] add NoUnwind attribute inference to PostOrderFunctionAttrs pass.
Mar 19 2018, 4:33 AM
fedor.sergeev updated the diff for D44415: [PM][FunctionAttrs] add NoUnwind attribute inference to PostOrderFunctionAttrs pass.

addressing most of Justin's comments on comments ;)

Mar 19 2018, 4:27 AM

Mar 16 2018

fedor.sergeev added a comment to D44464: allow custom OptBisect classes set to LLVMContext.

Is it a matter of properly wording commit message?

I think that the class name OptBisect is a bit misleading. The word Bisect explains just one way of using the skip functionality. I would change to something like OptPassGate, meaning that the class just decides if the pass should be skipped or run.

This suggestion is kinda in line with my first suggestion on generalization - introduce a base class OptPassGate, and derive OptBisect from it; routines that skip the passes should operate with OptPassGate, not with OptBisect directly.

Mar 16 2018, 12:31 AM

Mar 15 2018

fedor.sergeev added a comment to D44464: allow custom OptBisect classes set to LLVMContext.

I would like to stress that we consider OptBisect to be more than just "bisector", but rather the only
available way of skipping optimization passes with an arbitrarily complex control pattern.

I would prefer not to overload the behavior of OptBisect. It's meant to be a simple mechanism that does just this one thing.

Agreed. And this one thing is - "skip the pass".
So in this sense I do not believe we overload the behavior.

Mar 15 2018, 12:27 PM
fedor.sergeev committed rL327619: [New PM][IRCE] port of Inductive Range Check Elimination pass to the new pass….
[New PM][IRCE] port of Inductive Range Check Elimination pass to the new pass…
Mar 15 2018, 4:04 AM
fedor.sergeev closed D43795: [New PM][IRCE] port of Inductive Range Check Elimination pass to the new pass manager.
Mar 15 2018, 4:04 AM
fedor.sergeev updated the diff for D44415: [PM][FunctionAttrs] add NoUnwind attribute inference to PostOrderFunctionAttrs pass.

fixing comments, adding struct instead of tuple.
Addressed pretty much everything except of bitvectors and virtual functions,
just to have some ready base for comparison with other solutions.

Mar 15 2018, 2:51 AM

Mar 14 2018

fedor.sergeev added inline comments to D44415: [PM][FunctionAttrs] add NoUnwind attribute inference to PostOrderFunctionAttrs pass.
Mar 14 2018, 4:14 PM
fedor.sergeev added a comment to D44464: allow custom OptBisect classes set to LLVMContext.

Compiling in multiple threads probably isn't handled well with the current implementation (i.e. prior to this patch) but having different instances of the OptBisect object for each thread may not fix it.

Btw, we do compile in multiple threads, having separate LLVMContexts for each thread.

Mar 14 2018, 2:45 PM
fedor.sergeev added a comment to D44464: allow custom OptBisect classes set to LLVMContext.

Maybe if I knew what is motivating this change it would make more sense to me.

For our custom JIT compiler pipeline we want to have a flexible control on pipeline behavior,
which might have a number of different applications. Immediate one is the ability to gracefully
shut down the compilation that takes too long.
Most likely this particular need is completely off the roster of a static compiler though.

Mar 14 2018, 2:41 PM
fedor.sergeev added a comment to D44415: [PM][FunctionAttrs] add NoUnwind attribute inference to PostOrderFunctionAttrs pass.

Thanks everybody for good comments, will see into them.
Will put virtual functions issue at the very end of the queue.

Mar 14 2018, 2:05 PM

Mar 13 2018

fedor.sergeev updated the summary of D44415: [PM][FunctionAttrs] add NoUnwind attribute inference to PostOrderFunctionAttrs pass.
Mar 13 2018, 10:11 AM
fedor.sergeev added a comment to D43795: [New PM][IRCE] port of Inductive Range Check Elimination pass to the new pass manager.

If anybody has any other comments/objectsions, please, speak up.
I'm going to push this tomorrow.

Mar 13 2018, 10:07 AM
fedor.sergeev updated the diff for D44415: [PM][FunctionAttrs] add NoUnwind attribute inference to PostOrderFunctionAttrs pass.

all the tests updated, one FIXME left on convergent.ll test as intended

Mar 13 2018, 10:07 AM
fedor.sergeev updated the diff for D44415: [PM][FunctionAttrs] add NoUnwind attribute inference to PostOrderFunctionAttrs pass.

cleaning up some tests, still a couple FIXMEs in tests

Mar 13 2018, 9:26 AM
fedor.sergeev updated the summary of D44415: [PM][FunctionAttrs] add NoUnwind attribute inference to PostOrderFunctionAttrs pass.
Mar 13 2018, 4:12 AM
fedor.sergeev created D44415: [PM][FunctionAttrs] add NoUnwind attribute inference to PostOrderFunctionAttrs pass.
Mar 13 2018, 4:11 AM

Mar 12 2018

fedor.sergeev updated the summary of D43795: [New PM][IRCE] port of Inductive Range Check Elimination pass to the new pass manager.
Mar 12 2018, 4:02 PM
fedor.sergeev updated the diff for D43795: [New PM][IRCE] port of Inductive Range Check Elimination pass to the new pass manager.

making BPI completely optional (removing all the LPM changes), cleanup

Mar 12 2018, 4:00 PM
fedor.sergeev planned changes to D43795: [New PM][IRCE] port of Inductive Range Check Elimination pass to the new pass manager.

It appears that turning BPI analysis into the standard one might prove to be rather costly (as that will cause this analysis to be run for every loop pass manager invocation, and currently there are none of the loop passes that use BPI in default pipelines).
I will craft a version of this patch that makes BPI completely optional in IRCE w/o any changes to LoopPassManager.
That will allow to decouple IRCE port from (a rather nonobvious) solution for BPI.

Mar 12 2018, 3:18 PM
fedor.sergeev added a comment to D43795: [New PM][IRCE] port of Inductive Range Check Elimination pass to the new pass manager.

I think before.

Mar 12 2018, 12:45 PM
fedor.sergeev added inline comments to D43795: [New PM][IRCE] port of Inductive Range Check Elimination pass to the new pass manager.
Mar 12 2018, 12:42 PM
fedor.sergeev added a comment to D43795: [New PM][IRCE] port of Inductive Range Check Elimination pass to the new pass manager.

I would just suggest making BPI required as part of the loop standard analyses.
It should be a separate patch (with clear discussion pointing out why it is useful), but BPI is useful to *many* loop passes I suspect, I just think we hadn't wired it up previously.

Should this separate BPI patch go before or after this change?

Mar 12 2018, 12:30 PM
fedor.sergeev added inline comments to D43795: [New PM][IRCE] port of Inductive Range Check Elimination pass to the new pass manager.
Mar 12 2018, 12:23 PM

Mar 5 2018

fedor.sergeev added a reviewer for D43795: [New PM][IRCE] port of Inductive Range Check Elimination pass to the new pass manager: asbirlea.

ping.

Mar 5 2018, 5:27 AM

Feb 28 2018

fedor.sergeev added a reviewer for D43795: [New PM][IRCE] port of Inductive Range Check Elimination pass to the new pass manager: sanjoy.
Feb 28 2018, 2:47 AM

Feb 26 2018

fedor.sergeev added a comment to D43795: [New PM][IRCE] port of Inductive Range Check Elimination pass to the new pass manager.

I'm not happy with BPI solution in this patch.
It is mainly to provoke discussions on how to solve this more gracefully, since no answers were given on llvm-dev@ :

http://lists.llvm.org/pipermail/llvm-dev/2018-January/120768.html
Feb 26 2018, 4:23 PM