lib/LLVMSupport/Support/YAMLParser.cpp (13 lines): - line 726: // FIXME: Error reporting. - line 1252: // FIXME: Remove once llvm optimizes this to the faster version without hints. - line 1653: // FIXME: Actually parse the c-ns-shorthand-tag rule. - line 1842: // TODO: Tag resolution. - line 1878: // TODO: Handle newlines properly. We need to remove leading whitespace. - line 1938: // TODO: Report error. - line 2011: // TODO: Report error. - line 2015: // TODO: Report error. - line 2023: // TODO: Report error. - line 2027: // TODO: Report error. - line 2035: // TODO: Report error. - line 2039: // TODO: Report error. - line 2394: // TODO: Properly handle tags. "[!!str ]" should resolve to !!str "", not lib/LLVMSupport/include/llvm/Support/ARMTargetParser.h (8 lines): - line 57: // FIXME: TableGen this. - line 76: // FIXME: TableGen this. - line 98: // FIXME: TableGen this. - line 150: // FIXME: BE8 vs. BE32? - line 158: // FIXME: TableGen this. - line 182: // FIXME: SubArch values were simplified to fit into the expectations - line 185: // FIXME: TableGen this. - line 224: // FIXME: These should be moved to TargetTuple once it exists include/llvm/Support/ARMTargetParser.h (8 lines): - line 57: // FIXME: TableGen this. - line 76: // FIXME: TableGen this. - line 98: // FIXME: TableGen this. - line 150: // FIXME: BE8 vs. BE32? - line 158: // FIXME: TableGen this. - line 182: // FIXME: SubArch values were simplified to fit into the expectations - line 185: // FIXME: TableGen this. - line 224: // FIXME: These should be moved to TargetTuple once it exists lib/Database/lmdb/lmdb++.h (7 lines): - line 150: // TODO: mdb_version() - line 151: // TODO: mdb_strerror() - line 185: // TODO: mdb_env_set_assert() - line 186: // TODO: mdb_reader_list() - line 187: // TODO: mdb_reader_check() - line 554: // TODO: mdb_cmp() - line 555: // TODO: mdb_dcmp() lib/Index/IndexDatastore.cpp (7 lines): - line 330: // FIXME: https://bugs.swift.org/browse/SR-10319 - line 353: // FIXME: the database should recover. - line 375: // FIXME: the database should recover. - line 558: // FIXME: Fix this on swift compiler and remove this. - line 720: // FIXME: This doesn't remove unit dependencies. Probably a task for `purgeStaleData`. - line 743: // FIXME: Get referenced records from the database. - line 893: // FIXME: Use modification-time + file-size to check for updated files. lib/LLVMSupport/Support/SourceMgr.cpp (7 lines): - line 217: // FIXME: Handle multibyte characters. - line 299: // FIXME: Handle multibyte characters in the source line. - line 317: // FIXME: This assertion is intended to catch unintended use of multibyte - line 334: // FIXME: Handle multibyte characters in the source line. - line 419: // FIXME: If there are multibyte or multi-column characters in the source, all - line 442: // FIXME: Find the beginning of the line properly for multibyte characters. - line 495: // FIXME: This is trying not to break up replacements, but then to re-sync lib/LLVMSupport/Support/APFloat.cpp (6 lines): - line 44: /// TODO: If clang source code is ever allowed to use constexpr in its own - line 111: TODO: Implement all operations in semPPCDoubleDouble, and delete these - line 2252: /* FIXME: should this just take a const APInt reference? */ - line 3361: // FIXME: this probably shouldn't use 'round half up'. - line 3445: // FIXME: Using a formula based purely on the precision is conservative; - line 3520: // TODO: don't do more precise calculations above than are required. lib/Database/lmdb/mdb.c (6 lines): - line 2250: * lot of pages and never spilled them. TODO: we probably should do - line 2501: /* Use original snapshot. TODO: Should need less care in code - line 3814: rc = EIO; /* TODO: Use which error code? */ - line 4034: /* TODO: Prevent failure here, so parent does not fail */ - line 4305: #if (__GNUC__ * 100 + __GNUC_MINOR__ >= 404) && /* TODO: portability */ \ - line 4551: return EBUSY; /* TODO: Make a new MDB_* error code? */ Sources/ISDBTibs/TibsBuilder.swift (5 lines): - line 86: // FIXME: this should be the -Swift.h file, but ninja doesn't support - line 203: // FIXME: handle via 'directory' field? - line 294: // FIXME: rdar://83355591 avoid -c, since we don't want to spend time writing .o files. - line 348: // FIXME: the generated -Swift.h header should be considered an output, but ninja does not - line 351: // FIXME: some of these are deleted by the compiler!? lib/LLVMSupport/include/llvm/ADT/SmallVector.h (4 lines): - line 108: Size = Capacity = 0; // FIXME: Setting Capacity to 0 is suspect. - line 413: // FIXME: Consider assigning over existing elements, rather than clearing & - line 737: // FIXME: don't do this if they're efficiently moveable. - line 796: // FIXME: this may not actually make any sense if we can efficiently move include/llvm/ADT/SmallVector.h (4 lines): - line 108: Size = Capacity = 0; // FIXME: Setting Capacity to 0 is suspect. - line 413: // FIXME: Consider assigning over existing elements, rather than clearing & - line 737: // FIXME: don't do this if they're efficiently moveable. - line 796: // FIXME: this may not actually make any sense if we can efficiently move lib/LLVMSupport/Support/MemoryBuffer.cpp (4 lines): - line 277: // TODO: Is 16-byte alignment enough? We copy small object files with large - line 330: // FIXME: this chunk of code is duplicated, but it avoids a fstat when - line 354: // FIXME: We should reorganize again getPageSize() on Win32. - line 510: // FIXME: That isn't necessarily true, we should try to mmap stdin and lib/LLVMSupport/Support/Host.cpp (4 lines): - line 445: // FIXME: should we save this for Clang? - line 485: // FIXME: should we save this for Clang? - line 838: // FIXME: this poorly matches the generated SubtargetFeatureKV table. There - line 1069: //FIXME: include cpuid.h from clang or copy __get_cpuid_max here include/llvm/ADT/SetVector.h (3 lines): - line 176: // FIXME: No need to use the non-const iterator when built with - line 242: /// TODO: We should be able to use set_union from SetOperations.h, but - line 257: /// TODO: We should be able to use set_subtract from SetOperations.h, but lib/LLVMSupport/include/llvm/ADT/STLExtras.h (3 lines): - line 421: /// FIXME: Currently if RangeT && is a rvalue reference to a temporary, the - line 545: // ^ TODO: Implement random access methods. - line 549: // ^ FIXME: This follows boost::make_zip_iterator's assumption that all lib/LLVMSupport/include/llvm/ADT/SetVector.h (3 lines): - line 176: // FIXME: No need to use the non-const iterator when built with - line 242: /// TODO: We should be able to use set_union from SetOperations.h, but - line 257: /// TODO: We should be able to use set_subtract from SetOperations.h, but include/llvm/ADT/APFloat.h (3 lines): - line 813: // FIXME: This is due to clang 3.3 (or older version) always checks for the - line 838: // TODO: Remove this constructor. This isn't faster than the first one. - line 1011: // TODO: bool parameters are not readable and a source of bugs. lib/LLVMSupport/include/llvm/ADT/APFloat.h (3 lines): - line 813: // FIXME: This is due to clang 3.3 (or older version) always checks for the - line 838: // TODO: Remove this constructor. This isn't faster than the first one. - line 1011: // TODO: bool parameters are not readable and a source of bugs. include/llvm/ADT/STLExtras.h (3 lines): - line 421: /// FIXME: Currently if RangeT && is a rvalue reference to a temporary, the - line 545: // ^ TODO: Implement random access methods. - line 549: // ^ FIXME: This follows boost::make_zip_iterator's assumption that all include/llvm/ADT/ilist.h (2 lines): - line 80: /// TODO: Remove this layer of indirection. It's not necessary. - line 184: // TODO: Drop this assertion and the transitive type traits anytime after lib/LLVMSupport/include/llvm/ADT/Twine.h (2 lines): - line 59: /// We maintain a number of invariants on Twine objects (FIXME: Why): - line 351: // FIXME: Unfortunately, to make sure this is as efficient as possible we lib/LLVMSupport/include/llvm/ADT/iterator_range.h (2 lines): - line 37: //TODO: Add SFINAE to test that the Container's iterators match the range's - line 41: //TODO: Consider ADL/non-member begin/end calls. lib/LLVMSupport/include/llvm/ADT/Hashing.h (2 lines): - line 325: // FIXME: This needs to be a per-execution seed. This is just a placeholder - line 343: // FIXME: We want to replace is_integral_or_enum and is_pointer here with lib/LLVMSupport/Support/Windows/Process.inc (2 lines): - line 54: // FIXME: FileOffset in MapViewOfFile() should be aligned to not dwPageSize, - line 167: // FIXME: This assumes the wildcard is only in the file name and not in the lib/LLVMSupport/Support/NativeFormatting.cpp (2 lines): - line 191: // FIXME: Implement our formatter to here or Support/Format.h! - line 193: // FIXME: It should be generic to C++11. include/llvm/Support/YAMLParser.h (2 lines): - line 275: /// TODO: Consider making this not a child of Node. - line 389: // TODO: support skipping from the middle of a parsed collection ;/ include/llvm/Support/MachO.h (2 lines): - line 1573: // TODO swap: fp_control_t fpu_fcw; - line 1574: // TODO swap: fp_status_t fpu_fsw; lib/LLVMSupport/include/llvm/Support/COFF.h (2 lines): - line 510: // FIXME: This should be DllCharacteristics to match the COFF spec. - line 517: // FIXME: This should be NumberOfRvaAndSizes to match the COFF spec. include/llvm/ADT/Hashing.h (2 lines): - line 325: // FIXME: This needs to be a per-execution seed. This is just a placeholder - line 343: // FIXME: We want to replace is_integral_or_enum and is_pointer here with lib/Index/StoreSymbolRecord.cpp (2 lines): - line 196: // FIXME: Cache this using libcache ? We may need to repeat searches. - line 302: // FIXME: Ideally we would stop looking for a USR once we found it, but lib/Support/FilePathWatcher.cpp (2 lines): - line 38: // FIXME: We should do something smarter than watching all of root. - line 128: // TODO: implement for platforms without CoreServices. lib/LLVMSupport/Support/Unix/Memory.inc (2 lines): - line 117: // FIXME: Handle huge page requests (MF_HUGE_HINT). - line 236: // FIXME: Can we safely always call this for __GNUC__ everywhere? include/llvm/Support/TargetParser.h (2 lines): - line 17: // FIXME: vector is used because that's what clang uses for subtarget feature - line 31: // FIXME: To TableGen this we need to make some table generated files available include/llvm/Support/COFF.h (2 lines): - line 510: // FIXME: This should be DllCharacteristics to match the COFF spec. - line 517: // FIXME: This should be NumberOfRvaAndSizes to match the COFF spec. include/llvm/Support/Compiler.h (2 lines): - line 162: // FIXME: Provide this for PE/COFF targets. - line 471: // FIXME: Move this to a private config.h as it's not usable in public headers. lib/LLVMSupport/include/llvm/Support/Compiler.h (2 lines): - line 162: // FIXME: Provide this for PE/COFF targets. - line 471: // FIXME: Move this to a private config.h as it's not usable in public headers. lib/LLVMSupport/Support/ARMTargetParser.cpp (2 lines): - line 347: // FIXME: Clang uses it, but it's bogus, since neon defaults to vfpv3. - line 555: // FIXME: this is invalid for WindowsCE. include/llvm/ADT/iterator_range.h (2 lines): - line 37: //TODO: Add SFINAE to test that the Container's iterators match the range's - line 41: //TODO: Consider ADL/non-member begin/end calls. lib/LLVMSupport/include/llvm/Support/JSON.h (2 lines): - line 191: // FIXME: insert() takes const_iterator since C++11, old libstdc++ disagrees. - line 532: // FIXME: this is unneccesarily large (3 pointers). Pointer + length + owned lib/LLVMSupport/include/llvm/ADT/ilist.h (2 lines): - line 80: /// TODO: Remove this layer of indirection. It's not necessary. - line 184: // TODO: Drop this assertion and the transitive type traits anytime after lib/LLVMSupport/include/llvm/Support/SourceMgr.h (2 lines): - line 233: // FIXME: Twine.str() is not very efficient. - line 239: // FIXME: Twine.str() is not very efficient. include/llvm/Support/AArch64TargetParser.h (2 lines): - line 22: // FIXME:This should be made into class design,to avoid dupplication. - line 93: // FIXME: These should be moved to TargetTuple once it exists include/llvm/ADT/Twine.h (2 lines): - line 59: /// We maintain a number of invariants on Twine objects (FIXME: Why): - line 351: // FIXME: Unfortunately, to make sure this is as efficient as possible we include/llvm/Support/JSON.h (2 lines): - line 191: // FIXME: insert() takes const_iterator since C++11, old libstdc++ disagrees. - line 532: // FIXME: this is unneccesarily large (3 pointers). Pointer + length + owned lib/LLVMSupport/include/llvm/Support/YAMLParser.h (2 lines): - line 275: /// TODO: Consider making this not a child of Node. - line 389: // TODO: support skipping from the middle of a parsed collection ;/ lib/LLVMSupport/Support/APInt.cpp (2 lines): - line 394: // TODO - there is scope for optimization here, but at the moment this code - line 467: // FIXME: base 36 lib/LLVMSupport/Support/Signals.cpp (2 lines): - line 115: // FIXME: Subtract necessary number from StackTrace entries to turn return addresses - line 164: // FIXME: Make this the default for llvm-symbolizer. lib/LLVMSupport/Support/ConvertUTFWrapper.cpp (2 lines): - line 35: // FIXME: Make the type of the result buffer correct instead of - line 48: // FIXME: Make the type of the result buffer correct instead of lib/LLVMSupport/Support/Unix/Signals.inc (2 lines): - line 69: // FIXME: We should be able to use for any target that has an - line 542: // FIXME: When we move to C++11, use %t length modifier. It's not in lib/LLVMSupport/include/llvm/Support/MachO.h (2 lines): - line 1573: // TODO swap: fp_control_t fpu_fcw; - line 1574: // TODO swap: fp_status_t fpu_fsw; Sources/ISDBTestSupport/TibsTestWorkspace.swift (2 lines): - line 215: // FIXME: support editing the project.json and update the build settings. - line 313: // FIXME: Use Bundle.module.resourceURL once the fix for SR-12912 is released. lib/LLVMSupport/include/llvm/Support/AArch64TargetParser.h (2 lines): - line 22: // FIXME:This should be made into class design,to avoid dupplication. - line 93: // FIXME: These should be moved to TargetTuple once it exists include/llvm/Support/SourceMgr.h (2 lines): - line 233: // FIXME: Twine.str() is not very efficient. - line 239: // FIXME: Twine.str() is not very efficient. lib/LLVMSupport/include/llvm/Support/YAMLTraits.h (2 lines): - line 809: // FIXME: Force integral conversion to allow strong typedefs to convert. - line 1854: // FIXME: The parser does not support explicit documents terminated with a include/llvm/Support/YAMLTraits.h (2 lines): - line 809: // FIXME: Force integral conversion to allow strong typedefs to convert. - line 1854: // FIXME: The parser does not support explicit documents terminated with a lib/LLVMSupport/include/llvm/Support/TargetParser.h (2 lines): - line 17: // FIXME: vector is used because that's what clang uses for subtarget feature - line 31: // FIXME: To TableGen this we need to make some table generated files available lib/LLVMSupport/Support/Triple.cpp (2 lines): - line 382: // FIXME: Do we need to support these? - line 1589: // FIXME: this is invalid for WindowsCE lib/Index/SymbolIndex.cpp (2 lines): - line 107: // FIXME: The records may contain duplicate USRs at the symbol array, the following - line 125: // FIXME: Make this part of the compiler indexing output. E.g. a C++-like 'struct' should be a 'class' kind. include/llvm/ADT/ilist_iterator.h (1 line): - line 184: /// FIXME: remove this, since there is no implicit conversion to NodeTy. include/llvm/Support/Program.h (1 line): - line 167: /// FIXME: We use EM_CurrentCodePage to write response files for GNU tools in lib/Database/include/IndexStoreDB/Support/LLVM.h (1 line): - line 49: // TODO: DenseMap, ... include/llvm/Support/CommandLine.h (1 line): - line 1425: // FIXME: Reduce this API to a more narrow subset of std::vector include/llvm/ADT/SmallSet.h (1 line): - line 178: // FIXME: Add iterators that abstract over the small and large form, and then lib/LLVMSupport/include/llvm/Support/MathExtras.h (1 line): - line 705: /// \todo FIXME: remove when \c constexpr becomes really \c constexpr Sources/ISDBTibs/TibsToolchain.swift (1 line): - line 96: // FIXME: this method is very incomplete. If we're running on a filesystem that doesn't support lib/LLVMSupport/Support/SmallPtrSet.cpp (1 line): - line 225: // FIXME: From here on we assume that both sets have the same small size. include/llvm/Config/abi-breaking.h (1 line): - line 37: // FIXME: Implement checks without weak. include/indexstore/indexstore_functions.h (1 line): - line 64: // FIXME: we need a runtime check as well since the library may have been built lib/LLVMSupport/Support/Windows/Signals.inc (1 line): - line 580: // FIXME: open files cannot be deleted. lib/LLVMSupport/include/llvm/Support/SmallVectorMemoryBuffer.h (1 line): - line 34: /// FIXME: It'd be nice for this to be a non-templated constructor taking a lib/LLVMSupport/Support/Unix/Threading.inc (1 line): - line 214: char Buffer[len] = {'\0'}; // FIXME: working around MSan false positive. lib/LLVMSupport/Support/Debug.cpp (1 line): - line 141: // TODO: Add a handler for SIGUSER1-type signals so the user can lib/Index/include/IndexStoreDB/Index/IndexSystem.h (1 line): - line 80: // FIXME: Accept a list of active main files so that it can remove stale unit lib/CIndexStoreDB/include/IndexStoreDB/Index/IndexSystem.h (1 line): - line 80: // FIXME: Accept a list of active main files so that it can remove stale unit lib/LLVMSupport/include/llvm/Support/FormatCommon.h (1 line): - line 33: // TODO: Make the format method return the number of bytes written, that lib/Core/include/IndexStoreDB/Index/IndexSystem.h (1 line): - line 80: // FIXME: Accept a list of active main files so that it can remove stale unit lib/Index/include/IndexStoreDB/Support/LLVM.h (1 line): - line 49: // TODO: DenseMap, ... include/llvm/ADT/DenseMap.h (1 line): - line 42: // FIXME: Switch to inheriting constructors when we drop support for older include/llvm/Support/FileSystem.h (1 line): - line 1149: // FIXME: different platforms make different information available "for free" include/llvm/Support/SmallVectorMemoryBuffer.h (1 line): - line 34: /// FIXME: It'd be nice for this to be a non-templated constructor taking a lib/LLVMSupport/include/llvm/Support/FileSystem.h (1 line): - line 1149: // FIXME: different platforms make different information available "for free" lib/Support/include/IndexStoreDB/Support/LLVM.h (1 line): - line 49: // TODO: DenseMap, ... lib/LLVMSupport/include/llvm/ADT/DenseMap.h (1 line): - line 42: // FIXME: Switch to inheriting constructors when we drop support for older lib/LLVMSupport/Support/Windows/Watchdog.inc (1 line): - line 13: // TODO: implement. lib/Database/include/IndexStoreDB/Index/IndexSystem.h (1 line): - line 80: // FIXME: Accept a list of active main files so that it can remove stale unit lib/LLVMSupport/Support/Windows/Program.inc (1 line): - line 65: PathExts.push_back(".exe"); // FIXME: This must be in %PATHEXT%. lib/LLVMSupport/include/llvm/Support/Program.h (1 line): - line 167: /// FIXME: We use EM_CurrentCodePage to write response files for GNU tools in include/llvm/Support/UnicodeCharRanges.h (1 line): - line 54: // FIXME: This could use constexpr + static_assert. This way we lib/CIndexStoreDB/include/IndexStoreDB/Support/LLVM.h (1 line): - line 49: // TODO: DenseMap, ... lib/LLVMSupport/include/llvm/ADT/ilist_iterator.h (1 line): - line 184: /// FIXME: remove this, since there is no implicit conversion to NodeTy. lib/LLVMSupport/Support/Hashing.cpp (1 line): - line 25: // FIXME: Use atomic operations here so that there is no data race. Sources/ISDBTestSupport/TestSources.swift (1 line): - line 84: // FIXME: update incrementally without rescanning everything. lib/CIndexStoreDB/include/indexstore/indexstore_functions.h (1 line): - line 64: // FIXME: we need a runtime check as well since the library may have been built lib/Support/Logging-NonMac.cpp (1 line): - line 19: // FIXME: this can interleave output. lib/Database/ReadTransaction.cpp (1 line): - line 371: // FIXME: When adding a path in the database mark it explicitely whether it lib/LLVMSupport/include/llvm/Config/abi-breaking.h (1 line): - line 37: // FIXME: Implement checks without weak. lib/Database/Database.cpp (1 line): - line 361: // FIXME: With `IgnoreErrors` set to true it can hit an assertion if an error occurs. include/llvm/Support/MathExtras.h (1 line): - line 705: /// \todo FIXME: remove when \c constexpr becomes really \c constexpr include/llvm/Support/FormatCommon.h (1 line): - line 33: // TODO: Make the format method return the number of bytes written, that include/llvm/ADT/SmallString.h (1 line): - line 268: // TODO: Make this const, if it's safe... lib/Core/include/IndexStoreDB/Support/LLVM.h (1 line): - line 49: // TODO: DenseMap, ... Sources/IndexStoreDB/SymbolOccurrence.swift (1 line): - line 32: // FIXME: incorporate relations include/IndexStoreDB/Support/LLVM.h (1 line): - line 49: // TODO: DenseMap, ... lib/LLVMSupport/Support/Error.cpp (1 line): - line 25: // FIXME: This class is only here to support the transition to llvm::Error. It lib/Index/include/indexstore/indexstore_functions.h (1 line): - line 64: // FIXME: we need a runtime check as well since the library may have been built lib/LLVMSupport/include/llvm/Support/UnicodeCharRanges.h (1 line): - line 54: // FIXME: This could use constexpr + static_assert. This way we lib/Support/include/IndexStoreDB/Index/IndexSystem.h (1 line): - line 80: // FIXME: Accept a list of active main files so that it can remove stale unit lib/LLVMSupport/include/llvm/Support/CommandLine.h (1 line): - line 1425: // FIXME: Reduce this API to a more narrow subset of std::vector lib/LLVMSupport/Support/Windows/WindowsSupport.h (1 line): - line 30: #define _WIN32_IE 0x0800 // MinGW at it again. FIXME: verify if still needed. include/IndexStoreDB/Index/IndexSystem.h (1 line): - line 80: // FIXME: Accept a list of active main files so that it can remove stale unit lib/LLVMSupport/include/llvm/ADT/SmallString.h (1 line): - line 268: // TODO: Make this const, if it's safe... lib/LLVMSupport/include/llvm/ADT/SmallSet.h (1 line): - line 178: // FIXME: Add iterators that abstract over the small and large form, and then