[PATCH] D53129: [InstCombine] Fixed crash with aliased functions

Julian Büning via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 11 02:38:22 PDT 2018


jbuening added a comment.

I just noticed that the same `inferLibFuncAttributes(*M->getFunction(...))` pattern also occurs in `LoopIdiomRecognize::processLoopStridedStore()` in `lib/Transforms/Scalar/LoopIdiomRecognize.cpp` which is not fixed by this.

Also, there is `inferLibFuncAttributes(*CI->getCalledFunction(), *TLI);` in `isLocallyOpenedFile()` of `lib/Transforms/Utils/SimplifyLibCalls.cpp` which might suffer from a similar problem (although I have not tested that).


https://reviews.llvm.org/D53129





More information about the llvm-commits mailing list