[PATCH] D59522: [X86] Don't avoid folding multiple use sign extended 8-bit immediate into instructions under optsize.
Sanjay Patel via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Mar 18 17:40:34 PDT 2019
spatel added a comment.
Seems good, but let Simon have a look too in case there's some uarch concern that I'm not aware of.
================
Comment at: test/CodeGen/X86/immediate_merging.ll:14-15
; Test -Os to make sure immediates with multiple users don't get pulled in to
; instructions.
define i32 @foo() optsize {
----------------
Partly stale comment. Add a line for 8-bit exception.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D59522/new/
https://reviews.llvm.org/D59522
More information about the llvm-commits
mailing list