[llvm-commits] [llvm] r137237 - in /llvm/trunk: include/llvm/CodeGen/LexicalScopes.h lib/CodeGen/CMakeLists.txt lib/CodeGen/LexicalScopes.cpp
dpatel at apple.com
Thu Aug 11 14:52:18 CDT 2011
On Aug 11, 2011, at 11:46 AM, Jakob Stoklund Olesen wrote:
> The public interface on LexicalScope is a bit confusing. It seems that dominates() is the only function that was supposed to be public.
The other functions are used by dwarf writer.
> How about hiding the entire class from the public interface? You could provide LexicalScopes::dominates(A, B) instead, just like Dominators does.
More information about the llvm-commits