<div dir="ltr">I don't think this makes sense in MC. It's largely used at the DAG/MI/Lowering level and not in MC at all (by a quick grep).<br><div><br></div><div>-eric</div></div><br><div class="gmail_quote">On Mon, Apr 20, 2015 at 1:51 PM Pete Cooper <<a href="mailto:peter_cooper@apple.com">peter_cooper@apple.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">This would help :)<br>
<br>
<br>
> On Apr 20, 2015, at 1:37 PM, Pete Cooper <<a href="mailto:peter_cooper@apple.com" target="_blank">peter_cooper@apple.com</a>> wrote:<br>
><br>
> Hi all<br>
><br>
> This patch moves MachineValueType.h from CodeGen to MC.  There are a few places in X86 MC level tools (assembly instrumentation, inst comments, shuffle decode) which include this file but shouldn’t due to layering violations.<br>
><br>
> Given that I believe the M in MC stands for ‘Machine’, it makes sense to me that MachineValueType lives there too.  Its also a completely self contained header, so is relatively painless to move.<br>
><br>
> I’ve updated all places which include this file and sorted the includes appropriately.  Let me know if there’s anything else to be done.<br>
><br>
> I’ll send out a message to the list to make it clear to out of tree target owners that this could impact them, although its an easy thing to update if they have issues.<br>
><br>
> Note, i also considered actually renaming it to MCValueType.h to match other MC headers, but ConstantPool.h already broke that convention.  I’m happy to do that in this commit or in a later one if anyone is strongly for the idea.<br>
><br>
> Also, if this is approved i’ll do a separate commit to stop X86 MC tools from linking against X86 CodeGen.<br>
><br>
> Cheers,<br>
> Pete<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</blockquote></div>