[PATCH] D112117: [llvm][LV] Drop poison-generating flags from FP instructions

Fri Dec 10 14:05:31 PST 2021

dcaballe added a comment.

In D112117#3185648 <https://reviews.llvm.org/D112117#3185648>, @reames wrote:

> The change I'm working on is D115460 <https://reviews.llvm.org/D115460>.  To my knowledge, there's no interaction with the vectorizer here, it's simply a code cleanup.  e.g. Do the same thing for all types of poison generating flags.  We may be too conservative in some places, but consistency has a strong value of it's own.  At least per the tests being changed, the vectorizer is not impacted.

Yes, sorry, I introduced new calls to `dropPoisonGeneratingFlags` as part of this change that didn't land so the vectorizer shouldn't be impacted by your change. There are some existing calls but they are very specific and shouldn't have a significant impact on FMF.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D112117