- User Since
- Apr 25 2018, 1:47 PM (37 w, 6 d)
Regarding tests, I couldn't actually find any under clang/unittests that test APValue specifically. Is it worth it to add some still?
Mon, Jan 14
Fri, Jan 11
I ended up adding a lot to this patch, so feel free to let me know if I should split this up if it's more difficult to review.
Wed, Jan 9
Any more comments on this patch?
Tue, Jan 8
Posted second attempt at porting ASan using an analysis at D56470.
Thu, Jan 3
Tue, Dec 18
Mon, Dec 17
Dec 14 2018
Dec 12 2018
Dec 11 2018
Analysis can be any information which is not an IR itself. TargetLibraryInfo, DominatorTree, GlobalsMetadata... whatever.
The only problem that you might find when using your custom analysis from the pass will be similar to the current dichotomy of module & function ASan.
You can not *run* module analysis from within a function pass. It can only be run on a module level.
*ping* Any more comments on this patch?
Dec 10 2018
- Added the CXX11 test
Dec 7 2018
Dec 6 2018
Dec 5 2018
Dec 4 2018
*ping* @rsmith Any more comments on this patch or the one before it (https://reviews.llvm.org/D54014)? That one has the fix for pushing and popping another ExprEvalContext before acting on a function body in this patch.
Dec 3 2018
Nov 30 2018
*ping* Does anyone have any more comments on this patch?
Nov 29 2018
It's certainly interesting to degenerate integer-with-fixedpoint to just a mul (since the scaling factor is 2^-n * 2^0, which is just 2^-n), but in the general case you can't avoid doing the scale alignment. Unless I'm missing something.