[PATCH] D82998: [BasicAA] Enable -basic-aa-recphi by default

Mon Jul 13 09:37:19 PDT 2020

hans added a comment.

In D82998#2144643 <https://reviews.llvm.org/D82998#2144643>, @dmgreen wrote:

> Not a problem. These problems can get very complex in real code.
>
> I have put up D83576 <https://reviews.llvm.org/D83576> to hopefully fix the issue, but won't attempt to turn the option back on until after the new branch gets created.

I tried out D83576 <https://reviews.llvm.org/D83576> on top of af839a96187e3538d63ad57571e4bdf01e2b15c5 <https://reviews.llvm.org/rGaf839a96187e3538d63ad57571e4bdf01e2b15c5> (the flag enabling), but the miscompile in Chromium still remains.

I identified the .o file that changes in a bad way and put together steps to reproduce here: https://bugs.chromium.org/p/chromium/issues/detail?id=1103818#c12

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D82998/new/

https://reviews.llvm.org/D82998