jyknight added a comment.

This change seems like it's done in the wrong layer -- why do we add a 
special-case in Clang to emit "={r11},{r11}", instead of fixing LLVM to not 
break when given the previously-output "={r11},0"?

Furthermore, since earlyclobber was exempted from this change, doesn't the same 
TwoAddressInstructionPass bug still affect earlyclobber uses?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D87279/new/

https://reviews.llvm.org/D87279

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D87279: [clang] F... Nick Desaulniers via Phabricator via cfe-commits
    • [PATCH] D87279: [cla... James Y Knight via Phabricator via cfe-commits

Reply via email to