[PATCH] Some minor update in PassManager.h

Dinesh Dwivedi dinesh.d at samsung.com
Wed Apr 30 01:21:27 PDT 2014


Resending with my comments [adding cc to Phab hasn't added my comments 
when I added llvm-commits in cc]


This patch results in reduction of few PassModel classes. I have checked these 
changes with existing test cases.

As per my understanding, runImpl will be statically resolved and inlined and will 
not cause any function call overhead. In case, run methods changes to take more
arguments, though possibility is rare, this will be easy to extend.

Are these changes ok?

http://reviews.llvm.org/D3543






More information about the llvm-commits mailing list