aaron.ballman added a comment.

In D51650#3126569 <https://reviews.llvm.org/D51650#3126569>, @akuegel wrote:

> Since it is not clear whether the semantic change was intended, I think it 
> makes sense to revert the patch for now. If it is intended, it might be good 
> to mention it in the change description, so that people are warned.

That looks like an unintended change to me, likely due to the new mutual 
exclusion checks. Thanks for letting us know!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D51650

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

Reply via email to