[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