SjoerdMeijer 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();
----------------
eopXD wrote:
> 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.
I was hoping that modifying the "loop" case would work:
.Case("loop", (llvm::Twine("clang loop ") +
Option.getIdentifierInfo()->getName()).str());
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
