[PATCH] [Hexagon] Handle ENDLOOP0 in InsertBranch and RemoveBranch
Krzysztof Parzyszek
kparzysz at codeaurora.org
Wed Mar 18 09:10:12 PDT 2015
REPOSITORY
rL LLVM
================
Comment at: lib/Target/Hexagon/HexagonInstrInfo.cpp:326-329
@@ +325,6 @@
+ switch (Opc1) {
+ case Hexagon::J2_jump:
+ case Hexagon::J2_jumpt:
+ case Hexagon::J2_jumpf:
+ case Hexagon::ENDLOOP0:
+ I->eraseFromParent();
----------------
djasper wrote:
> LLVM style generally does not indent the case labels.
>
> Do you think that these switches are really more readable? They seem a lot longer...
Lack of phabricator skill. I replied in the top-level comments instead of here.
http://reviews.llvm.org/D8418
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the llvm-commits
mailing list