<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-family:courier new,monospace;font-size:small">I quickly went through the <span style="font-family:Arial,Helvetica,sans-serif">D120573. There are many overlapping.  </span></div><div class="gmail_default" style="font-family:courier new,monospace;font-size:small"><br></div><div class="gmail_default" style="font-family:courier new,monospace;font-size:small"><span style="font-family:Arial,Helvetica,sans-serif"><br></span></div></div><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Mar 22, 2022 at 4:23 PM Johannes Doerfert 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">jdoerfert added subscribers: ggeorgakoudis, mikerice, cchen.<br>
jdoerfert added a comment.<br>
<br>
This contains a lot of unrelated changes, leftover comments, etc. As you clean up the code, please also include tests. Clang format the patch, go over the new code and make sure you follow the coding style. Remove unneeded parts (e.g., a map into which you simply push objects then take them out to push them into a vector).<br>
<br>
Also, have you seen D120573 <<a href="https://reviews.llvm.org/D120573" rel="noreferrer" target="_blank">https://reviews.llvm.org/D120573</a>>?<br>
<br>
<br>
Repository:<br>
  rG LLVM Github Monorepo<br>
<br>
CHANGES SINCE LAST ACTION<br>
  <a href="https://reviews.llvm.org/D122255/new/" rel="noreferrer" target="_blank">https://reviews.llvm.org/D122255/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D122255" rel="noreferrer" target="_blank">https://reviews.llvm.org/D122255</a><br>
<br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div>Abid M. Malik<br>******************************************************<br>"I have learned silence from the talkative, toleration from the intolerant, and kindness from the unkind"---Gibran<br>"Success is not for the chosen few, but for the few who choose" --- John Maxwell<br>"Being a good person does not depend on your religion or status in life, your race or skin color, political views or culture. IT DEPENDS ON HOW GOOD YOU TREAT OTHERS"--- Abid<br>"The Universe is talking to us, and the language of the Universe is mathematics."----Abid<br> </div></div></div></div>