diff options
| author | JF Bastien <jfb@google.com> | 2014-12-16 20:15:45 +0000 |
|---|---|---|
| committer | JF Bastien <jfb@google.com> | 2014-12-16 20:15:45 +0000 |
| commit | 5d3280c7a7fd9ff33631bf696a383cb758505230 (patch) | |
| tree | 9696f1bda6b443c6f60e99996aacbc09cafedcb5 /clang/lib/Frontend/CompilerInvocation.cpp | |
| parent | 5bdd67778f63b23d055324f9adc45b85a8cfb8c5 (diff) | |
| download | llvm-5d3280c7a7fd9ff33631bf696a383cb758505230.zip llvm-5d3280c7a7fd9ff33631bf696a383cb758505230.tar.gz llvm-5d3280c7a7fd9ff33631bf696a383cb758505230.tar.bz2 | |
x86-32: PUSHF/POPF use/def EFLAGS
Summary: As a side-quest for D6629 jvoung pointed out that I should use -verify-machineinstrs and this found a bug in x86-32's handling of EFLAGS for PUSHF/POPF. This patch fixes the use/def, and adds -verify-machineinstrs to all x86 tests which contain 'EFLAGS'. One exception: this patch leaves inline-asm-fpstack.ll as-is because it fails -verify-machineinstrs in a way unrelated to EFLAGS. This patch also modifies cmpxchg-clobber-flags.ll along the lines of what D6629 already does by also testing i386.
Test Plan: ninja check
Reviewers: t.p.northover, jvoung
Subscribers: llvm-commits
Differential Revision: http://reviews.llvm.org/D6687
llvm-svn: 224359
Diffstat (limited to 'clang/lib/Frontend/CompilerInvocation.cpp')
0 files changed, 0 insertions, 0 deletions