- User Since
- Oct 16 2019, 1:22 AM (18 w, 6 d)
Thu, Feb 20
Wed, Feb 12
Should be fixed in rGd8a2ea9fd5c7
Tue, Feb 11
Separate into 2 different patches.
Added asserts for the opStatus returned by the operations.
Mon, Feb 10
Simplified logics according to the notes.
Also added more extensive tests.
@efriedma , I updated the files according to your requests.
Sun, Feb 9
Fri, Feb 7
@fhahn does it look right?
Thu, Feb 6
Added IR checks of all the tests.
Fixed noted issues.
@fhahn , do I have a green light?
Wed, Jan 29
Jan 22 2020
Added test cases for the bugs, as well as for testing a simple extraction and a no extraction in the edge case where the function is just a minimal wrapper around a loop.
Jan 21 2020
Done requested nit changes.
Jan 20 2020
@reames any comment on the change?
Jan 15 2020
@fhahn, what do you say?
Jan 14 2020
I agree. This seems like too much of a hassle. I'll abandon this patch.
Jan 13 2020
Jan 10 2020
The old PassManager has the function getAnalysisIfAvailable that should be used in case you only need to update an analysis (equivalent to getCachedResult of the new PassManager). You should probably switch to use this function, instead of the SmallPtrSet<> DTRequested.
Jan 9 2020
This is a welcoming improvement!
Is it sufficient that I'll approve it for you? Or are you waiting for the code-owner?
Jan 8 2020
Jan 7 2020
Reduced the patch to only the extensions of:
- Signaling NaN
- NaN Pyaload
Jan 6 2020
Jan 2 2020
Jan 1 2020
Dec 23 2019
I thought I might add a dynamic detection to the CMake files, as it already does that for headers and other supported functionalities. Didn't find the time for it, as it is a pretty low priority. But, what do you think?
Dec 11 2019
Dec 4 2019
Dec 2 2019
Note that this is a safe overload, as multiplySignificand is private.
Nov 27 2019
Nov 24 2019
Remove unrelated changes. Now only the missing cache queries are present in the patch.
Added comments and fix tests to meet gtestism.