<div dir="ltr">I looked at the error message and I do believe this should fix it.  I will rename it for now.  Long term I plan to put some of this in a new namespace, but for now to get the build green I'm fine renaming this.</div><br><div class="gmail_quote"><div dir="ltr">On Mon, Oct 17, 2016 at 4:06 PM Zachary Turner <<a href="mailto:zturner@google.com">zturner@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr" class="gmail_msg">I can easily just rename this to something else.  However, I can't do a modules build to test this out.  If I change the name of this enum, is that likely to fix it?</div><br class="gmail_msg"><div class="gmail_quote gmail_msg"><div dir="ltr" class="gmail_msg">On Mon, Oct 17, 2016 at 4:04 PM Tim Northover <<a href="mailto:t.p.northover@gmail.com" class="gmail_msg" target="_blank">t.p.northover@gmail.com</a>> wrote:<br class="gmail_msg"></div><blockquote class="gmail_quote gmail_msg" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Zachary,<br class="gmail_msg">
<br class="gmail_msg">
On 17 October 2016 at 13:57, Zachary Turner via llvm-commits<br class="gmail_msg">
<<a href="mailto:llvm-commits@lists.llvm.org" class="gmail_msg" target="_blank">llvm-commits@lists.llvm.org</a>> wrote:<br class="gmail_msg">
> +enum class HexStyle { Upper, Lower, PrefixUpper, PrefixLower };<br class="gmail_msg">
<br class="gmail_msg">
It looks like the modules build doesn't like this:<br class="gmail_msg">
<a href="http://lab.llvm.org:8080/green/job/clang-stage2-cmake-modulesRDA_build/670/console" rel="noreferrer" class="gmail_msg" target="_blank">http://lab.llvm.org:8080/green/job/clang-stage2-cmake-modulesRDA_build/670/console</a>.<br class="gmail_msg">
It seems to be conflicting with an existing namespace HexStyle in<br class="gmail_msg">
MCInstPrinter.h.<br class="gmail_msg">
<br class="gmail_msg">
I *think* it's probably an ODR-violation that is just being picked up<br class="gmail_msg">
now because the modules build is stricter about such things. This is<br class="gmail_msg">
based on Clang complaining normally if it's done in the same<br class="gmail_msg">
translation unit, and the situation not being specifically mentioned<br class="gmail_msg">
in the allowable multi-definition clause of the standard as far as I<br class="gmail_msg">
can see.<br class="gmail_msg">
<br class="gmail_msg">
Could you have a bit of a think about the situation before recommitting?<br class="gmail_msg">
<br class="gmail_msg">
Cheers.<br class="gmail_msg">
<br class="gmail_msg">
Tim.<br class="gmail_msg">
</blockquote></div></blockquote></div>