This is an archive of the discontinued LLVM Phabricator instance.

rewording static_assert to more generic static assertion
AbandonedPublic

Authored by Codesbyusman on Jul 1 2022, 1:21 PM.

Diff Detail

Event Timeline

Codesbyusman created this revision.Jul 1 2022, 1:21 PM
Herald added a project: Restricted Project. · View Herald TranscriptJul 1 2022, 1:21 PM
Codesbyusman requested review of this revision.Jul 1 2022, 1:21 PM
Herald added a project: Restricted Project. · View Herald TranscriptJul 1 2022, 1:21 PM
Herald added a subscriber: cfe-commits. · View Herald Transcript
xgupta added a comment.Jul 1 2022, 7:35 PM

Running check-clang after applying the patch, results into 36 failed test cases. Please update them all.
Also add a description of patch motivation(via the Edit Revision option on the top) for why you are making the changes (like any wrong behavior you are seeing etc).

Ok ... I am working on the tests I will update it

Codesbyusman abandoned this revision.Jul 3 2022, 11:00 AM

*ERR_CLOSED: <differential.updaterevision> This revision has already been
closed*

@Shivam Gupta <reviews+D129014+public+5e9f0937aab6af7b@reviews.llvm.org> can
not update the patch. it seems it is closed.