[PATCH] D79963: [llvm][test] Add COM: directives before colon-less non-CHECKs in comments. NFC

Jon Roelofs via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 22 05:18:31 PDT 2020


jroelofs added a comment.

In D79963#2050521 <https://reviews.llvm.org/D79963#2050521>, @jhenderson wrote:

> In D79963#2049161 <https://reviews.llvm.org/D79963#2049161>, @jroelofs wrote:
>
> > 183d6af081899973f00fc24aeafcfc32de732f02 <https://reviews.llvm.org/rG183d6af081899973f00fc24aeafcfc32de732f02>
>
>
> Why was this landed? There was varying degrees of dislike from more than one reviewer (myself, and @MaskRay at least). Please revert pending a conclusion to the discussion. This has hardly reached consensus.


5a8db275f8fc8ee19b184f831bda1cdfc6771776 <https://reviews.llvm.org/rG5a8db275f8fc8ee19b184f831bda1cdfc6771776>

Sorry, misread the tone of the discussion as having been addressed by @jdenny & my follow-up comments.


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

https://reviews.llvm.org/D79963





More information about the llvm-commits mailing list