Page MenuHomePhabricator

[MDA] change BlockScanLimit into a command line option.
ClosedPublic

Authored by wengxt on Jul 20 2015, 12:41 PM.

Details

Summary

In the benchmark (https://github.com/vetter/shoc) we are researching,
the duplicated load is not eliminated because MemoryDependenceAnalysis
hit the BlockScanLimit. This patch change it into a command line option
instead of a hardcoded value.

Patched by Xuetian Weng.

Test Plan: test/Analysis/MemoryDependenceAnalysis/memdep-block-scan-limit.ll

Diff Detail

Repository
rL LLVM

Event Timeline

wengxt updated this revision to Diff 30181.Jul 20 2015, 12:41 PM
wengxt retitled this revision from to Change BlockScanLimit into a command line option..
wengxt updated this object.
wengxt added reviewers: jingyue, reames.
wengxt added a subscriber: llvm-commits.
reames edited edge metadata.Jul 21 2015, 2:37 PM
reames added a subscriber: reames.

LGTM

jingyue retitled this revision from Change BlockScanLimit into a command line option. to [MDA] change BlockScanLimit into a command line option..Jul 21 2015, 2:50 PM
jingyue updated this object.
jingyue edited edge metadata.
This revision was automatically updated to reflect the committed changes.