
<html><head><meta http-equiv="Content-Type" content="text/html charset=windows-1252"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;">Hi Paul, Eric, and Adrian,<div><br></div><div>the constant hoisting pass is only enabled starting with optimization level -O1 - the same as for CodeGenPrepare. It won’t interfere with -O0 debug builds. </div><div><br></div><div>I also don’t want to make the debugger life more difficult and after checking with Adrian I updated the patch to also add debug locations (I just simply forgot them).</div><div>I just copy the debug location from the instruction I generate the rematerialization code for, because they belong together. For the hoisted constants I don’t have a single debug location anymore to pin them to, so I just use the debug location from the following instruction.</div><div><br></div><div>What you guy think?</div><div><br></div><div>Cheers,</div><div>Juergen</div><div><br></div><div><br></div><div></div></body></html>