[llvm-commits] [llvm] r137237 - in /llvm/trunk: include/llvm/CodeGen/LexicalScopes.h lib/CodeGen/CMakeLists.txt lib/CodeGen/LexicalScopes.cpp

[Devang Patel]

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.
-
Devang
> 
> How about hiding the entire class from the public interface? You could provide LexicalScopes::dominates(A, B) instead, just like Dominators does.