<div dir="ltr"><br><br><div class="gmail_quote">On Thu Feb 05 2015 at 11:07:46 AM Hal Finkel <<a href="mailto:hfinkel@anl.gov">hfinkel@anl.gov</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">----- Original Message -----<br>
> From: "Eric Christopher" <<a href="mailto:echristo@gmail.com" target="_blank">echristo@gmail.com</a>><br>
> To: "Hal Finkel" <<a href="mailto:hfinkel@anl.gov" target="_blank">hfinkel@anl.gov</a>>, <a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
> Sent: Thursday, February 5, 2015 12:59:44 PM<br>
> Subject: Re: [llvm] r228328 - [PowerPC] Prepare loops for pre-increment loads/stores<br>
><br>
><br>
> +<br>
> + const PPCSubtarget *ST =<br>
> + TM ? TM->getSubtargetImpl(*Header-> getParent()) : nullptr;<br>
> +<br>
><br>
><br>
> I believe you can always depend on the TargetMachine existing.<br>
<br>
This may be something of a left-over (when I had originally worked on this, I had setup the pass so that it could also be invoked from opt, and thus no TM). We can probably clean that up now since I don't see any near-term use for running it outside of the backend.<br>
<br></blockquote><div><br></div><div>Makes sense. It's a little weird that we allow this in general, but ... :)</div><div><br></div><div>-eric</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Thanks again,<br>
Hal<br>
<br>
><br>
><br>
> -eric<br>
<br>
--<br>
Hal Finkel<br>
Assistant Computational Scientist<br>
Leadership Computing Facility<br>
Argonne National Laboratory<br>
</blockquote></div></div>