alexfh added a comment.

In D136539#3877872 <https://reviews.llvm.org/D136539#3877872>, @sammccall wrote:

> This is a subtle change that needs careful review, and honestly should have a 
> test.
>
> I realize there's a breakage that needs to be fixed with some urgency and you 
> believe you're just restoring the old behavior, however in that case the 
> right course of action is to revert the patch.

You're right. Let's first revert the change that caused the problem and then 
figure out what amount of testing is necessary here.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D136539

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to