goldstein.w.n added a comment.

In D137181#3935752 <https://reviews.llvm.org/D137181#3935752>, @owenpan wrote:

> Please mark review comments as done if you have addressed them. Can you also 
> clean up the test cases, removing overlapping/redundant ones, making sure a 
> test case doesn't end with a newline (e.g., line 5380), etc?

Regarding newlines. I have new line before changing the style. Otherwise done. 
(let me know if you want me to remove those).

Regarding removing redundant test cases. I think I've had each of them fail 
independently at some point (although many when I was doing the previous 
PPLevel tracking) so I wouldn't call any of them truly redundant.
I could remove either the `PPDIS_BeforeHash` or `PPDIS_AfterHash` though as the 
they really no independent logic in this commit.

Thoughts?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D137181

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

Reply via email to