This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG481079e2841f: [NFC] Unify FIME with FIXME in comments (authored by b1f6c1c4, committed by alexfh).
Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D98321/new/ 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,
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,
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits