<div dir="ltr">It should be testable at some point, but I'm not terribly worried about it as a prerequisite.<div><br></div><div>-eric<br><br><div class="gmail_quote"><div dir="ltr">On Tue, Jan 9, 2018 at 8:06 AM David Blaikie <<a href="mailto:dblaikie@gmail.com">dblaikie@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">[+echristo who did the per-function-subtarget work & might have some suggestions on how this might be tested/testable or not]</div><div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Tue, Jan 9, 2018 at 6:15 AM Alex Bradbury <<a href="mailto:asb@lowrisc.org" target="_blank">asb@lowrisc.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 8 January 2018 at 17:10, David Blaikie via llvm-commits<br>
<<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>> wrote:<br>
> No test coverage?<br>
><br>
> I'm assuming creating the MCSubtargetInfo with the function's feature string<br>
> (rather than an empty string) would fix bugs where function-specific feature<br>
> strings were not respected? Perhaps an example of that could be tested?<br>
<br>
That could make sense. The change is motivated primarily by code<br>
cleanup rather than addressing any known bug. I would think observable<br>
differences in behaviour could be observed in tests like<br>
t2-modified-immediate-fixup-error{1,2}.s, but I can't see an<br>
appropriate baseline -march to use.  Suggestions from those of you<br>
more familiar with the Arm product line would be welcome. Any ideas<br>
anyone?<br>
<br>
Thanks,<br>
<br>
Alex<br>
</blockquote></div></div></blockquote></div></div></div>