<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div><div>On Feb 7, 2012, at 4:56 PM, Kostya Serebryany wrote:</div><blockquote type="cite"><div class="gmail_quote">On Tue, Feb 7, 2012 at 4:55 PM, John McCall <span dir="ltr"><<a href="mailto:rjmccall@apple.com">rjmccall@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div style="word-wrap:break-word"><div><div class="h5"><div><div>On Feb 7, 2012, at 4:04 PM, Kostya Serebryany wrote:</div><blockquote type="cite"><div class="gmail_quote">On Tue, Feb 7, 2012 at 3:46 PM, John McCall <span dir="ltr"><<a href="mailto:rjmccall@apple.com" target="_blank">rjmccall@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
On Feb 7, 2012, at 2:07 PM, Kostya Serebryany wrote:<br>
> Slightly formatted/commented patch.<br>
> WDYT?<br>
<br>
This seems to work fine, except that reading a field from a const<br>
AttrConst is not a constant expression in C++03, so the "set"<br>
declarations (ParameterOnly, FunctionOnly, VarArgsIncompatible,<br>
and MutuallyIncompatible) still require a global constructor.</blockquote><div>OMG, yes, indeed. </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"> You<br>
can split the values into separate 'const uint64_t' declarations and<br>
use those; it makes the header even uglier, but it works.<br></blockquote><div><br></div><div>like this (see the new patch)? </div></div></blockquote><br></div></div></div><div>Yes, that's great, thanks!</div></div>
</blockquote><div><br></div><div>Shall I commit? </div></div></blockquote><br></div><div>Go ahead; we'll adjust as necessary if it doesn't fix our internal problem.</div><div><br></div><div>John.</div></body></html>