Page MenuHomePhabricator

brunodefraine (Bruno De Fraine)


User does not belong to any projects.

User Details

User Since
Nov 14 2017, 2:39 AM (95 w, 5 d)

Recent Activity

Nov 14 2017

brunodefraine added a comment to rL317729: Add an @llvm.sideeffect intrinsic.

I think it is a good idea to model the sideeffect intrinsic as "inaccessiblememonly"/"nounwind". That should already treat it as desired in most optimizations. I wonder if the additional special treatment of the intrinsic in optimizations here, is not already implied by these properties (or should be implied by these properties), especially when it comes after a failing test for e.g. mayReadOrWriteMemory() (i.e. inaccessiblememonly fails this, but normally the memory access is irrelevant to the optimization).

Nov 14 2017, 3:06 AM