maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

[LV] Remove FIXME about NoImplicitFloat. NFC 

My understanding is that NoImplicitFloat, despite it's name, is
supposed to disable all vectors not just float vectors.

Reviewed By: efriedma

Differential Revision: https://reviews.llvm.org/D134084
This commit is contained in:
Craig Topper 2022-09-19 09:45:27 -07:00
parent 449a7c2c47
commit 90a004b4a1
1 changed files with 2 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
llvm/lib/Transforms/Vectorize/LoopVectorize.cpp
View File

@ -10234,10 +10234,8 @@ bool LoopVectorizePass::processLoop(Loop *L) {
} }
} }
// Check the function attributes to see if implicit floats are allowed. // Check the function attributes to see if implicit floats or vectors are
// FIXME: This check doesn't seem possibly correct -- what if the loop is // allowed.
// an integer loop and the vector instructions selected are purely integer
// vector instructions?
if (F->hasFnAttribute(Attribute::NoImplicitFloat)) { if (F->hasFnAttribute(Attribute::NoImplicitFloat)) {
reportVectorizationFailure( reportVectorizationFailure(
"Can't vectorize when the NoImplicitFloat attribute is used", "Can't vectorize when the NoImplicitFloat attribute is used",