[PATCH] D57180: [LV] Avoid adding into interleaved group in presence of WAW dependency

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 1 05:55:48 PST 2019


fhahn added a comment.

> ! In D57180#1380108 <https://reviews.llvm.org/D57180#1380108>, @hsaito wrote:
>  Anyway, do we ever deal with so many loads/stores for this efficiency to avoid extra scanning to actually matter? I'm just curious.

I'd say usually the number would be quite small and there would be no significant compile time problem. But it is not impossible to hit a worst case scenario, given the wide range of frontends and source code that gets thrown at LLVM. Also, compile time is a key metric for some users and unnecessarily wasting a bit here and there leads to a death by a thousand paper cuts :)


Repository:
  rL LLVM

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

https://reviews.llvm.org/D57180





More information about the llvm-commits mailing list