sp4r74n-117 added a comment.
I do not have commit access so it would be great if you could do it for me.
Thanks.
https://reviews.llvm.org/D42745
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
sp4r74n-117 updated this revision to Diff 133447.
sp4r74n-117 added a comment.
Thanks for the explanation, seems like I was on the wrong track.
I now do realize that my initial approach was in fact duplicating the logic of
'BI__builtin_object_size'.
I've removed most of the changes applied to
sp4r74n-117 updated this revision to Diff 132738.
sp4r74n-117 added a comment.
Thanks for the feedback.
I've added two additional tests to 'test_constant_p' for the case where an
expression evaluating to a constant 0 is passed to the builtin.
The existing tests have been refactored to check
sp4r74n-117 created this revision.
sp4r74n-117 added reviewers: george.karpenkov, dcoughlin, dergachev.a.
sp4r74n-117 created this object with visibility "All Users".
Herald added subscribers: cfe-commits, a.sidorin, szepet, xazax.hun.
Added evaluation of __builtin_constant_p to the dedicated