[PATCH] D98321: [NFC] Unify FIME with FIXME in comments
Jinzheng Tu via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Mar 9 23:36:33 PST 2021
b1f6c1c4 created this revision.
b1f6c1c4 added reviewers: alexfh, craig.topper.
Herald added subscribers: kbarton, hiraditya, nemanjai.
b1f6c1c4 requested review of this revision.
Herald added projects: clang, LLVM.
Herald added subscribers: llvm-commits, cfe-commits.
There are 5 occurrences FIME and 15333 FIXME. All of them should be FIXME.
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D98321
Files:
clang-tools-extra/clang-tidy/ExpandModularHeadersPPCallbacks.cpp
llvm/lib/CodeGen/SelectionDAG/LegalizeVectorOps.cpp
Index: llvm/lib/CodeGen/SelectionDAG/LegalizeVectorOps.cpp
===================================================================
--- llvm/lib/CodeGen/SelectionDAG/LegalizeVectorOps.cpp
+++ llvm/lib/CodeGen/SelectionDAG/LegalizeVectorOps.cpp
@@ -529,7 +529,7 @@
return RecursivelyLegalizeResults(Op, ResultVals);
}
-// FIME: This is very similar to the X86 override of
+// FIXME: This is very similar to the X86 override of
// TargetLowering::LowerOperationWrapper. Can we merge them somehow?
bool VectorLegalizer::LowerOperationWrapper(SDNode *Node,
SmallVectorImpl<SDValue> &Results) {
Index: clang-tools-extra/clang-tidy/ExpandModularHeadersPPCallbacks.cpp
===================================================================
--- clang-tools-extra/clang-tidy/ExpandModularHeadersPPCallbacks.cpp
+++ clang-tools-extra/clang-tidy/ExpandModularHeadersPPCallbacks.cpp
@@ -230,7 +230,7 @@
void ExpandModularHeadersPPCallbacks::PragmaOpenCLExtension(
SourceLocation NameLoc, const IdentifierInfo *, SourceLocation StateLoc,
unsigned) {
- // FIME: Figure out whether it's the right location to parse to.
+ // FIXME: Figure out whether it's the right location to parse to.
parseToLocation(NameLoc);
}
void ExpandModularHeadersPPCallbacks::PragmaWarning(SourceLocation Loc,
@@ -256,7 +256,7 @@
const MacroDefinition &,
SourceRange Range,
const MacroArgs *) {
- // FIME: Figure out whether it's the right location to parse to.
+ // FIXME: Figure out whether it's the right location to parse to.
parseToLocation(Range.getBegin());
}
void ExpandModularHeadersPPCallbacks::MacroDefined(const Token &MacroNameTok,
@@ -271,12 +271,12 @@
void ExpandModularHeadersPPCallbacks::Defined(const Token &MacroNameTok,
const MacroDefinition &,
SourceRange Range) {
- // FIME: Figure out whether it's the right location to parse to.
+ // FIXME: Figure out whether it's the right location to parse to.
parseToLocation(Range.getBegin());
}
void ExpandModularHeadersPPCallbacks::SourceRangeSkipped(
SourceRange Range, SourceLocation EndifLoc) {
- // FIME: Figure out whether it's the right location to parse to.
+ // FIXME: Figure out whether it's the right location to parse to.
parseToLocation(EndifLoc);
}
void ExpandModularHeadersPPCallbacks::If(SourceLocation Loc, SourceRange,
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D98321.329546.patch
Type: text/x-patch
Size: 2602 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210310/1da4b3a8/attachment.bin>
More information about the llvm-commits
mailing list