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

Soumi Manna via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu May 18 12:36:30 PDT 2023


Manna marked an inline comment as done.
Manna added inline comments.


================
Comment at: clang/lib/Parse/ParsePragma.cpp:4051
 
+  if (II->isStr("vector"))
+    Actions.DeclareRISCVVBuiltins = true;
----------------
erichkeane wrote:
> 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.
Thank you @erichkeane!


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

https://reviews.llvm.org/D150895



More information about the cfe-commits mailing list