[PATCH] D67968: [TableGen] Introduce a generic automaton (DFA) backend

Mikael Holmén via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Oct 6 23:47:44 PDT 2019


uabelho added a comment.

clang-cuda-build buildbot failed too:
http://lab.llvm.org:8011/builders/clang-cuda-build/builds/37865/steps/ninja%20check%201/logs/stdio

I think it can be fixed with

  @@ -371,20 +371,20 @@ uint64_t Transition::transitionFrom(uint64_t State) {
   void CustomDfaEmitter::printActionType(raw_ostream &OS) { OS << TypeName; }
   
   void CustomDfaEmitter::printActionValue(action_type A, raw_ostream &OS) {
     const ActionTuple &AT = Actions[A];
     if (AT.size() > 1)
  -    OS << "{";
  +    OS << "std::make_tuple(";
     bool First = true;
     for (const auto &SingleAction : AT) {
       if (!First)
         OS << ", ";
       First = false;
       SingleAction.print(OS);
     }
     if (AT.size() > 1)
  -    OS << "}";
  +    OS << ")";
   }
   
   namespace llvm {
   
   void EmitAutomata(RecordKeeper &RK, raw_ostream &OS) {

similar to the fix in r372384 (169cb6347 <https://reviews.llvm.org/rG169cb63478aa047451786d8ccf6af4b721e3b271>).


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D67968/new/

https://reviews.llvm.org/D67968





More information about the llvm-commits mailing list