[Lldb-commits] [PATCH] D17492: Case sensitive path compare on Windows breaks breakpoints

Zachary Turner via lldb-commits lldb-commits at lists.llvm.org
Tue Feb 23 19:37:56 PST 2016


zturner added inline comments.

================
Comment at: source/Host/common/FileSpec.cpp:519
@@ -515,3 +518,3 @@
     // in one of the FileSpec objects.
 
     if (full || (a.m_directory && b.m_directory))
----------------
Did the case sensitive check get removed here?  I thought this would need to be

```
result = ConstString::Compare(a.m_directory, b.m_directory, a.IsCaseSensitive() || b.IsCaseSensitive());
```

The same later in this function, and also for `FileSpec::Equal`


http://reviews.llvm.org/D17492





More information about the lldb-commits mailing list