<p dir="ltr">Yes, asserts and llvm_unreachable are the right tools for finding out if something changes.</p>
<p dir="ltr">Cheers,<br>
Rafael</p>
<div class="gmail_quote">On Jan 21, 2016 5:03 AM, "George Rimar" <<a href="mailto:grimar@accesssoftek.com">grimar@accesssoftek.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">>> +template <class ELFT><br>
>> +void EhFrameHeader<ELFT>::assignEhFrame(EHOutputSection<ELFT> *Sec) {<br>
>> +  if (this->Sec && this->Sec != Sec) {<br>
>> +    warning("multiple .eh_frame sections not supported for .eh_frame_hdr");<br>
>> +    Live = false;<br>
>> +    return;<br>
><br>
>Can this warning ever fire? If not it should be just an assert.<br>
<br>
In current code this should never be fired. I was just afraid about future changes, I am not sure about what can be done with linkerscript,<br>
for example and if it is possible to fire it. I think we can replace warning + "Live = false" with llvm_unreachable for now then, will it be ok ?<br>
<br>
>Cheers,<br>
>Rafael<br>
<br>
George.</blockquote></div>