[PATCH] D92928: [analyzer] Highlight arrows for currently selected event

Denys Petrov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jul 20 08:16:40 PDT 2021


ASDenysPetrov added a comment.

In D92928#2890448 <https://reviews.llvm.org/D92928#2890448>, @vsavchenko wrote:

> IE doesn't seem to support anything.  And the majority of developers (on Linux and on MacOS) have literally no way to test it.  What we gain from supporting IE for non-existing users, we lose in the ability to actually improve this code!

Oh, I absolutely agree with every of your statements. I'm not really excited to test and adjust this stuff for IE. But I occasionally got requests from users which use IE and, //get ready//, the internal browser of Visual Studio :D which exploits IE core. So I kindly ask you to keep the JS IE friendly at least for a while. We don't really make changes too often here.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D92928



More information about the cfe-commits mailing list