[PATCH] D117836: Fix obvious typo
Gabriel Ferreira Teles Gomes via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Jan 21 05:40:25 PST 2022
gftg created this revision.
gftg added a comment.
gftg updated this revision to Diff 401949.
gftg updated this revision to Diff 401963.
gftg edited the summary of this revision.
gftg published this revision for review.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.
/me wondering how to convert this draft submission into a regular one.
gftg added a comment.
Testing that using arc diff --edit --verbatim converts from draft to
regular diff.
gftg added a comment.
Update the commit message with something meaningful for the git history.
The -fcf-protection option takes an optional argument, which allows the
requesting of control-flow protection on returns, or on indirect jumps
and calls, or both. One of the comments in the code applies to returns,
yet it mentions branches (for indirect calls and jumps). This patch
fixes this obvious typo.
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D117836
Files:
clang/include/clang/Basic/TargetInfo.h
Index: clang/include/clang/Basic/TargetInfo.h
===================================================================
--- clang/include/clang/Basic/TargetInfo.h
+++ clang/include/clang/Basic/TargetInfo.h
@@ -1507,7 +1507,7 @@
virtual bool
checkCFProtectionBranchSupported(DiagnosticsEngine &Diags) const;
- /// Check if the target supports CFProtection branch.
+ /// Check if the target supports CFProtection return.
virtual bool
checkCFProtectionReturnSupported(DiagnosticsEngine &Diags) const;
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D117836.401963.patch
Type: text/x-patch
Size: 510 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20220121/0bce2954/attachment.bin>
More information about the cfe-commits
mailing list