Project Ortega BCM5719 LLVM https://git.raptorcs.com/git/bcm5719-llvm/atom?h=meklort-10.0.1 2020-01-01T19:01:37+00:00 2020-01-01T19:01:37+00:00 Mark de Wever koraq@xs4all.nl 2020-01-01T16:23:21+00:00 urn:sha1:8dc7b982b4556c243e0502e6e230bdd53ddd65ff This avoids new warnings due to D68912 adds -Wrange-loop-analysis to -Wall. Differential Revision: https://reviews.llvm.org/D71857 2019-12-13T18:00:59+00:00 Jonas Devlieghere jonas@devlieghere.com 2019-12-12T18:00:45+00:00 urn:sha1:3011d55f725e280fe29a49bd70a2e0157587b17a Rename LLDB_DISABLE_POSIX to LLDB_ENABLE_POSIX and use cmakedefine01 for consistency. 2019-12-12T19:53:24+00:00 Alex Langford apl@fb.com 2019-11-14T21:41:52+00:00 urn:sha1:3031818a2e9fca1e53cd882ccfcc3718699991b4 Target doesn't really need to know about ClangASTContext more than any other TypeSystem. We can create a method ClangASTContext::GetScratch for anything who needs a ClangASTContext specifically instead of just a generic TypeSystem. 2019-12-04T09:49:25+00:00 Pavel Labath pavel@labath.sk 2019-11-28T16:02:07+00:00 urn:sha1:28e4942b2c3b8961b91b362b4b76b9ca0f735cc2 This constructor was the cause of some pretty weird behavior. Remove it, and update all code to properly dereference the argument instead. 2019-12-04T09:42:32+00:00 Pavel Labath pavel@labath.sk 2019-11-29T10:31:00+00:00 urn:sha1:532290e69fcb0e1f2005853241bc2cac6941e0f7 Summary: The FileSpec class is often used as a sort of a pattern -- one specifies a bare file name to search, and we check if in matches the full file name of an existing module (for example). These comparisons used FileSpec::Equal, which had some support for it (via the full=false argument), but it was not a good fit for this job. For one, it did a symmetric comparison, which makes sense for a function called "equal", but not for typical searches (when searching for "/foo/bar.so", we don't want to find a module whose name is just "bar.so"). This resulted in patterns like: if (FileSpec::Equal(pattern, file, pattern.GetDirectory())) which would request a "full" match only if the pattern really contained a directory. This worked, but the intended behavior was very unobvious. On top of that, a lot of the code wanted to handle the case of an "empty" pattern, and treat it as matching everything. This resulted in conditions like: if (pattern && !FileSpec::Equal(pattern, file, pattern.GetDirectory()) which are nearly impossible to decipher. This patch introduces a FileSpec::Match function, which does exactly what most of FileSpec::Equal callers want, an asymmetric match between a "pattern" FileSpec and a an actual FileSpec. Empty paterns match everything, filename-only patterns match only the filename component. I've tried to update all callers of FileSpec::Equal to use a simpler interface. Those that hardcoded full=true have been changed to use operator==. Those passing full=pattern.GetDirectory() have been changed to use FileSpec::Match. There was also a handful of places which hardcoded full=false. I've changed these to use FileSpec::Match too. This is a slight change in semantics, but it does not look like that was ever intended, and it was more likely a result of a misunderstanding of the "proper" way to use FileSpec::Equal. [In an ideal world a "FileSpec" and a "FileSpec pattern" would be two different types, but given how widespread FileSpec is, it is unlikely we'll get there in one go. This at least provides a good starting point by centralizing all matching behavior.] Reviewers: teemperor, JDevlieghere, jdoerfert Subscribers: emaste, lldb-commits Tags: #lldb Differential Revision: https://reviews.llvm.org/D70851 2019-11-26T14:16:26+00:00 Pavel Labath pavel@labath.sk 2019-11-26T14:11:16+00:00 urn:sha1:5871cba86172c5bd947952a9441acf80332455ea This quashes a -Wformat-truncation warning. 2019-11-13T23:44:51+00:00 Jonas Devlieghere jonas@devlieghere.com 2019-11-13T23:38:43+00:00 urn:sha1:8ac053eea20b56f80653191a210682f8bd6fc10d This commit removes unused methods from the DataEncoder class and cleans up the API by making all the internal methods private. 2019-10-17T19:56:40+00:00 Adrian Prantl aprantl@apple.com 2019-10-17T19:56:40+00:00 urn:sha1:1ad655e255090620705eb4ce85d869a54d971912 This patch removes the size_t return value and the append parameter from the remainder of the Find.* functions in LLDB's internal API. As in the previous patches, this is motivated by the fact that these parameters aren't really used, and in the case of the append parameter were frequently implemented incorrectly. Differential Revision: https://reviews.llvm.org/D69119 llvm-svn: 375160 2019-10-16T19:14:49+00:00 Jason Molenda jmolenda@apple.com 2019-10-16T19:14:49+00:00 urn:sha1:7dd7a3607596a51044b8706ebf6df2e613ce1e9b that runs on arm64 ISA targets, specifically Apple watches. Differential Revision: https://reviews.llvm.org/D68858 llvm-svn: 375032 2019-10-14T20:15:34+00:00 Lawrence D'Anna lawrence_danna@apple.com 2019-10-14T20:15:34+00:00 urn:sha1:62c9fe4273e8f2a0f3f0f4c86de3a90668532354 Summary: This patch re-types everywhere that passes a File::OpenOptions as a uint32_t so it actually uses File::OpenOptions. It also converts some OpenOptions related functions that fail by returning 0 or NULL into llvm::Expected split off from https://reviews.llvm.org/D68737 Reviewers: JDevlieghere, jasonmolenda, labath Reviewed By: labath Subscribers: lldb-commits Tags: #lldb Differential Revision: https://reviews.llvm.org/D68853 llvm-svn: 374817