[PATCH] D49900: [GlobalISel] Add a G_BLOCK_ADDR opcode to handle IR blockaddress constants.
Roman Tereshin via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jul 30 10:52:16 PDT 2018
rtereshin accepted this revision.
rtereshin added a comment.
This revision is now accepted and ready to land.
Besides a nitpick, LGTM
================
Comment at: include/llvm/CodeGen/GlobalISel/MachineIRBuilder.h:952
+ /// \pre \p Res must be a generic virtual register of a pointer type.
+ /// \pre \p BA must be an IR block address value.
+ ///
----------------
If this is evident from the operand type I'd drop the comment. Could a `BlockAddress` be something else, not "an IR block address value"?
Just a nitpick, not blocking.
Repository:
rL LLVM
https://reviews.llvm.org/D49900
More information about the llvm-commits
mailing list