xbolva00 added a comment.

In D71486#1784079 <https://reviews.llvm.org/D71486#1784079>, @sammccall wrote:

> It should be possible to test this by adapting a test case from the original 
> ed8dadb 
> <https://reviews.llvm.org/rGed8dadb37c7e1a7f4889d868ac9b19bfe7762237>, but 
> I'm not certain...


This concern was not addressed/answered.

The commit https://reviews.llvm.org/rG5623bd52acd34db2e9cfc11d1510407610a14db0 
claims that this was reviewed by rsmith, but I don't see rsmith's LGTM here..


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71486



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

Reply via email to