<div dir="ltr"><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">Thanks. turns out 2 is a blocker for 1, so I've been working on it today. I think I'm almost there. I'll send you the patch to review or ask for help shortly...</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, May 27, 2020 at 3:29 PM Stephen Kelly <<a href="mailto:steveire@gmail.com">steveire@gmail.com</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">
  
    
  
  <div bgcolor="#FFFFFF">
    <p><br>
    </p>
    <div>On 27/05/2020 16:33, Yitzhak Mandelbaum
      wrote:<br>
    </div>
    <blockquote type="cite">
      
      <div dir="ltr">
        <div class="gmail_default" style="font-family:arial,helvetica,sans-serif">Stephen --
          there are three upstream changes that we'll need to fix all of
          our tests:</div>
        <div class="gmail_default" style="font-family:arial,helvetica,sans-serif">1.
          Transformers' `applyFirst`, as discussed in the thread on
          D80606. </div>
        <div class="gmail_default" style="font-family:arial,helvetica,sans-serif">2. Add an
          overload of the `traverse` matcher for `DynTypedMatcher`.</div>
        <div class="gmail_default" style="font-family:arial,helvetica,sans-serif">3. Support
          `traverse` in the matcher parser (the one used by
          clang-query).</div>
        <div class="gmail_default" style="font-family:arial,helvetica,sans-serif"><br>
        </div>
        <div class="gmail_default" style="font-family:arial,helvetica,sans-serif">I'm working on
          the first one now. But, do you have any thoughts/plans for the
          latter two?</div>
      </div>
    </blockquote>
    <p>Yes, I started a patch in January for (3), but didn't continue it
      as the RFC stalled from then. I've uploaded it as a WIP patch:</p>
    <p> <a href="https://reviews.llvm.org/D80654" target="_blank">https://reviews.llvm.org/D80654</a><br>
    </p>
    <p>I can look into (2) and see if I can finish the patch for (3) in
      the coming days.</p>
    <p>Related to (3), note that you can use</p>
    <p> set traversal AsIs</p>
    <p>and<br>
    </p>
    <p> set traversal IgnoreUnlessSpelledInSource<br>
    </p>
    <p>in clang-query now too. It affects matchers and AST dumps.<br>
    </p>
    Thanks,
    <p>Stephen.<br>
    </p>
    <div class="gmail_quote">
      <blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
        <div dir="auto">
          <div class="gmail_quote" dir="auto">
            <blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
              <div bgcolor="#FFFFFF">
                <div dir="auto">
                  <div class="gmail_quote">
                    <blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"> </blockquote>
                  </div>
                </div>
              </div>
            </blockquote>
          </div>
        </div>
      </blockquote>
    </div>
  </div>

</blockquote></div>