This is an archive of the discontinued LLVM Phabricator instance.

[NFC][InstCombine] Let update_test_checks change variable names of integer-round-up-pow2-allignment.ll
AbandonedPublic

Authored by emgullufsen on Aug 24 2022, 7:55 AM.

Details

Summary

Run update_test_checks.py against integer-round-up-pow2-allignment.ll,
letting it choose different names for variables.

Separating NFC changes here from forthcoming changes

Diff Detail

Event Timeline

emgullufsen created this revision.Aug 24 2022, 7:55 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 24 2022, 7:55 AM
emgullufsen requested review of this revision.Aug 24 2022, 7:55 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 24 2022, 7:55 AM

changed commit msg

emgullufsen retitled this revision from [NFC][InstCombine] Run update_test_checks.py against to [NFC][InstCombine] Let update_test_checks change variable names.Aug 24 2022, 8:45 AM
emgullufsen edited the summary of this revision. (Show Details)
emgullufsen edited the summary of this revision. (Show Details)
emgullufsen retitled this revision from [NFC][InstCombine] Let update_test_checks change variable names to [NFC][InstCombine] Let update_test_checks change variable names of integer-round-up-pow2-allignment.ll.Aug 24 2022, 9:15 AM

These NFC variable name changes were in anticipation of functional changes from D131142, but since that patch no longer changes the test in question (llvm/test/Transforms/InstCombine/integer-round-up-pow2-alignment.ll), these NFC changes made by update_test_checks.py separated out here probably can be abandoned.

I'd leave this file alone if there are no planned changes from the other patch. We can update as needed whenever real changes are made here.

emgullufsen abandoned this revision.Aug 26 2022, 12:48 PM