[llvm-commits] [llvm] r135006 - in /llvm/trunk: include/llvm/Target/TargetAsmInfo.h lib/Target/TargetAsmInfo.cpp

[Chandler Carruth]

On Wed, Jul 13, 2011 at 12:02 AM, Chris Lattner <clattner at apple.com> wrote:

> This is a serious bug.  MC stuff shouldn't be including Target stuff.  This
> a clear layering violation.  Evan, can you tackle this one too? :)
>

Once this is fixed, we'll remove where we have hacked around the layering
violation in our builds, and can start verifying that it doesn't regress.

I thought Nick or I had at one point reported this layering violation, but
hadn't had the time / motivation to go and refactor all of it. Sorry if we
messed up and didn't report it. =[ I'm going to look at the other places
we've hacked around layering violations via header files and make sure there
is a clear list.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20110713/aa8b8bcf/attachment.html>