8 Commits (186b10e4f59f835774fb8195290b6fc04d673e2b)

Author	SHA1	Message	Date
Sungkeun Cho	2886e849da	[frontend,pulse] Avoid dereference of C struct ... Summary: This diff avoids dereference of C struct, in its frontend and its semantics of Pulse. In SIL, C struct is not first-class value, thus dereferencing on it does not make sense. Reviewed By: ezgicicek Differential Revision: D27953258 fbshipit-source-id: 348d56338	4 years ago
Sungkeun Cho	851fb3267b	[frontend] ObjC setter/getter for C struct field ... Summary: This diff copies each field values inside setter/getter of ObjC. Reviewed By: ezgicicek Differential Revision: D27940521 fbshipit-source-id: 9977cae75	4 years ago
Sungkeun Cho	fa10bb225a	[uninit/pulse] Do not track captured variables of block ... Summary: When a block value is passed via more than one-depth of function calls, it is not analyzed correctly because current inlining mechanism (specializing objc block parameters) of the frontend works for only one-depth of block passing. This diff gives up analyzing initialized-ness of captured variables in ObjC to avoid FPs. Reviewed By: da319 Differential Revision: D27885395 fbshipit-source-id: fc6b4663c	4 years ago
Sungkeun Cho	346380c649	[pulse] Init struct fields with zero when a message is sent to nil ... Summary: By D26664187 (e4ff4b500a), it returns nil when a message is sent to nil. On the other hand, when the return type is a struct type, the semantics should be different: it should return a struct with all fields are initialized with nil. According to https://developer.apple.com/library/archive/documentation/Cocoa/Conceptual/ProgrammingWithObjectiveC/WorkingwithObjects/WorkingwithObjects.html#//apple_ref/doc/uid/TP40011210-CH4-SW28 > Note: If you expect a return value from a message sent to nil, the return value will be nil for object return types, 0 for numeric types, and NO for BOOL types. Returned structures have all members initialized to zero. Reviewed By: jvillard Differential Revision: D27289996 fbshipit-source-id: 1ddd9b088	4 years ago
Daiva Naudziuniene	e7aa401777	[pulse][uninit] Remove special case for ObjC block captured variables ... Summary: The translation of captured by reference variables has been fixed for ObjC blocks (D26945575 (778c629401)), so we do not need to ignore them in uninit analysis anymore. Reviewed By: skcho Differential Revision: D27063663 fbshipit-source-id: 447084d37	4 years ago
Sungkeun Cho	1bf82d9f86	[pulse] Give semantics of dispatch_sync in ObjC ... Summary: This diff gives semantics of dispatch_sync to call the closure parameter. Reviewed By: jvillard Differential Revision: D25423175 fbshipit-source-id: a45309073	4 years ago
Sungkeun Cho	fa29098376	[pulse] Inter-procedural uninit analysis ... Summary: This diff supports inter-procedural uninit analysis in pulse. * Added `MustBeInitialized` attribute to pre state when an address is read * Remove `Uninitialized` attribute when callee has `WrittenTo` for the same address Reviewed By: jvillard Differential Revision: D25368492 fbshipit-source-id: cbc74d4dc	4 years ago
Sungkeun Cho	0cbe2f9b08	[pulse] Uninitialized value check in pulse ... Summary: This diff adds uninitialized value check in pulse. For now, it supports only simple cases, - declared variables with a type of integer, float, void, and pointer - malloced pointer variables that points to integer, float, void, and pointer TODOs: I will add more cases in the following diffs. - declared/malloced array - declared/malloced struct - inter-procedural checking Reviewed By: jvillard Differential Revision: D25269073 fbshipit-source-id: 317df9a85	4 years ago