Fixes #64867
Diff Detail
Event Timeline
llvm/lib/Transforms/IPO/AttributorAttributes.cpp | ||
---|---|---|
10350–10365 | Do we really follow any use? |
llvm/lib/Transforms/IPO/AttributorAttributes.cpp | ||
---|---|---|
10350–10365 | The only uses I could think of being worth skipping would be stores or void calls, which would naturally have no uses. Maybe unrecognizable generic calls? Filtering out those is more effort if you want to consider libcalls |
llvm/lib/Transforms/IPO/AttributorAttributes.cpp | ||
---|---|---|
10350–10365 | I mean, this could just flow into a call, what has the call return to do with the value we are tracking, right? |
llvm/lib/Transforms/IPO/AttributorAttributes.cpp | ||
---|---|---|
10350–10365 | So, we want it to follow fadd and friends, is that correct? |
llvm/lib/Transforms/IPO/AttributorAttributes.cpp | ||
---|---|---|
10350–10365 | Should be anything that computeKnownFPClass is implemented for, which ideally would be all recognized FP operations |
llvm/lib/Transforms/IPO/AttributorAttributes.cpp | ||
---|---|---|
10350–10365 | Probably could also filter out cases where we don't know anything about the value |
Do we really follow any use?