[PATCH] D100721: [SCEVExpander] Try to create ASHR instr for expanded SCEV expr.

Dávid Bolvanský via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Apr 18 06:48:27 PDT 2021


xbolva00 added a comment.

>> If you don't mind me asking, i forgot when was the last time

i have seen any (coherent?) message longer than two phrases from you;

1. Here I just agree with Florian and Nikita. Florian shared good motivation for revert (regressions and pessimized IR) and I think the revert is reasonable solution (as the problem is not so easy to solve other way), so my "+1".

2. Please avoid words like these. Really.

I would accept if you said "hey,  you wrote +1 but why do you think so? Same concerns like florian or nikita or something else?" as a productive criticism but now it looks like an aggressive tone from you.

- We have a freedom of speech and my comment "+1" cannot offend you.
- And are you stalking me?  Do you count number of sentences in my messages on phab? Why do you judge my messages on Phab?

>> all recent comments are simply non-comments flippantly reiterating the last point made in whatever thread.

Stop lying -  https://reviews.llvm.org/p/xbolva00/.

Bluh, and are you stalking me and checking all my comments?  Do you count number of sentences in my messages on phab? Why do you even judge my messages on Phab?

I dont think my comments are somehow bad or inappropriate, others can check it:
https://reviews.llvm.org/p/xbolva00/


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100721



More information about the llvm-commits mailing list