[llvm-commits] [llvm] r137237 - in /llvm/trunk: include/llvm/CodeGen/LexicalScopes.h lib/CodeGen/CMakeLists.txt lib/CodeGen/LexicalScopes.cpp
Devang Patel
dpatel at apple.com
Thu Aug 11 12:52:18 PDT 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.
-
Devang
>
> 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
mailing list