[PATCH] [ARMv8] Add MC assembly/disassembly for VSEL
    Amaury de la Vieuville 
    amaury.dlv at gmail.com
       
    Thu Jul  4 08:13:40 PDT 2013
    
    
  
================
Comment at: lib/Target/ARM/Disassembler/ARMDisassembler.cpp:778
@@ +777,3 @@
+    Size = 4;
+    UpdateThumbVFPPredicate(MI);
+    return result;
----------------
I think this is unnecessary. As you said, these instructions are not predicable, `UpdateThumbVFPPredicate` wouldn't find any predicate to update. However that would still update the IT state despite VSEL being unpredictable inside an IT block (which is not yet enforced).
http://llvm-reviews.chandlerc.com/D1098
    
    
More information about the llvm-commits
mailing list