<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Feb 11, 2015 at 11:41 PM, Yury Gribov <span dir="ltr"><<a href="mailto:y.gribov@samsung.com" target="_blank">y.gribov@samsung.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 02/11/2015 11:01 PM, Alexey Samsonov wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Wed, Feb 11, 2015 at 12:37 AM, Yury Gribov <<a href="mailto:y.gribov@samsung.com" target="_blank">y.gribov@samsung.com</a>> wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On 02/11/2015 09:04 AM, Yury Gribov wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On 02/11/2015 06:07 AM, Justin Bogner wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Alexey Samsonov <<a href="mailto:vonosmas@gmail.com" target="_blank">vonosmas@gmail.com</a>> writes:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Ah, right...<br>
Feel free to revert this change. I have plans to deal with different<br>
UBSan<br>
initialization issues soon, and will see if I would be able to<br>
reinstantiate<br>
this warning on all platforms.<br>
<br>
</blockquote>
<br>
Reverted in r228795.<br>
<br>
</blockquote>
<br>
Can't we just move initializer to separate file for which we then<br>
disable this warning?<br>
<br>
</blockquote>
<br>
Does something like this make sense? It passes tests on Linux x64. I<br>
haven't verified OSX but I can see that -Wglobal-constructors is indeed<br>
disabled for ubsan_init.cc.<br>
</blockquote>
<br>
<br>
In fact, I wanted to split UBSan runtime into three parts: UBSan<br>
(UBSAN_SOURCES without ubsan_init.cc), UBSanCXX (UBSAN_CXX_SOURCES) and<br>
UBSanStandalone (~ubsan_init.cc).<br>
Standalone UBSan would contain all three of them, while the first two would<br>
also be embedded into ASan runtime. After we do this separation, it would<br>
be easy<br>
to specify -Wglobal-constructors just for UBSan and UBSanCXX.<br>
</blockquote>
<br></span>
I wonder if we should move ubsan_init.cc to ubsan_init.o and prelink it similar to how we do this with asan_preinit.o.</blockquote><div><br></div><div>Not sure I understand this. We don't have shared UBSan runtime at the moment.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="HOEnZb"><font color="#888888"><br>
<br>
-Y<br>
<br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr">Alexey Samsonov<br><a href="mailto:vonosmas@gmail.com" target="_blank">vonosmas@gmail.com</a></div></div>
</div></div>