This is an archive of the discontinued LLVM Phabricator instance.

[clang] NFC: Dead code removal in SemaDecl.cpp, CheckMultiVersionFunction().
ClosedPublic

Authored by tahonermann on Mar 17 2022, 2:20 PM.

Details

Summary

This change removes dead code in the definition of CheckMultiVersionFunction()
in clang/lib/Sema/SemaDecl.cpp. The removed code was made dead by commit
fc53eb69c26cdd7efa6b629c187d04326f0448ca: "Reapply 'Implement target_clones multiversioning'".
See the added code just above the code being deleted; it contains the same
return statement with the previous condition now distributed across an if
statement and a switch statement.

Diff Detail

Event Timeline

tahonermann created this revision.Mar 17 2022, 2:20 PM
Herald added a project: Restricted Project. · View Herald TranscriptMar 17 2022, 2:20 PM
tahonermann published this revision for review.Mar 17 2022, 2:44 PM
tahonermann added inline comments.
clang/lib/Sema/SemaDecl.cpp
10918–10920

These lines are what makes the removed code dead code.

Herald added a project: Restricted Project. · View Herald TranscriptMar 17 2022, 2:44 PM
Herald added a subscriber: cfe-commits. · View Herald Transcript
This revision is now accepted and ready to land.Mar 18 2022, 4:29 AM
erichkeane accepted this revision.Mar 18 2022, 6:47 AM