HomePhabricator

Use SCEV to avoid inserting some bounds checks.
AuditedrL337830

Description

Use SCEV to avoid inserting some bounds checks.

This patch uses SCEV to avoid inserting some bounds checks when they are not needed. This slightly improves the performance of code compiled with the bounds check sanitizer.

Differential Revision: https://reviews.llvm.org/D49602

Details

Auditors
kparzysz
Committed
jgalensonJul 24 2018, 8:21 AM
Differential Revision
D49602: Use SCEV to avoid inserting some bounds checks.
Parents
rL337829: [ELF][HEXAGON] Add R_HEX_B15_PCREL_X relocation
Branches
Unknown
Tags
Unknown

Event Timeline

kparzysz raised a concern with this commit.Aug 3 2018, 8:15 AM
kparzysz added a subscriber: kparzysz.
This commit now has outstanding concerns.Aug 3 2018, 8:15 AM

That bug is fixed by https://reviews.llvm.org/D49946, which is awaiting review.

kparzysz accepted this commit.Aug 8 2018, 6:03 AM

The crash seems to be gone. Thanks!

All concerns with this commit have now been addressed.Aug 8 2018, 6:03 AM