diff options
author | Michael Gottesman <mgottesman@apple.com> | 2013-06-18 20:49:40 +0000 |
---|---|---|
committer | Michael Gottesman <mgottesman@apple.com> | 2013-06-18 20:49:40 +0000 |
commit | b2a70564a753d0992cda953e55a51fb1dcc2a9a4 (patch) | |
tree | 6d9e3e31ece756e39442b0c1b91163018bc17b54 /clang/lib/Frontend/CompilerInstance.cpp | |
parent | 3025f2366fd1b4e57f17110be39e352d52267b2c (diff) | |
download | bcm5719-llvm-b2a70564a753d0992cda953e55a51fb1dcc2a9a4.tar.gz bcm5719-llvm-b2a70564a753d0992cda953e55a51fb1dcc2a9a4.zip |
Converted an overly aggressive assert to a conditional check in AddCombineTo64bitMLAL.
Said assert assumes that ADDC will always have a glue node as its second
argument and is checked before we even know that we are actually performing the
relevant MLAL optimization. This is incorrect since on ARM we *CAN* codegen ADDC
with a use list based second argument. Thus to have both effects, I converted
the assert to a conditional check which if it fails we do not perform the
optimization.
In terms of tests I can not produce an ADDC from the IR level until I get in my
multiprecision optimization patch which is forthcoming. The tests for said patch
would cause this assert to fail implying that said tests will provide the
relevant tests.
llvm-svn: 184230
Diffstat (limited to 'clang/lib/Frontend/CompilerInstance.cpp')
0 files changed, 0 insertions, 0 deletions