This is an archive of the discontinued LLVM Phabricator instance.

[APInt][NFC] Fix typo vlalue->value
ClosedPublic

Authored by werat on May 17 2021, 6:11 AM.

Diff Detail

Event Timeline

werat created this revision.May 17 2021, 6:11 AM
werat requested review of this revision.May 17 2021, 6:11 AM
Herald added a project: Restricted Project. · View Herald TranscriptMay 17 2021, 6:11 AM
werat retitled this revision from Fix typo vlalue->value to [APInt][NFC] Fix typo vlalue->value.May 17 2021, 6:13 AM
fhahn accepted this revision.May 17 2021, 6:13 AM
fhahn added a subscriber: fhahn.

LGTM, thanks!

This revision is now accepted and ready to land.May 17 2021, 6:13 AM
werat removed a reviewer: RKSimon.
werat added a comment.EditedMay 17 2021, 7:05 AM

I see some unit tests have failed in the pre-merge checks. Are those flaky? I don't think fixing spelling in the comment should affect the tests :)

I see some unit tests have failed in the pre-merge checks. Are those flaky? I don't think fixing spelling in the comment should affect the tests :)

Yes, some of the openmp tests can be intermittent

This revision was landed with ongoing or failed builds.May 17 2021, 7:19 AM
This revision was automatically updated to reflect the committed changes.