<div dir="auto">Thanks Mikael,<div dir="auto"><br></div><div dir="auto">That looks easier than plumbing the type name into that code. And thanks for pointing at the bot , I will commit the change and watch it for greenness. </div><div dir="auto"><br></div><div dir="auto">(Just arriving into work, was going to look at this in a few minutes anyway :))</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, 7 Oct 2019, 07:47 Mikael Holmén 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">uabelho added a comment.<br>
<br>
clang-cuda-build buildbot failed too:<br>
<a href="http://lab.llvm.org:8011/builders/clang-cuda-build/builds/37865/steps/ninja%20check%201/logs/stdio" rel="noreferrer noreferrer" target="_blank">http://lab.llvm.org:8011/builders/clang-cuda-build/builds/37865/steps/ninja%20check%201/logs/stdio</a><br>
<br>
I think it can be fixed with<br>
<br>
  @@ -371,20 +371,20 @@ uint64_t Transition::transitionFrom(uint64_t State) {<br>
   void CustomDfaEmitter::printActionType(raw_ostream &OS) { OS << TypeName; }<br>
<br>
   void CustomDfaEmitter::printActionValue(action_type A, raw_ostream &OS) {<br>
     const ActionTuple &AT = Actions[A];<br>
     if (AT.size() > 1)<br>
  -    OS << "{";<br>
  +    OS << "std::make_tuple(";<br>
     bool First = true;<br>
     for (const auto &SingleAction : AT) {<br>
       if (!First)<br>
         OS << ", ";<br>
       First = false;<br>
       SingleAction.print(OS);<br>
     }<br>
     if (AT.size() > 1)<br>
  -    OS << "}";<br>
  +    OS << ")";<br>
   }<br>
<br>
   namespace llvm {<br>
<br>
   void EmitAutomata(RecordKeeper &RK, raw_ostream &OS) {<br>
<br>
similar to the fix in r372384 (169cb6347 <<a href="https://reviews.llvm.org/rG169cb63478aa047451786d8ccf6af4b721e3b271" rel="noreferrer noreferrer" target="_blank">https://reviews.llvm.org/rG169cb63478aa047451786d8ccf6af4b721e3b271</a>>).<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
CHANGES SINCE LAST ACTION<br>
  <a href="https://reviews.llvm.org/D67968/new/" rel="noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D67968/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D67968" rel="noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D67968</a><br>
<br>
<br>
<br>
</blockquote></div>