Skip to content

Conversation

@tqchen
Copy link
Member

@tqchen tqchen commented Oct 26, 2018

cc @jroesch , directly using pointer to refer to the static constexpr variable may not be very well defined? In my local compiler, I got symbol missing error, change to proper static reference

@zhiics
Copy link
Member

zhiics commented Oct 26, 2018

I think use class::static_member is more preferable because these members are actually not associated with a specific object.

@tqchen tqchen merged commit cbf4fdb into apache:master Oct 26, 2018
eqy pushed a commit to eqy/tvm that referenced this pull request Oct 29, 2018
eqy pushed a commit to eqy/tvm that referenced this pull request Oct 29, 2018
@tqchen tqchen deleted the fixhash branch November 2, 2018 20:54
FrozenGene pushed a commit to FrozenGene/tvm that referenced this pull request Dec 27, 2018
wweic pushed a commit to neo-ai/tvm that referenced this pull request Feb 20, 2019
wweic pushed a commit to neo-ai/tvm that referenced this pull request Feb 20, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants