[PM] Sink the reference vs. value decision for IR units out of the
authorChandler Carruth <chandlerc@gmail.com>
Mon, 12 Jan 2015 22:53:31 +0000 (22:53 +0000)
committerChandler Carruth <chandlerc@gmail.com>
Mon, 12 Jan 2015 22:53:31 +0000 (22:53 +0000)
commit21c4458d3c3fd59ea9f8dc91d8eb05a0c1795965
treea8b84b26c3366ab1cd8be38c2a8d39ad8b9cabcb
parentbb71589cf0def0327ed465cbe3e2f7cc25e179aa
[PM] Sink the reference vs. value decision for IR units out of the
templated interface.

So far, every single IR unit I can come up with has address-identity.
That is, when two units of IR are both active in LLVM, their addresses
will be distinct of the IR is distinct. This is clearly true for
Modules, Functions, BasicBlocks, and Instructions. It turns out that the
only practical way to make the CGSCC stuff work the way we want is to
make it true for SCCs as well. I expect this pattern to continue.

When first designing the pass manager code, I kept this dimension of
freedom in the type parameters, essentially allowing for a wrapper-type
whose address did not form identity. But that really no longer makes
sense and is making the code more complex or subtle for no gain. If we
ever have an actual use case for this, we can figure out what makes
sense then and there. It will be better because then we will have the
actual example in hand.

While the simplifications afforded in this patch are fairly small
(mostly sinking the '&' out of many type parameters onto a few
interfaces), it would have become much more pronounced with subsequent
changes. I have a sequence of changes that will completely remove the
code duplication that currently exists between all of the pass managers
and analysis managers. =] Should make things much cleaner and avoid bug
fixing N times for the N pass managers.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@225723 91177308-0d34-0410-b5e6-96231b3b80d8
include/llvm/Analysis/CGSCCPassManager.h
include/llvm/IR/PassManager.h
include/llvm/IR/PassManagerInternal.h
lib/IR/PassManager.cpp