Please use GitHub pull requests for new patches. Avoid migrating existing patches. Phabricator shutdown timeline

Differential D143255

[NFC] Remove isSafeToSpeculativelyExecute checks for casts
ClosedPublic
Actions

Authored by kachkov98 on Feb 3 2023, 2:29 AM.

Tags

Subscribers

Details

Reviewers

Commits

rG4e37f068dcf4: [NFC] Remove isSafeToSpeculativelyExecute checks for casts

Summary

isSafeToSpeculativelyExecute always return true for casts, so remove
this redundant checks.

Diff Detail

Repository: rG LLVM Github Monorepo

Event Timeline

kachkov98 created this revision.Feb 3 2023, 2:29 AM

Herald added a project: Restricted Project. · View Herald TranscriptFeb 3 2023, 2:29 AM

Herald added a subscriber: hiraditya. · View Herald Transcript

kachkov98 requested review of this revision.Feb 3 2023, 2:29 AM

Herald added a project: Restricted Project. · View Herald TranscriptFeb 3 2023, 2:29 AM

Herald added a subscriber: llvm-commits. · View Herald Transcript

kachkov98 added a reviewer: mkazantsev.Feb 3 2023, 2:29 AM

Herald added a subscriber: StephenFan. · View Herald TranscriptFeb 3 2023, 2:29 AM

kachkov98 mentioned this in D143167: [PHITransAddr] Simplify casts in PHITransAddr.Feb 3 2023, 2:30 AM

LG

This revision is now accepted and ready to land.Feb 3 2023, 2:36 AM

This revision was landed with ongoing or failed builds.Feb 3 2023, 2:39 AM

Closed by commit rG4e37f068dcf4: [NFC] Remove isSafeToSpeculativelyExecute checks for casts (authored by kachkov98). · Explain Why

This revision was automatically updated to reflect the committed changes.

kachkov98 added a commit: rG4e37f068dcf4: [NFC] Remove isSafeToSpeculativelyExecute checks for casts.

Harbormaster completed remote builds in B211678: Diff 494566.Feb 3 2023, 3:44 AM

Revision Contents
Changeset List

Path

Size

llvm/

lib/

Analysis/

PHITransAddr.cpp

9 lines

Diff 494569

llvm/lib/Analysis/PHITransAddr.cpp

Loading...