[PATCH] D64564: Loop pragma parsing. NFC.

Sjoerd Meijer via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jul 11 06:18:07 PDT 2019


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


================
Comment at: clang/lib/Parse/ParsePragma.cpp:1015
+               .Default("");
+  assert(Str.size() && "Unexpected pragma name");
+  return Str;
----------------
I guess this does behave slightly different, i.e. the assert in a release build.

Before, in a release build, "unroll" was returned for an unexpected pragma, but now we return the empty string "".


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

https://reviews.llvm.org/D64564





More information about the cfe-commits mailing list