search for: safe_r47

Hi all, There might be a bug in DeadStoreElimination.cpp. This pass eliminates stores backwards aggressively in an end BB. It does not check dependencies on stores in an end BB though. For example, in this code snippet: ... 1. %sum.safe_r47.pre-phi = phi i64* [ %sum.safe_r47.pre, %entry.for.end_crit_edge ], [ %sum.safe_r42, %for.body ] 2. %call9 = call i32 @gettimeofday(%struct.timeval* %end, %struct.timeval* null) nounwind 3. %0 = bitcast %struct.timeval* %start to i64* // eliminated by HandleEndBlock in DeadStoreEliminat...

...: >> Hi all, >> >> There might be a bug in DeadStoreElimination.cpp. This pass eliminates >> stores backwards aggressively in an end BB. It does not check dependencies >> on stores in an end BB though. For example, in this code snippet: >>   ... >> 1.  %sum.safe_r47.pre-phi = phi i64* [ %sum.safe_r47.pre, >> %entry.for.end_crit_edge ], [ %sum.safe_r42, %for.body ] >> 2.  %call9 = call i32 @gettimeofday(%struct.timeval* %end, %struct.timeval* >> null) nounwind >> 3.  %0 = bitcast %struct.timeval* %start to i64*           // eliminated by...

...ngbin at gmail.com> wrote: > Hi all, > > There might be a bug in DeadStoreElimination.cpp. This pass eliminates > stores backwards aggressively in an end BB. It does not check dependencies > on stores in an end BB though. For example, in this code snippet: >   ... > 1.  %sum.safe_r47.pre-phi = phi i64* [ %sum.safe_r47.pre, > %entry.for.end_crit_edge ], [ %sum.safe_r42, %for.body ] > 2.  %call9 = call i32 @gettimeofday(%struct.timeval* %end, %struct.timeval* > null) nounwind > 3.  %0 = bitcast %struct.timeval* %start to i64*           // eliminated by > HandleEndB...

...t; >> There might be a bug in DeadStoreElimination.cpp. This pass eliminates > >> stores backwards aggressively in an end BB. It does not check > dependencies > >> on stores in an end BB though. For example, in this code snippet: > >> ... > >> 1. %sum.safe_r47.pre-phi = phi i64* [ %sum.safe_r47.pre, > >> %entry.for.end_crit_edge ], [ %sum.safe_r42, %for.body ] > >> 2. %call9 = call i32 @gettimeofday(%struct.timeval* %end, > %struct.timeval* > >> null) nounwind > >> 3. %0 = bitcast %struct.timeval* %start to i64*...

...bug in DeadStoreElimination.cpp. This pass eliminates >> >> stores backwards aggressively in an end BB. It does not check >> >> dependencies >> >> on stores in an end BB though. For example, in this code snippet: >> >>   ... >> >> 1.  %sum.safe_r47.pre-phi = phi i64* [ %sum.safe_r47.pre, >> >> %entry.for.end_crit_edge ], [ %sum.safe_r42, %for.body ] >> >> 2.  %call9 = call i32 @gettimeofday(%struct.timeval* %end, >> >> %struct.timeval* >> >> null) nounwind >> >> 3.  %0 = bitcast %str...