[PATCH] D31885: Remove TBAA information from LValues representing union members

Hal Finkel via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Apr 14 09:12:49 PDT 2017


hfinkel added a comment.

In https://reviews.llvm.org/D31885#727307, @kparzysz wrote:

> This is not meant as a fine-grained solution to the TBAA problem, but a temporary fix for generating wrong information.  Just yesterday I helped diagnose yet another problem related to this, so this issue is causing trouble out there.


I understand. But temporary solutions have a tendency to become fairly long lived and I'd like to make sure we're not removing correct information (i.e. the scalar type information) along with the incorrect information. This has been broken for a long time, if it takes another few days to get a better fix then that's worthwhile.


Repository:
  rL LLVM

https://reviews.llvm.org/D31885





More information about the cfe-commits mailing list