Reenable use of TBAA during CodeGen
authorHal Finkel <hfinkel@anl.gov>
Sat, 12 Apr 2014 01:26:00 +0000 (01:26 +0000)
committerHal Finkel <hfinkel@anl.gov>
Sat, 12 Apr 2014 01:26:00 +0000 (01:26 +0000)
commite13b87c08df1a0cde0ccd1a83a7e26ebc1f84117
treed22b9595c9354df8ad4ee2c6a1abb8bdd7b6bcbf
parent24517d023ff353515817ea2107d6eb0d57017bbd
Reenable use of TBAA during CodeGen

We had disabled use of TBAA during CodeGen (even when otherwise using AA)
because the ptrtoint/inttoptr used by CGP for address sinking caused BasicAA to
miss basic type punning that it should catch (and, thus, we'd fail to override
TBAA when we should).

However, when AA is in use during CodeGen, CGP now uses normal GEPs and
bitcasts, instead of ptrtoint/inttoptr, when doing address sinking. As a
result, BasicAA should be able to make us do the right thing in the face of
type-punning, and it seems safe to enable use of TBAA again. self-hosting seems
fine on PPC64/Linux on the P7, with TBAA enabled and -misched=shuffle.

Note: We still don't update TBAA when merging stack slots, although because
BasicAA should now catch all such cases, this is no longer a blocking issue.
Nevertheless, I plan to commit code to deal with this properly in the near
future.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@206093 91177308-0d34-0410-b5e6-96231b3b80d8
lib/CodeGen/ScheduleDAGInstrs.cpp
lib/CodeGen/SelectionDAG/DAGCombiner.cpp
test/CodeGen/PowerPC/aa-tbaa.ll
test/CodeGen/SystemZ/alias-01.ll