<div dir="ltr">I meant the functions move the header file. In the header file, we can achieve the same by adding an inline keyword to the out of class function definition (vs in class definition). Out of class definitions have better readability because some of the functions are really large.</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Feb 2, 2021 at 12:40 PM David Li via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">davidxl added a comment.<br>
<br>
Is the intention to move them (inline bodies) into the header? For those functions that are not, leave them as being defined out of line. Otherwise moving them into class body is a preparation for the next step (moving to header).<br>
<br>
<br>
CHANGES SINCE LAST ACTION<br>
  <a href="https://reviews.llvm.org/D95832/new/" rel="noreferrer" target="_blank">https://reviews.llvm.org/D95832/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D95832" rel="noreferrer" target="_blank">https://reviews.llvm.org/D95832</a><br>
<br>
</blockquote></div>