AFAICT these dyncasts cannot fail -- we've already checked the ELF format when loading up the executable, and will have already bailed there if there was a mismatch.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Paths
| Differential D148848
[BOLT] Remove unneeded dyncasts ClosedPublic Authored by urnathan on Apr 20 2023, 3:14 PM.
Details
Summary AFAICT these dyncasts cannot fail -- we've already checked the ELF format when loading up the executable, and will have already bailed there if there was a mismatch.
Diff Detail
Event TimelineThis revision is now accepted and ready to land.Apr 20 2023, 7:39 PM Closed by commit rGffb42e313d54: [BOLT] Remove unneeded dyncasts (authored by urnathan). · Explain WhyApr 21 2023, 10:41 AM This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 515829 bolt/lib/Rewrite/RewriteInstance.cpp
|