eopXD marked an inline comment as done.
eopXD added inline comments.
================
Comment at: clang/lib/Parse/ParsePragma.cpp:1306
StringRef Str = PragmaName.getIdentifierInfo()->getName();
+ if (Str == "loop")
+ return (llvm::Twine("clang loop ") +
Option.getIdentifierInfo()->getName()).str();
----------------
SjoerdMeijer wrote:
> We've got some duplication now, as "loop" is also checked in line 1310. Can
> you see if you can merge these checks, simplify things here a bit?
Yes, the case for `loop` in below can now be removed. Thank you.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D136784/new/
https://reviews.llvm.org/D136784
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
