[PATCH] D41633: [InstCombine] Remove unneeded VarArg casts.

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 2 13:49:37 PST 2018


fhahn abandoned this revision.
fhahn added a comment.

Happy new year and thank you very much for having a look!

If `ext_method` is a definition InstCombine already gets rid of the bitcast in the test case. It sounds like this is the best we can do. I'll abandon this review. But I think it would be worth to commit the test case (with an definition of `ext_method` instead of declare ) to the file, as this case seems not to be covered. Unless there are any objection, I'll do that as a NFC.


https://reviews.llvm.org/D41633





More information about the llvm-commits mailing list