[PATCH] D150895: [NFC][CLANG] Fix dereference issue before null check found by Coverity static analyzer tool

Erich Keane via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu May 18 12:22:13 PDT 2023


erichkeane added inline comments.


================
Comment at: clang/lib/Parse/ParsePragma.cpp:4051
 
+  if (II->isStr("vector"))
+    Actions.DeclareRISCVVBuiltins = true;
----------------
So I'd say leave this where it was.  Once line 4043 happens, II is stable, we just need to make sure it doesn't get changed (which I see it doesnt).  So I'd probably just put this back where it was.


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

https://reviews.llvm.org/D150895



More information about the cfe-commits mailing list