[PATCH] D90982: Ignore implicit nodes in IgnoreUnlessSpelledInSource mode

[Leonard Chan via Phabricator via cfe-commits]

leonardchan added a comment.

Hi. I suspect it might be this patch or D90984 <https://reviews.llvm.org/D90984> that might be leading to the test failure we're seeing on our arm64 builders

  FAIL: Clang-Unit :: AST/./ASTTests/Traverse.IgnoreUnlessSpelledInSourceImplicit (15871 of 26886)
  ******************** TEST 'Clang-Unit :: AST/./ASTTests/Traverse.IgnoreUnlessSpelledInSourceImplicit' FAILED ********************
  Note: Google Test filter = Traverse.IgnoreUnlessSpelledInSourceImplicit
  [==========] Running 1 test from 1 test case.
  [----------] Global test environment set-up.
  [----------] 1 test from Traverse
  [ RUN      ] Traverse.IgnoreUnlessSpelledInSourceImplicit
  clang/unittests/AST/ASTTraverserTest.cpp:1114: Failure
        Expected: dumpASTString(TK_AsIs, TUDecl)
        Which is: "\nTranslationUnitDecl\n|-TypedefDecl '__int128_t'\n| ...

Would you mind taking a look? Thanks.

Builder link: https://luci-milo.appspot.com/p/fuchsia/builders/ci/clang-linux-arm64/b8863364626069720736


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D90982