diff --git a/include/retdec/llvmir2hll/analysis/def_use_analysis.h b/include/retdec/llvmir2hll/analysis/def_use_analysis.h index 4adcc08d..c15ff8cf 100644 --- a/include/retdec/llvmir2hll/analysis/def_use_analysis.h +++ b/include/retdec/llvmir2hll/analysis/def_use_analysis.h @@ -45,7 +45,7 @@ public: /// A def-use chain (see [ItC]). // Implementation note: we have to use std::vector instead of std::map to - // make the chain deterministic (#1386). + // make the chain deterministic. using DefUseChain = std::vector>; public: diff --git a/scripts/retdec-unpacker.sh b/scripts/retdec-unpacker.sh index 3abd0e0d..145a81c9 100644 --- a/scripts/retdec-unpacker.sh +++ b/scripts/retdec-unpacker.sh @@ -194,7 +194,7 @@ while [ "$CONTINUE" = "1" ]; do IN="$OUT" else # Remove the temporary file, just in case some of the unpackers crashed - # during unpacking and left it on the disk (e.g. upx, see #1669). + # during unpacking and left it on the disk (e.g. upx). rm -f "$OUT.tmp" CONTINUE=0 fi diff --git a/src/ar-extractor/archive_wrapper.cpp b/src/ar-extractor/archive_wrapper.cpp index 93487bae..cb323511 100644 --- a/src/ar-extractor/archive_wrapper.cpp +++ b/src/ar-extractor/archive_wrapper.cpp @@ -268,7 +268,7 @@ bool ArchiveWrapper::extract( /** * @todo In writeFile function call, separator is added to path, this may - * or may not work on Windows OS. Fix this once issue #1908 is resolved. + * or may not work on Windows OS. */ // Map for non-unique names - counts number of name occurrences. diff --git a/src/bin2llvmir/analyses/symbolic_tree.cpp b/src/bin2llvmir/analyses/symbolic_tree.cpp index 57e2a339..b35f33eb 100644 --- a/src/bin2llvmir/analyses/symbolic_tree.cpp +++ b/src/bin2llvmir/analyses/symbolic_tree.cpp @@ -95,7 +95,7 @@ SymbolicTree& SymbolicTree::operator=(SymbolicTree&& other) value = other.value; user = other.user; // Do NOT use `ops = std::move(other.ops);` to allow use like - // `*this = ops[0];`. Use std::swap() instead. See #1581. + // `*this = ops[0];`. Use std::swap() instead. std::swap(ops, other.ops); _failed = other._failed; } diff --git a/src/bin2llvmir/optimizations/control_flow/control_flow.cpp b/src/bin2llvmir/optimizations/control_flow/control_flow.cpp index c723cfe1..16f4f346 100644 --- a/src/bin2llvmir/optimizations/control_flow/control_flow.cpp +++ b/src/bin2llvmir/optimizations/control_flow/control_flow.cpp @@ -565,7 +565,7 @@ bool ControlFlow::runGenericBr(AsmInstruction& ai, llvm::CallInst* call) SymbolicTree root(_RDA, call->getArgOperand(0)); - // PIC code, see #1066. + // PIC code. // User code is calling stub in .plt. // Stub in .plt is computing jmp address (address of import). // 4-form_grabber-b794ce9e.so.elf, .plt:00001BE0 ___ctype_toupper_loc: diff --git a/src/bin2llvmir/optimizations/idioms/idioms_analysis.cpp b/src/bin2llvmir/optimizations/idioms/idioms_analysis.cpp index 11bcc2f9..43a2f4d0 100644 --- a/src/bin2llvmir/optimizations/idioms/idioms_analysis.cpp +++ b/src/bin2llvmir/optimizations/idioms/idioms_analysis.cpp @@ -261,7 +261,7 @@ bool IdiomsAnalysis::doAnalysis(Function & f, Pass * p) { "IdiomsLLVM::exchangeCompareEq"); #if 0 - /* We do not recognize this well, see #546 */ + /* We do not recognize this well */ change_made |= analyse(bb, &IdiomsLLVM::exchangeCompareNeq, "IdiomsLLVM::exchangeCompareNeq"); #endif diff --git a/src/bin2llvmir/optimizations/idioms/idioms_magicdivmod.cpp b/src/bin2llvmir/optimizations/idioms/idioms_magicdivmod.cpp index 05214b85..57c50df7 100644 --- a/src/bin2llvmir/optimizations/idioms/idioms_magicdivmod.cpp +++ b/src/bin2llvmir/optimizations/idioms/idioms_magicdivmod.cpp @@ -1476,7 +1476,7 @@ Instruction * IdiomsMagicDivMod::magicSignedDiv8neg(BasicBlock::iterator iter) c */ Instruction * IdiomsMagicDivMod::signedMod1(BasicBlock::iterator iter) const { /* - * Derivation tree (ARM/ELF, idiom based on #943): + * Derivation tree (ARM/ELF): * * add * /\ diff --git a/src/bin2llvmir/optimizations/syscalls/arm.cpp b/src/bin2llvmir/optimizations/syscalls/arm.cpp index cfecde44..f7fc58ad 100644 --- a/src/bin2llvmir/optimizations/syscalls/arm.cpp +++ b/src/bin2llvmir/optimizations/syscalls/arm.cpp @@ -407,7 +407,7 @@ namespace retdec { namespace bin2llvmir { /** - * Sample in #730 8A5C: + * Sample 8A5C: * 160d8: ea00043b b 0x171cc * 160dc: ef900026 svc 0x00900026 * 160e0: ea000439 b 0x171cc diff --git a/src/bin2llvmir/optimizations/unreachable_funcs/unreachable_funcs.cpp b/src/bin2llvmir/optimizations/unreachable_funcs/unreachable_funcs.cpp index fa9b58fe..f21a8a23 100644 --- a/src/bin2llvmir/optimizations/unreachable_funcs/unreachable_funcs.cpp +++ b/src/bin2llvmir/optimizations/unreachable_funcs/unreachable_funcs.cpp @@ -157,8 +157,7 @@ bool UnreachableFuncs::optimizationCanRun() const { } // The main function has to be a definition, not just a declaration. This - // is needed when decompiling shared libraries containing an import of main - // (see #1387). + // is needed when decompiling shared libraries containing an import of main. if (mainFunc->isDeclaration()) { return false; } diff --git a/src/demangler/gparser.cpp b/src/demangler/gparser.cpp index 5849c0d0..4b69b8f6 100644 --- a/src/demangler/gparser.cpp +++ b/src/demangler/gparser.cpp @@ -3010,7 +3010,6 @@ string cGram::subanalyze(const string input, cGram::errcode *err) { showsubs(substitutions); #endif - //Fix of the bug #914 unsigned tempPos = b36toint(current_sub_id); if ((tempPos+1) >= substitutions.size()) { errString = string("") + "cGram::subanalyze: Syntax error: Non-existent substitution " + current_sub_id + "."; diff --git a/src/dwarfparser/dwarf_file.cpp b/src/dwarfparser/dwarf_file.cpp index 7ef0fe8d..40f7cd70 100644 --- a/src/dwarfparser/dwarf_file.cpp +++ b/src/dwarfparser/dwarf_file.cpp @@ -303,7 +303,7 @@ void DwarfFile::loadDIE(Dwarf_Die die, DwarfBaseElement* &parent, int lvl) DwarfBaseElement* parentElement = parent; parent = nullptr; - // DIE name -- unused, but without it there is a SEG FAULT for some reason, see #1027. + // DIE name -- unused, but without it there is a SEG FAULT for some reason. // char *dieName = nullptr; bool localname = false; diff --git a/src/dwarfparser/dwarf_vars.cpp b/src/dwarfparser/dwarf_vars.cpp index 0f69866b..5ee453a2 100644 --- a/src/dwarfparser/dwarf_vars.cpp +++ b/src/dwarfparser/dwarf_vars.cpp @@ -151,7 +151,7 @@ DwarfVar *DwarfVarContainer::getVarByName(string n) * @return Added parameter or existing one. * * TODO: podla mena to nie je 100%, v jednom DIE nemusi byt meno nastavene. - * v priklade z #820 by sa dali unikatne prepojit podla DW_AT_decl_file + DW_AT_decl_line. + * V jednom samplu sa dali unikatne prepojit podla DW_AT_decl_file + DW_AT_decl_line. * neviem ale ci je toto nastaven vzdy, ideal by bolo kontextove nacitanie, ktore by vedelo * ku ktorej funkcii parameter partri, ktory v poradi je a ak by na tom indexe uz nieco * existovalo tak by sa to len aktualizovalo a nevytvaralo nove. diff --git a/src/fileformat/utils/format_detection.cpp b/src/fileformat/utils/format_detection.cpp index cc477a83..f5f4f6ac 100644 --- a/src/fileformat/utils/format_detection.cpp +++ b/src/fileformat/utils/format_detection.cpp @@ -168,7 +168,6 @@ bool isStrangeFeedface(const std::string &filePath) } } - // See #1912 for detailed information on this condition if (ints[0] == 0xfeedface && ints[1] == 0x10 && ints[2] == 0x02) { // Maximal valid Mach-O value is 0x0b but 0x10 will be safer and diff --git a/src/fileinfo/file_detector/coff_detector.cpp b/src/fileinfo/file_detector/coff_detector.cpp index 1a338ef1..05d59922 100644 --- a/src/fileinfo/file_detector/coff_detector.cpp +++ b/src/fileinfo/file_detector/coff_detector.cpp @@ -428,7 +428,7 @@ void CoffDetector::detectArchitecture() result = "ARM (little endian)"; break; case PELIB_IMAGE_FILE_MACHINE_THUMB: - // "ARM or THUMB (interworking)"; see bug #523 + // "ARM or THUMB (interworking)"; result = "ARM"; break; case PELIB_IMAGE_FILE_MACHINE_ARMNT: diff --git a/src/fileinfo/file_detector/pe_detector.cpp b/src/fileinfo/file_detector/pe_detector.cpp index f3485440..15ea2387 100644 --- a/src/fileinfo/file_detector/pe_detector.cpp +++ b/src/fileinfo/file_detector/pe_detector.cpp @@ -403,7 +403,7 @@ void PeDetector::detectArchitecture() result = "ARM (little endian)"; break; case PELIB_IMAGE_FILE_MACHINE_THUMB: - // "ARM or THUMB (interworking)"; see bug #523 + // "ARM or THUMB (interworking)"; result = "ARM"; break; case PELIB_IMAGE_FILE_MACHINE_ARMNT: diff --git a/src/llvmir2hll/hll/hll_writer.cpp b/src/llvmir2hll/hll/hll_writer.cpp index ffbd807c..5f40860e 100644 --- a/src/llvmir2hll/hll/hll_writer.cpp +++ b/src/llvmir2hll/hll/hll_writer.cpp @@ -630,14 +630,14 @@ bool HLLWriter::emitFunction(ShPtr func) { emitModuleNameForFuncIfAvailable(func); emitAddressRangeForFuncIfAvailable(func); emitLineRangeForFuncIfAvailable(func); - // TODO Disable emission of wrapper info until #1399 is solved. + // TODO Disable emission of wrapper info until #189 is solved. // emitWrapperInfoForFuncIfAvailable(func); emitClassInfoIfAvailable(func); emitDemangledNameIfAvailable(func); emitDetectedCryptoPatternsForFuncIfAvailable(func); emitLLVMIRFixerWarningForFuncIfAny(func); // The comment HAS to be put as the LAST info, right before the function's - // signature. IDA plugin relies on that (#1645). + // signature. IDA plugin relies on that. emitCommentIfAvailable(func); func->accept(this); diff --git a/src/llvmir2hll/hll/hll_writers/c_hll_writer.cpp b/src/llvmir2hll/hll/hll_writers/c_hll_writer.cpp index 12eb81bc..7585bcf0 100644 --- a/src/llvmir2hll/hll/hll_writers/c_hll_writer.cpp +++ b/src/llvmir2hll/hll/hll_writers/c_hll_writer.cpp @@ -1053,7 +1053,7 @@ void CHLLWriter::visit(ShPtr type) { } // Emit 8-bit integers as chars, not as int8_t/uint8_t, because char is - // more readable. See #1150 for more details. + // more readable. if (type->getSize() == 8) { if (type->isUnsigned()) { out << "unsigned "; diff --git a/src/llvmir2hll/ir/variable.cpp b/src/llvmir2hll/ir/variable.cpp index 86e09820..bed5948a 100644 --- a/src/llvmir2hll/ir/variable.cpp +++ b/src/llvmir2hll/ir/variable.cpp @@ -89,7 +89,7 @@ ShPtr Variable::getType() const { * - Internal variables correspond to variables that either have 'internal * linkage' in LLVM IR or are ordinary local variables. * - External variables correspond to variables that either have 'external -* linkage' or are used in a volatile load/store operation (see #1146). +* linkage' or are used in a volatile load/store operation. * * By default, variables are created as internal variables. To make them * external, call @c markAsExternal(). diff --git a/src/llvmir2hll/llvm/llvmir2bir_converters/orig_llvmir2bir_converter/llvm_branch_info.cpp b/src/llvmir2hll/llvm/llvmir2bir_converters/orig_llvmir2bir_converter/llvm_branch_info.cpp index a0e5fd74..fd9662a7 100644 --- a/src/llvmir2hll/llvm/llvmir2bir_converters/orig_llvmir2bir_converter/llvm_branch_info.cpp +++ b/src/llvmir2hll/llvm/llvmir2bir_converters/orig_llvmir2bir_converter/llvm_branch_info.cpp @@ -34,7 +34,7 @@ namespace llvmir2hll { namespace { -// To produce deterministic results (#1386), we need to order the basic blocks +// To produce deterministic results, we need to order the basic blocks // by their name. struct ByNameComparator { bool operator()(const llvm::BasicBlock *b1, const llvm::BasicBlock *b2) const { diff --git a/src/llvmir2hll/optimizer/optimizer_manager.cpp b/src/llvmir2hll/optimizer/optimizer_manager.cpp index 1b11f903..68a5b426 100644 --- a/src/llvmir2hll/optimizer/optimizer_manager.cpp +++ b/src/llvmir2hll/optimizer/optimizer_manager.cpp @@ -245,8 +245,8 @@ void OptimizerManager::optimize(ShPtr m) { // WhileTrueToForLoopOptimizer should be run before // WhileTrueToWhileCondOptimizer. run(m, va, arithmExprEvaluator); - // TODO The WhileTrueToUForLoopOptimizer does nothing at the moment (see - // commit a5f4988 and #1358), so it makes no sense to run it. + // TODO The WhileTrueToUForLoopOptimizer does nothing at the moment, so it + // makes no sense to run it. #if 0 // WhileTrueToUForLoopOptimizer should be run after // WhileTrueToForLoopOptimizer (WhileTrueToForLoopOptimizer may produce diff --git a/src/llvmir2hll/optimizer/optimizers/copy_propagation_optimizer.cpp b/src/llvmir2hll/optimizer/optimizers/copy_propagation_optimizer.cpp index 20a0bccf..13f71d16 100644 --- a/src/llvmir2hll/optimizer/optimizers/copy_propagation_optimizer.cpp +++ b/src/llvmir2hll/optimizer/optimizers/copy_propagation_optimizer.cpp @@ -234,7 +234,7 @@ void CopyPropagationOptimizer::performOptimization() { // For each def-use chain... // We have to iterate over an ordered DU chain to make the optimization - // deterministic (#1386). + // deterministic. for (const auto &du : ordered(ducs->du)) { // Currently, the optimizer is unable to optimize cases when there is // more than one use. If this is the case, skip the chain to make the @@ -272,7 +272,7 @@ void CopyPropagationOptimizer::performOptimization() { // Remove statements that are to be removed and update the CFG. // We have to iterate over ordered statements to make the optimization - // deterministic (#1386). + // deterministic. for (const auto &stmt : ordered(toRemoveStmtsPreserveCalls)) { // Since there may be function calls in the statement, we have to // preserve them. Therefore, we store the result of @@ -368,8 +368,7 @@ void CopyPropagationOptimizer::handleCaseEmptyUses(ShPtr stmt, return; } - // Do not optimize external variables (used in a volatile load/store, see - // #1146). + // Do not optimize external variables (used in a volatile load/store). if (stmtLhsVar->isExternal()) { return; } @@ -423,8 +422,7 @@ void CopyPropagationOptimizer::handleCaseSingleUse(ShPtr stmt, return; } - // Do not optimize external variables (used in a volatile load/store, see - // #1146). + // Do not optimize external variables (used in a volatile load/store). if (stmtLhsVar->isExternal()) { return; } @@ -698,8 +696,7 @@ bool CopyPropagationOptimizer::shouldBeIncludedInDefUseChains( return false; } - // Do not optimize external variables (used in a volatile load/store, see - // #1146). + // Do not optimize external variables (used in a volatile load/store). if (var->isExternal()) { return false; } diff --git a/src/llvmir2hll/optimizer/optimizers/dead_local_assign_optimizer.cpp b/src/llvmir2hll/optimizer/optimizers/dead_local_assign_optimizer.cpp index 39db360a..67a75360 100644 --- a/src/llvmir2hll/optimizer/optimizers/dead_local_assign_optimizer.cpp +++ b/src/llvmir2hll/optimizer/optimizers/dead_local_assign_optimizer.cpp @@ -87,7 +87,7 @@ bool DeadLocalAssignOptimizer::canBeOptimized(ShPtr var, } // We do not want to optimize external variables (used in a volatile - // load/store, see #1146). + // load/store). if (var->isExternal()) { return false; } diff --git a/src/llvmir2hll/optimizer/optimizers/global_to_local_optimizer.cpp b/src/llvmir2hll/optimizer/optimizers/global_to_local_optimizer.cpp index c561c942..68f57c3e 100644 --- a/src/llvmir2hll/optimizer/optimizers/global_to_local_optimizer.cpp +++ b/src/llvmir2hll/optimizer/optimizers/global_to_local_optimizer.cpp @@ -552,7 +552,7 @@ bool GlobalToLocalOptimizer::globalVarMayBeRemovedAsUnused(ShPtr var) // Note: Even though we do not optimize external global variables, we want // to remove them if they are not used anywhere in the module. If we // did not do that, all such external global variables would remain - // in the source code. See #1166 for an example. + // in the source code. return true; } diff --git a/src/llvmir2hll/optimizer/optimizers/simple_copy_propagation_optimizer.cpp b/src/llvmir2hll/optimizer/optimizers/simple_copy_propagation_optimizer.cpp index 711a2be3..bb1105d1 100644 --- a/src/llvmir2hll/optimizer/optimizers/simple_copy_propagation_optimizer.cpp +++ b/src/llvmir2hll/optimizer/optimizers/simple_copy_propagation_optimizer.cpp @@ -124,7 +124,7 @@ void SimpleCopyPropagationOptimizer::tryOptimization(ShPtr stmt) { if (lhsVar->isExternal()) { // We do not want to optimize external variables (used in a volatile - // load/store, see #1146). + // load/store). return; } diff --git a/src/llvmir2hll/support/const_symbol_converter.cpp b/src/llvmir2hll/support/const_symbol_converter.cpp index a91486b0..c20898bf 100644 --- a/src/llvmir2hll/support/const_symbol_converter.cpp +++ b/src/llvmir2hll/support/const_symbol_converter.cpp @@ -44,7 +44,7 @@ ShPtr getArgAsConstInt(ShPtr arg) { // // signal(SIGSTOP, SIG_IGN); // - // TODO Is this valid under all circumstances? See #726. + // TODO Is this valid under all circumstances? ShPtr argWithoutCasts(skipCasts(arg)); // Treat the null pointer as zero (0). In this way, we can convert, e.g., diff --git a/src/llvmir2hll/var_renamer/var_renamers/readable_var_renamer.cpp b/src/llvmir2hll/var_renamer/var_renamers/readable_var_renamer.cpp index 4a0a3caf..c05051bb 100644 --- a/src/llvmir2hll/var_renamer/var_renamers/readable_var_renamer.cpp +++ b/src/llvmir2hll/var_renamer/var_renamers/readable_var_renamer.cpp @@ -377,7 +377,7 @@ ShPtr ReadableVarRenamer::getDeclaredFunc(ShPtr expr) const */ ShPtr ReadableVarRenamer::getVarFromCallArg(ShPtr arg) const { // Motivation: For example, the given function call is present when - // decompiling file enc_1_.exe from #1457: + // decompiling file enc_1_.exe: // // GetSystemTimeAsFileTime((struct FILETIME *)&v1); // diff --git a/src/patterngen/pattern_extractor/pattern_extractor.cpp b/src/patterngen/pattern_extractor/pattern_extractor.cpp index 0f5cc72d..e4731496 100644 --- a/src/patterngen/pattern_extractor/pattern_extractor.cpp +++ b/src/patterngen/pattern_extractor/pattern_extractor.cpp @@ -192,7 +192,7 @@ bool PatternExtractor::processFile() /** - * Check if we can use this 64-bit PowerPC file (issue #1892). + * Check if we can use this 64-bit PowerPC file. * * Problem is there is only one '.opd' section common for all code sections. * This is problem if multiple code sections are present because we do not know diff --git a/src/pdbparser/pdb_file.cpp b/src/pdbparser/pdb_file.cpp index 1f745808..070ab5fe 100644 --- a/src/pdbparser/pdb_file.cpp +++ b/src/pdbparser/pdb_file.cpp @@ -360,7 +360,7 @@ char *PDBFile::extract_stream(PDB_DWORD *pages, int num_pages) */ PDBFileState PDBFile::load_pdb_v200(void) { - //TODO - add support for PDB version 2.00 #1270 + //TODO - add support for PDB version 2.00 return PDB_STATE_UNSUPPORTED_VERSION; } diff --git a/src/pdbparser/pdb_symbols.cpp b/src/pdbparser/pdb_symbols.cpp index 777ae0eb..8c9b6dab 100644 --- a/src/pdbparser/pdb_symbols.cpp +++ b/src/pdbparser/pdb_symbols.cpp @@ -127,7 +127,6 @@ bool PDBFunction::parse_symbol(PDBGeneralSymbol *symbol, PDBTypes *types, PDBSym type_index = func_sym->typind; type_def = reinterpret_cast(types->get_type_by_index(type_index)); - // #1471 if (type_def == nullptr || type_def->type_class != PDBTYPE_FUNCTION) { return false; @@ -314,7 +313,6 @@ void PDBSymbols::parse_symbols(void) new_function = new PDBFunction(m); // Create new function new_function->parse_symbol(symbol, types, this); - // #1471 if (new_function == nullptr || new_function->type_def == nullptr || new_function->type_def->type_class != PDBTYPE_FUNCTION) { delete new_function; diff --git a/src/unpackertool/plugins/upx/upx_stub_signatures.cpp b/src/unpackertool/plugins/upx/upx_stub_signatures.cpp index 44f0bd53..4e453c67 100644 --- a/src/unpackertool/plugins/upx/upx_stub_signatures.cpp +++ b/src/unpackertool/plugins/upx/upx_stub_signatures.cpp @@ -1226,7 +1226,7 @@ Signature ppcElfNrv2eSignature = }; // Modified UPX stubs -// psyb0t - MIPS Little-endian ELF (#1255) +// psyb0t - MIPS Little-endian ELF Signature psyb0t_mipsLeElfNrv2bSignature = { 0xE0, 0x00, 0x11, 0x04, // BAL @@ -1239,7 +1239,7 @@ Signature psyb0t_mipsLeElfNrv2bSignature = 0xC2, 0x77, 0x09, 0x00 // SRL $t6, $t1, 31 }; -// lshttpd - x86 ELF (#1255) +// lshttpd - x86 ELF Signature lshttpd_x86ElfLzmaSignature = { 0xE8, 0x8E, 0x0C, 0x00, 0x00, // CALL diff --git a/src/utils/byte_value_storage.cpp b/src/utils/byte_value_storage.cpp index a8997437..2dfb406f 100644 --- a/src/utils/byte_value_storage.cpp +++ b/src/utils/byte_value_storage.cpp @@ -396,7 +396,7 @@ bool ByteValueStorage::getDouble(std::uint64_t address, double& res) const } } // 2.33 (0x4002a3d7 0a3d70a4) in data section as: a4703d0a d7a30240. - // New ARM compilers from #960 is also generating this kind of double constants. + // New ARM compilers are also generating this kind of double constants. // We are not sure, what part of binary determines which kind of double constants are used. // Currently we use new kind for ARMs > version 5. // To find relevant info, google: "ARM double mixed endian". diff --git a/tests/bin2llvmir/optimizations/phi2seq/phi2seq_tests.cpp b/tests/bin2llvmir/optimizations/phi2seq/phi2seq_tests.cpp index 7a0541a9..01103d84 100644 --- a/tests/bin2llvmir/optimizations/phi2seq/phi2seq_tests.cpp +++ b/tests/bin2llvmir/optimizations/phi2seq/phi2seq_tests.cpp @@ -456,7 +456,6 @@ TEST_F(PHI2SeqTests, cycleVarDepend06) { parseInput(R"( ; Testing cycle variable dependency. Two cycles with same predecessor block. Block with cycle doesn't have successor on its own. - ; This test was created for testing the problem #1055. @.str = private unnamed_addr constant [20 x i8] c"Argument %d %d %d:\0A\00", align 1 diff --git a/tests/bin2llvmir/optimizations/unreachable_funcs/unreachable_funcs_tests.cpp b/tests/bin2llvmir/optimizations/unreachable_funcs/unreachable_funcs_tests.cpp index 74ff5fbd..e3e4bd7e 100644 --- a/tests/bin2llvmir/optimizations/unreachable_funcs/unreachable_funcs_tests.cpp +++ b/tests/bin2llvmir/optimizations/unreachable_funcs/unreachable_funcs_tests.cpp @@ -502,7 +502,7 @@ TEST_F(UnreachableFuncsTests, mainOnlyDeclaration) parseInput(R"( ; When the main function is just a declaration (i.e. it has no body), behave ; like there is no main function. This is needed when decompiling shared - ; libraries containing an import of main (see #1387). + ; libraries containing an import of main. ; Can't be optimized (see above). define void @func() { diff --git a/tests/demangler/gcc_tests.cpp b/tests/demangler/gcc_tests.cpp index b2fedc0c..199b867a 100644 --- a/tests/demangler/gcc_tests.cpp +++ b/tests/demangler/gcc_tests.cpp @@ -50,7 +50,7 @@ TEST_F(GccDemanglerTests, DoNotDemangleCppClassNamesWhenCharacterCountIsBad) TEST_F(GccDemanglerTests, DoNotDemangleCppClassNamesWhenTheyDoNotMatchRegex) { - DEM_EQ("enc_vad_21363218732487324784rufdekdfbnerwquie2r6732", ""); // #1495 + DEM_EQ("enc_vad_21363218732487324784rufdekdfbnerwquie2r6732", ""); DEM_EQ(" 7Polygon", ""); // std::regex_match DEM_EQ("7Polygon ", ""); // std::regex_match DEM_EQ(" 7Polygon ", ""); // std::regex_match diff --git a/tests/llvmir2hll/optimizer/optimizers/auxiliary_variables_optimizer_tests.cpp b/tests/llvmir2hll/optimizer/optimizers/auxiliary_variables_optimizer_tests.cpp index e999f47e..0b480888 100644 --- a/tests/llvmir2hll/optimizer/optimizers/auxiliary_variables_optimizer_tests.cpp +++ b/tests/llvmir2hll/optimizer/optimizers/auxiliary_variables_optimizer_tests.cpp @@ -104,7 +104,7 @@ OptimizeNoAssignStmtOneUseEvenIfLhsVarIsExternal) { // Add a body to the testing function: // // a = 1 (VarDefStmt, where 'a' is an 'external' variable comming from a - // volatile load/store, see #1146) + // volatile load/store) // b = a (VarDefStmt) // return b // @@ -361,7 +361,7 @@ DoNotOptimizeWhenAuxiliaryVariableIsExternal) { // // a = 1 (VarDefStmt) // b = a (VarDefStmt, where 'b' is an 'external' variable comming from a - // volatile load/store, see #1146) + // volatile load/store) // return b // ShPtr varA(Variable::create("a", IntType::create(16))); diff --git a/tests/llvmir2hll/optimizer/optimizers/copy_propagation_optimizer_tests.cpp b/tests/llvmir2hll/optimizer/optimizers/copy_propagation_optimizer_tests.cpp index a94cc129..4271fa78 100644 --- a/tests/llvmir2hll/optimizer/optimizers/copy_propagation_optimizer_tests.cpp +++ b/tests/llvmir2hll/optimizer/optimizers/copy_propagation_optimizer_tests.cpp @@ -180,7 +180,7 @@ DoNotEliminateAssignStmtWhenVariableIsExternal) { // Set-up the module. // // void test() { - // d = 1; (d is 'external' and comes from a volatile store, see #1146) + // d = 1; (d is 'external' and comes from a volatile store) // } // ShPtr varD(Variable::create("d", IntType::create(32))); diff --git a/tests/llvmir2hll/optimizer/optimizers/dead_local_assign_optimizer_tests.cpp b/tests/llvmir2hll/optimizer/optimizers/dead_local_assign_optimizer_tests.cpp index 11e02bf5..66a31b73 100644 --- a/tests/llvmir2hll/optimizer/optimizers/dead_local_assign_optimizer_tests.cpp +++ b/tests/llvmir2hll/optimizer/optimizers/dead_local_assign_optimizer_tests.cpp @@ -306,7 +306,7 @@ DoNotOptimizeAssignmentIntoExternalVariable) { // // def test(): // a = 1 (VarDefStmt, where 'a' is external and comes from a volatile - // store, see #1146) + // store) // ShPtr varA(Variable::create("a", IntType::create(16))); varA->markAsExternal(); diff --git a/tests/llvmir2hll/optimizer/optimizers/simple_copy_propagation_optimizer_tests.cpp b/tests/llvmir2hll/optimizer/optimizers/simple_copy_propagation_optimizer_tests.cpp index da6b134f..a91aa959 100644 --- a/tests/llvmir2hll/optimizer/optimizers/simple_copy_propagation_optimizer_tests.cpp +++ b/tests/llvmir2hll/optimizer/optimizers/simple_copy_propagation_optimizer_tests.cpp @@ -325,8 +325,7 @@ DoNotOptimizeIfLhsIsExternalVariable) { // Set-up the module. // // void test() { - // a = b; // a is 'external' and comes from a volatile store, see - // // #1146). + // a = b; // a is 'external' and comes from a volatile store // return a; // } //