[PATCH] D63533: [analyzer] Fix clang-tidy crash on GCCAsmStmt

Nick Desaulniers via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jun 19 10:49:31 PDT 2019


nickdesaulniers added inline comments.


================
Comment at: clang/lib/StaticAnalyzer/Core/CoreEngine.cpp:401
+      case Stmt::GCCAsmStmtClass:
+        return;
     }
----------------
NoQ wrote:
> Please add a TODO to actually implement this functionality.
And an `assert` that this `Stmt::isAsmGoto() == true`. (We should not hit this for any inline assembly, just asm goto statements)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D63533





More information about the cfe-commits mailing list