[llvm-commits] CVS: llvm/include/llvm/PassManager.h

Devang Patel dpatel at apple.com
Tue Dec 19 12:05:26 PST 2006


On Dec 19, 2006, at 12:01 PM, Bill Wendling wrote:

> On 12/19/06, Devang Patel <dpatel at apple.com> wrote:
>>
>>
>> Changes in directory llvm/include/llvm:
>>
>> PassManager.h updated: 1.36 -> 1.37
>> ---
>> Log message:
>>
>> s/BasicBlockPassManager/BBPassManager/g
>> s/ModulePassManager/MPPassManager/g
>>
> Why these name changes? LLVM seems to enjoy longer, more descriptive
> names than abbreviations.

We had FunctionPassManager and FunctionPassManagerImpl.
And I needed another pass manager to manage FunctionPasses. So I used
FPPassManager name.

So, now

FunctionPassManager and PassManager are externally visible managers.

FunctionPassManagerImpl and PassManagerImpl provides implementation
support for these two externally visible managers.

And, BBPassManager, FPPassManager and MPPassManagers are internal
managers used to implement pass manager infrastructure. Here, I thought,
it would be confusing to use BasicBlockPassManager name. Because it may
imply that it is at the same level as FunctionPassManager (the one  
that is
externally visible in PassManager.h).

-
Devang




More information about the llvm-commits mailing list