<div><div dir="auto">I didn’t have a specific use case in mind for it, so no preference either way.</div></div><div dir="auto"><br></div><div dir="auto">LGTM as well</div><div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Jun 16, 2020 at 1:05 PM Aaron Ballman via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">aaron.ballman accepted this revision.<br>
aaron.ballman added a comment.<br>
This revision is now accepted and ready to land.<br>
<br>
LGTM unless @jroelofs has a reason why the code was originally written that way, but can you add test coverage for it?<br>
<br>
<br>
Repository:<br>
  rG LLVM Github Monorepo<br>
<br>
CHANGES SINCE LAST ACTION<br>
  <a href="https://reviews.llvm.org/D81953/new/" rel="noreferrer" target="_blank">https://reviews.llvm.org/D81953/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D81953" rel="noreferrer" target="_blank">https://reviews.llvm.org/D81953</a><br>
<br>
<br>
<br>
</blockquote></div></div>