|
From: Julian S. <js...@ac...> - 2012-10-12 10:36:11
|
On Friday, October 12, 2012, Jakub Jelinek wrote: > I'd vote for IRStmt_StoreC. Yeah, me too. > Conditional loads, being IRExprs, can be implemented already using the > current primitives (see the #305728 bug, two of the patches implements it > there for AVX/AVX2), by doing a Iex_Mux0X of a load and the value that is > set if condition is false, and doing another Iex_Mux0X inside of the > load's address (the hack I used was to load from stack pointer if the > condition was false). That is an ingenious hack. Never occurred to me to do that. > But surely, having a new expression type for that is cleaner Yes. It should give much better code too. J |