[PATCH] D29265: [legalize-types] Remove stale entries from SoftenedFloats.

Hal Finkel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 1 06:33:25 PST 2017


hfinkel added a comment.

In https://reviews.llvm.org/D29265#663071, @fhahn wrote:

> @hfinkel Thanks again for the review Hal. Any chance you could also have a look at https://reviews.llvm.org/D29265, which also deals with type legalization?


I don't quite understand what's going on here. Can we visit a node, decide it needs to be a softened float, and then visit it again and make a different decision? Can't we already have used the result from the map?


https://reviews.llvm.org/D29265





More information about the llvm-commits mailing list