[PATCH] ScopDetection: Only detect scops that have at least one read and one write

Johannes Doerfert doerfert at cs.uni-saarland.de
Thu Feb 19 06:22:43 PST 2015


Hey Tobias,

do you kow how much we actually gain by your change? Why shouldn't we
keep (and delete) SCoPs that have no stores (or at least delete the
loads and let the other passes delete the rest)? Why shouldn't we look at
generating memset intrinsics for loops that only write?

Regarding the patch, could you please comment your member variables
(even though it's pretty obvious).

- Johannes

- F395774: msg-20533-84.dat <http://reviews.llvm.org/F395774>


REPOSITORY
  rL LLVM

http://reviews.llvm.org/D7735

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list