Page MenuHomePhabricator
Differential D12979 Diff 35516 include/llvm/IR/CallSite.h

Changeset View

Standalone View

View Options

include/llvm/IR/CallSite.h

Show First 20 Lines • Show All 265 Lines • ▼ Show 20 Lines#define CALLSITE_DELEGATE_SETTER(METHOD) \
bool isNoBuiltin() const { bool isNoBuiltin() const {
CALLSITE_DELEGATE_GETTER(isNoBuiltin()); CALLSITE_DELEGATE_GETTER(isNoBuiltin());
} }
/// @brief Return true if the call should not be inlined. /// @brief Return true if the call should not be inlined.
bool isNoInline() const { bool isNoInline() const {
CALLSITE_DELEGATE_GETTER(isNoInline()); CALLSITE_DELEGATE_GETTER(isNoInline());
} }
void setIsNoInline(bool Value = true) { void setIsNoInline() {
CALLSITE_DELEGATE_SETTER(setIsNoInline(Value)); CALLSITE_DELEGATE_SETTER(setIsNoInline());
hfinkelUnsubmitted
Done
ReplyInline Actions

Why are you changing this?

hfinkel: Why are you changing this?
junbumlAuthorUnsubmitted
Not Done
ReplyInline Actions

Both CallInst and InvokeInst have only setIsNoInline(); there is no setIsNoInline(bool) in both CallInst and InvokeInst. I believe it was bug.

junbuml: Both CallInst and InvokeInst have only setIsNoInline(); there is no setIsNoInline(bool) in…
} }
/// @brief Determine if the call does not access memory. /// @brief Determine if the call does not access memory.
bool doesNotAccessMemory() const { bool doesNotAccessMemory() const {
CALLSITE_DELEGATE_GETTER(doesNotAccessMemory()); CALLSITE_DELEGATE_GETTER(doesNotAccessMemory());
} }
void setDoesNotAccessMemory() { void setDoesNotAccessMemory() {
CALLSITE_DELEGATE_SETTER(setDoesNotAccessMemory()); CALLSITE_DELEGATE_SETTER(setDoesNotAccessMemory());
▲ Show 20 Lines • Show All 148 Lines • Show Last 20 Lines