[PATCH] D20841: TII: Add documentation about conditional exits. NFC

Eric Christopher via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 1 17:45:24 PDT 2016


echristo added a subscriber: echristo.
echristo added a reviewer: echristo.
echristo added a comment.

So, I'll propose a different suggestion:

TII::BranchInfo {

  enum { } // Types of branches.
  // Arguments to the function with accessors

}

which encapsulates the existing arguments to make the code a bit more extensible and much easier to read. Then we can extend it for conditional return as a fairly straightforward change.

Sound good?

-eric


Repository:
  rL LLVM

http://reviews.llvm.org/D20841





More information about the llvm-commits mailing list