search for: d29014

Cool, thanks for debugging this issue and letting us know. We have a few patches to fix this issue: - Introduce freeze in IR: https://reviews.llvm.org/D29011 - Lowering freeze: https://reviews.llvm.org/D29014 - Fix loop unswitch: https://reviews.llvm.org/D29015 Bonus patches to recover perf: - Be less conservative in loop unswitching: https://reviews.llvm.org/D29016 - Instcombine support for freeze: https://reviews.llvm.org/D29013 It would be great if you could try out these patches to see if the b...

...imple COPY. This seems to me like SelectionDAG ignores the semantics of FREEZE, since COPY has other semantics? Also, would this be acceptable for the GlobalISel IRTranslator (even if only as a first step)? Best regards, Dominik [1] https://reviews.llvm.org/D76483 [2] https://reviews.llvm.org/D29014 P.S.: is there really no way to disable a single pass from command line? We would like to disable the DivRemPairs pass as a first workaround. -- ---------------------------------------------------------------------- Dominik Montada Email: dominik.montada at hightec-rt.com High...

...Lopes <nlopes at microsoft.com> wrote: > > Cool, thanks for debugging this issue and letting us know. > > > > We have a few patches to fix this issue: > > - Introduce freeze in IR: https://reviews.llvm.org/D29011 > > - Lowering freeze: https://reviews.llvm.org/D29014 > > - Fix loop unswitch: https://reviews.llvm.org/D29015 > > > > Bonus patches to recover perf: > > - Be less conservative in loop unswitching: https://reviews.llvm.org/ > D29016 > > - Instcombine support for freeze: https://reviews.llvm.org/D29013 > > >...

This is a bug found by internal compiler bootstrap, and possibly it is the root cause of https://bugs.llvm.org/show_bug.cgi?id=33652 and https://bugs.llvm.org/show_bug.cgi?id=33626. Here is the testcase 1.c. The original code is at MemorySSA.cpp:586 for rL307764. ------------------------- 1.c --------------------------- long a, c, d, e, f, m, cnt, i_hasval; volatile long b; void goo(long); void